Skip to content

Commit 580aedc

Browse files
committed
[IMP] website: adapt edit_menus tour
1 parent 6faab47 commit 580aedc

File tree

2 files changed

+9
-29
lines changed

2 files changed

+9
-29
lines changed

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

+9-28
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@ import {
44
clickOnEditAndWaitEditMode,
55
clickOnExtraMenuItem,
66
clickOnSave,
7+
goBackToBlocks,
78
insertSnippet,
9+
openLinkPopup,
810
registerWebsitePreviewTour,
911
} from '@website/js/tours/tour_utils';
1012

@@ -55,24 +57,17 @@ registerWebsitePreviewTour('edit_menus', {
5557
{
5658
trigger: "body:not(:has(.oe_menu_editor))",
5759
},
58-
{
59-
trigger: ":iframe body:contains(welcome to your)",
60-
},
6160
clickOnExtraMenuItem({}, true),
6261
{
6362
content: "There should be a new megamenu item.",
6463
trigger: ':iframe .top_menu .nav-item a.o_mega_menu_toggle:contains("Megaaaaa!")',
6564
},
6665
// Add a menu item in edit mode.
6766
...clickOnEditAndWaitEditMode(),
68-
{
69-
content: "Click on a menu item",
70-
trigger: ":iframe .top_menu .nav-item a",
71-
run: "click",
72-
},
67+
...openLinkPopup(":iframe .top_menu .nav-item a:contains('Home')", "Home"),
7368
{
7469
content: "Click on Edit Menu",
75-
trigger: ':iframe .o_edit_menu_popover a.js_edit_menu',
70+
trigger: '.o-we-linkpopover a.js_edit_menu',
7671
run: "click",
7772
},
7873
{
@@ -127,27 +122,16 @@ registerWebsitePreviewTour('edit_menus', {
127122
{
128123
trigger: "body:not(:has(.modal))",
129124
},
130-
{
131-
trigger: "#oe_snippets.o_loaded",
132-
},
133125
// Edit the new menu item from the "edit link" popover button
134126
clickOnExtraMenuItem({}, true),
135-
{
136-
trigger: ".o_website_preview.editor_enable.editor_has_snippets:not(.o_is_blocked)",
137-
},
138-
{
139-
content: "Menu should have a new link item",
140-
trigger: ':iframe .top_menu .nav-item a:contains("Random!")',
141-
// Don't click the new menu when the editor is still blocked.
142-
run: "click",
143-
},
127+
...openLinkPopup(":iframe .top_menu .nav-item a:contains('Random!')", "Random!"),
144128
{
145129
content: "navbar shouldn't have any zwnbsp and no o_link_in_selection class",
146130
trigger: ':iframe nav.navbar:not(:has(.o_link_in_selection)):not(:contains("\ufeff"))',
147131
},
148132
{
149133
content: "Click on Edit Link",
150-
trigger: ':iframe .o_edit_menu_popover a.o_we_edit_link',
134+
trigger: '.o-we-linkpopover a.o_we_edit_link',
151135
run: "click",
152136
},
153137
{
@@ -169,14 +153,10 @@ registerWebsitePreviewTour('edit_menus', {
169153
// Edit the menu item from the "edit menu" popover button
170154
...clickOnEditAndWaitEditMode(),
171155
clickOnExtraMenuItem({}, true),
172-
{
173-
content: "Click on the 'Modnar' link",
174-
trigger: ':iframe .top_menu .nav-item a:contains("Modnar")',
175-
run: "click",
176-
},
156+
...openLinkPopup(":iframe .top_menu .nav-item a:contains('Modnar')", "Modnar"),
177157
{
178158
content: "Click on the popover Edit Menu button",
179-
trigger: ':iframe .o_edit_menu_popover a.js_edit_menu',
159+
trigger: '.o-we-linkpopover a.js_edit_menu',
180160
run: "click",
181161
},
182162
{
@@ -206,6 +186,7 @@ registerWebsitePreviewTour('edit_menus', {
206186
run: "click",
207187
},
208188
// Drag a block to be able to scroll later.
189+
goBackToBlocks(),
209190
...insertSnippet({ id: "s_media_list", name: "Media List", groupName: "Content" }),
210191
...clickOnSave(),
211192
clickOnExtraMenuItem({}, true),

addons/website/tests/test_ui.py

-1
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,6 @@ def test_16_website_edit_megamenu(self):
517517
def test_website_megamenu_active_nav_link(self):
518518
self.start_tour(self.env['website'].get_client_action_url('/'), 'megamenu_active_nav_link', login='admin')
519519

520-
@unittest.skip
521520
def test_17_website_edit_menus(self):
522521
self.start_tour(self.env['website'].get_client_action_url('/'), 'edit_menus', login='admin')
523522

0 commit comments

Comments
 (0)