From fff405a6c2a88f2ff511ceae1ca111ababf935bd Mon Sep 17 00:00:00 2001 From: Antoine Guenet Date: Thu, 31 Aug 2023 15:08:55 +0200 Subject: [PATCH] [FIX] web_editor: temporarily skip mobile tests The editor mobile tests are making wrong assumptions about the way the editor behaves on mobile and are therefore wrong. This temporarily disables them until the discrepancy is resolved. Part-of: odoo/odoo#141303 --- .../static/src/js/editor/odoo-editor/test/utils.js | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/addons/web_editor/static/src/js/editor/odoo-editor/test/utils.js b/addons/web_editor/static/src/js/editor/odoo-editor/test/utils.js index e79d79910a8..97cd5c09a20 100644 --- a/addons/web_editor/static/src/js/editor/odoo-editor/test/utils.js +++ b/addons/web_editor/static/src/js/editor/odoo-editor/test/utils.js @@ -395,7 +395,19 @@ export async function testEditor(Editor = OdooEditor, spec, options = {}) { if (error) { throw error; } else if (hasMobileTest && !isMobileTest) { - await testEditor(Editor, spec, { ...options, isMobile: true }); + const li = document.createElement('li'); + li.classList.add('test', 'pass', 'pending'); + const h2 = document.createElement('h2'); + h2.textContent = 'FIXME: [Mobile Test] skipped'; + li.append(h2); + const mochaSuite = [...document.querySelectorAll('#mocha-report li.suite > ul')].pop(); + if (mochaSuite) { + mochaSuite.append(li); + } + // Mobile tests are temporarily disabled because they are not + // representative of reality. They will be re-enabled when the mobile + // editor will be ready. + // await testEditor(Editor, spec, { ...options, isMobile: true }); } }