[FIX] website_slides: fix resequencing based on category

When new slides are added through frontend a resequencing of slides and
categories sequence is done by calling _resequence_slides on the channel.
However this method is not working as expected, and resequencing should
take a category in order to better know where to start its resequencing.
Otherwise it simply reorder slides in the same order as before, or in an
unpredicted or prodicted but fasly order if new slide category has just been
created.

Task ID 2090927

X-original-commit: f038946ae2e3f174bee086281a541c6bc770e354

1 files changed, 18 insertions, 6 deletions

diff --git a/addons/website_slides/tests/test_slide_utils.py b/addons/website_slides/tests/test_slide_utils.py
index 158d59e94a3..489f2de0d0f 100644
--- a/addons/website_slides/tests/test_slide_utils.py
+++ b/addons/website_slides/tests/test_slide_utils.py
@@ -55,20 +55,32 @@ class TestSequencing(slides_common.SlidesCase):
 
     @users('user_officer')
     def test_resequence(self):
+        self.assertEqual(self.slide.sequence, 1)
         self.category.write({'sequence': 4})
         self.slide_2.write({'sequence': 8})
         self.slide_3.write({'sequence': 3})
 
         self.channel.invalidate_cache()
         self.assertEqual([s.id for s in self.channel.slide_ids], [self.slide.id, self.slide_3.id, self.category.id, self.slide_2.id])
+        self.assertEqual(self.slide.sequence, 1)
 
+        # insert a new category and check resequence_slides does as expected
+        new_category = self.env['slide.slide'].create({
+            'name': 'Sub-cooking Tips Category',
+            'channel_id': self.channel.id,
+            'is_category': True,
+            'is_published': True,
+            'sequence': 2,
+        })
+        new_category.flush()
+        self.channel.invalidate_cache()
+        self.channel._resequence_slides(self.slide_3, force_category=new_category)
         self.assertEqual(self.slide.sequence, 1)
-        # self.channel._resequence_slides(self.slide)
-        # self.channel.invalidate_cache()
-        # # self.assertEqual([s.id for s in self.channel.slide_ids], [self.slide.id, self.slide_3.id, self.category.id, self.slide_2.id])
-        # self.assertEqual(self.slide_3.sequence, 2)
-        # self.assertEqual(self.category.sequence, 3)
-        # self.assertEqual(self.slide_2.sequence, 4)
+        self.assertEqual(new_category.sequence, 2)
+        self.assertEqual(self.slide_3.sequence, 3)
+        self.assertEqual(self.category.sequence, 4)
+        self.assertEqual(self.slide_2.sequence, 5)
+        self.assertEqual([s.id for s in self.channel.slide_ids], [self.slide.id, new_category.id, self.slide_3.id, self.category.id, self.slide_2.id])
 
 
 class TestFromURL(slides_common.SlidesCase):