[IMP] website: adapt snippet_empty_parent_autoremove tour

Author: divy-odoo

addons/html_builder/static/src/core/remove_plugin.js

@@ -34,7 +34,12 @@ export function isRemovable(el) {
     return !unremovableNodePredicates.some((p) => p(el));
 }
 
-const layoutElementsSelector = [".o_we_shape", ".o_we_bg_filter"].join(",");
+const layoutElementsSelector = [
+    ".o_we_shape",
+    ".o_we_bg_filter",
+    ".s_parallax_bg",
+    ".o_bg_video_container",
+].join(",");
 
 export class RemovePlugin extends Plugin {
     static id = "remove";

addons/website/static/tests/tours/snippet_empty_parent_autoremove.js

@@ -3,78 +3,86 @@ import {
     clickOnSnippet,
     insertSnippet,
     registerWebsitePreviewTour,
+    changeOptionInPopover,
 } from "@website/js/tours/tour_utils";
 
 function removeSelectedBlock() {
     return {
         content: "Remove selected block",
-        trigger: '#oe_snippets we-customizeblock-options:nth-last-child(3) .oe_snippet_remove',
+        trigger: ".o-overlay-container .o_overlay_options .oe_snippet_remove",
         run: "click",
     };
 }
 
-registerWebsitePreviewTour('snippet_empty_parent_autoremove', {
-    url: '/',
-    edition: true,
-}, () => [
-    // Base case: remove both columns from text - image
-    ...insertSnippet({
-        id: 's_text_image',
-        name: 'Text - Image',
-        groupName: "Content",
-    }),
+registerWebsitePreviewTour(
+    "snippet_empty_parent_autoremove",
     {
-        content: "Click on second column",
-        trigger: ':iframe #wrap .s_text_image .row > :nth-child(2)',
-        run: "click",
-    },
-    removeSelectedBlock(),
-    {
-        content: "Click on first column",
-        trigger: ':iframe #wrap .s_text_image .row > :first-child',
-        run: "click",
-    },
-    removeSelectedBlock(),
-    {
-        content: "Check that #wrap is empty",
-        trigger: ':iframe #wrap:empty',
-    },
-
-    // Cover: test that parallax, bg-filter and shape are not treated as content
-    ...insertSnippet({
-        id: 's_cover',
-        name: 'Cover',
-        groupName: "Intro",
-    }),
-    ...clickOnSnippet({
-        id: 's_cover',
-        name: 'Cover',
-    }),
-    // Add a shape
-    changeOption('ColoredLevelBackground', 'Shape'),
-    {
-        content: "Check that the parallax element is present",
-        trigger: ':iframe #wrap .s_cover .s_parallax_bg',
-    },
-    {
-        content: "Check that the filter element is present",
-        trigger: ':iframe #wrap .s_cover .o_we_bg_filter',
-    },
-    {
-        content: "Check that the shape element is present",
-        trigger: ':iframe #wrap .s_cover .o_we_shape',
-    },
-    // Add a column
-    changeOption('layout_column', 'we-toggler'),
-    changeOption('layout_column', '[data-select-count="1"]'),
-    {
-        content: "Click on the created column",
-        trigger: ':iframe #wrap .s_cover .row > :first-child',
-        run: "click",
-    },
-    removeSelectedBlock(),
-    {
-        content: "Check that #wrap is empty",
-        trigger: ':iframe #wrap:empty',
+        url: "/",
+        edition: true,
     },
-]);
+    () => [
+        // Base case: remove both columns from text - image
+        ...insertSnippet({
+            id: "s_text_image",
+            name: "Text - Image",
+            groupName: "Content",
+        }),
+        {
+            content: "Click on second column",
+            trigger: ":iframe #wrap .s_text_image .row > :nth-child(2)",
+            run: "click",
+        },
+        removeSelectedBlock(),
+        {
+            content: "Click on first column",
+            trigger: ":iframe #wrap .s_text_image .row > :first-child",
+            run: "click",
+        },
+        removeSelectedBlock(),
+        {
+            content: "Check that #wrap is empty",
+            trigger: ":iframe #wrap:empty",
+        },
+        // Cover: test that parallax, bg-filter and shape are not treated as content
+        ...insertSnippet({
+            id: "s_cover",
+            name: "Cover",
+            groupName: "Intro",
+        }),
+        ...clickOnSnippet({
+            id: "s_cover",
+            name: "Cover",
+        }),
+        // Add a shape
+        changeOption("Cover", "toggleBgShape"),
+        {
+            content: "Click on the back button",
+            trigger: ".o_pager_nav_angle",
+            run: "click",
+        },
+        {
+            content: "Check that the parallax element is present",
+            trigger: ":iframe #wrap .s_cover .s_parallax_bg",
+        },
+        {
+            content: "Check that the filter element is present",
+            trigger: ":iframe #wrap .s_cover .o_we_bg_filter",
+        },
+        {
+            content: "Check that the shape element is present",
+            trigger: ":iframe #wrap .s_cover .o_we_shape",
+        },
+        // Add a column
+        ...changeOptionInPopover("Cover", "Layout", "[data-action-value='1']"),
+        {
+            content: "Click on the created column",
+            trigger: ":iframe #wrap .s_cover .row > :first-child",
+            run: "click",
+        },
+        removeSelectedBlock(),
+        {
+            content: "Check that #wrap is empty",
+            trigger: ":iframe #wrap:empty",
+        },
+    ]
+);

addons/website/tests/test_snippets.py

@@ -22,7 +22,6 @@ def fetch_proxy(self, url):
             return self.make_fetch_proxy_response('<body>Dummy page</body>')
         return super().fetch_proxy(url)
 
-    @unittest.skip
     def test_01_empty_parents_autoremove(self):
         self.start_tour(self.env['website'].get_client_action_url('/'), 'snippet_empty_parent_autoremove', login='admin')