From 0efd1c3bcebe086ac04e48535173c3b4a8e17c9a Mon Sep 17 00:00:00 2001 From: Pierre Pulinckx Date: Tue, 8 Jul 2025 11:37:37 +0200 Subject: [PATCH] [REM] theme_test_custo: no need waitIframeIsReady In this commit, we remove waitIframeIsReady. This is not usefull until it is checked in tour_step_automatic. We also remove empty run. closes odoo/design-themes#1111 Related: odoo/odoo#217854 Related: odoo/enterprise#89712 Signed-off-by: Quentin Smetz (qsm) --- .../static/tests/tours/theme_menu_hierarchies.js | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/theme_test_custo/static/tests/tours/theme_menu_hierarchies.js b/theme_test_custo/static/tests/tours/theme_menu_hierarchies.js index a8916ba5f..745c331b1 100644 --- a/theme_test_custo/static/tests/tours/theme_menu_hierarchies.js +++ b/theme_test_custo/static/tests/tours/theme_menu_hierarchies.js @@ -1,12 +1,10 @@ /** @odoo-module */ import * as wTourUtils from '@website/js/tours/tour_utils'; -import { stepUtils } from "@web_tour/tour_service/tour_utils"; wTourUtils.registerWebsitePreviewTour('theme_menu_hierarchies', { url: '/example', }, () => [ - stepUtils.waitIframeIsReady(), { content: 'Check Mega Menu is correctly created', trigger: ":iframe .top_menu a.o_mega_menu_toggle", @@ -14,7 +12,6 @@ wTourUtils.registerWebsitePreviewTour('theme_menu_hierarchies', { }, { content: 'Check Mega Menu content', trigger: ":iframe .top_menu div.o_mega_menu.show .fa-cube", - run: () => null, // It's a check. }, { content: 'Check new top level menu is correctly created', trigger: ':iframe .top_menu .nav-item.dropdown .dropdown-toggle:contains("Example 1")', @@ -22,15 +19,12 @@ wTourUtils.registerWebsitePreviewTour('theme_menu_hierarchies', { }, { content: 'Check sub menu are correctly created', trigger: ':iframe .top_menu .dropdown-menu.show a.dropdown-item:contains("Item 1")', - run: () => null, // It's a check. }, { content: 'The new menu hierarchy should not be included in the navbar', trigger: ':iframe body:not(:has(.top_menu a[href="/dogs"]))', - run: () => null, // It's a check. }, { content: 'The new menu hierarchy should be inside the footer', trigger: ':iframe footer ul li a[href="/dogs"]', - run: () => null, // It's a check. }, ...wTourUtils.clickOnEditAndWaitEditMode(), { @@ -40,7 +34,6 @@ wTourUtils.registerWebsitePreviewTour('theme_menu_hierarchies', { }, { content: 'The theme custom footer template should be listed and selected', trigger: '[data-container-title="Footer"] [data-label="Template"] button.btn-secondary svg.theme_test_custo_footer', - run: () => null, // It's a check. }, { content: 'Click on header', trigger: ':iframe header', @@ -48,7 +41,6 @@ wTourUtils.registerWebsitePreviewTour('theme_menu_hierarchies', { }, { content: 'The theme custom header template should be listed and selected', trigger: '[data-container-title="Header"] [data-label="Template"] button.btn-secondary svg.theme_test_custo_header', - run: () => null, // It's a check. }, { content: 'Click on image which has a shape', trigger: ':iframe #wrap .s_text_image img[data-shape]', @@ -56,7 +48,6 @@ wTourUtils.registerWebsitePreviewTour('theme_menu_hierarchies', { }, { content: 'The theme custom "Blob 01" shape should be listed and selected', trigger: '[data-container-title="Image"] [data-label="Shape"] div.dropdown:contains("Blob 01")', - run: () => null, // It's a check. }, { content: 'Click on section which has a bg shape', trigger: ':iframe #wrap .s_text_block[data-oe-shape-data]', @@ -64,6 +55,5 @@ wTourUtils.registerWebsitePreviewTour('theme_menu_hierarchies', { }, { content: 'The theme custom "Curve 01" shape should be listed and selected', trigger: '[data-container-title="Text"] [data-label="Shape"] button.btn-secondary:contains("Curve 01")', - run: () => null, // It's a check. }, ]);