From 52b2edf62338f5fc60e1982ae33dea486eed9047 Mon Sep 17 00:00:00 2001 From: Xavier Morel Date: Mon, 18 Oct 2021 12:59:23 +0000 Subject: [PATCH] [IMP] point_of_sale: cleanup unnecessary testing content * remove `component_extension_tests`, that's a test from web and already tested by web, no idea why it's rerunning in POS * remove all failfast specs, after discussion with ged, seb, and xdo it was considered not useful: it's mostly for the runbot but turns out to be pretty minor as far as positives go, and it makes fixing errors much harder Part-of: odoo/odoo#77735 --- addons/point_of_sale/__manifest__.py | 1 - addons/point_of_sale/tests/test_js.py | 5 +---- addons/web/tests/test_js.py | 4 ++-- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/addons/point_of_sale/__manifest__.py b/addons/point_of_sale/__manifest__.py index 2d39e2d11be..d306e9f47be 100644 --- a/addons/point_of_sale/__manifest__.py +++ b/addons/point_of_sale/__manifest__.py @@ -204,7 +204,6 @@ ('include', 'web.frontend_legacy'), ], 'point_of_sale.qunit_suite_tests': [ - 'web/static/tests/legacy/component_extension_tests.js', 'point_of_sale/static/tests/unit/**/*', ], 'point_of_sale.assets_backend_prod_only': [ diff --git a/addons/point_of_sale/tests/test_js.py b/addons/point_of_sale/tests/test_js.py index 560e4d402e7..02c4746a4fc 100644 --- a/addons/point_of_sale/tests/test_js.py +++ b/addons/point_of_sale/tests/test_js.py @@ -1,7 +1,6 @@ # -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. -import odoo.addons.web.tests.test_js import odoo.tests @@ -17,6 +16,4 @@ class WebSuite(odoo.tests.HttpCase): self.main_pos_config.open_session_cb(check_coa=False) # point_of_sale desktop test suite - self.browser_js( - "/pos/ui/tests?mod=web&failfast", "", "", login="admin", timeout=1800 - ) + self.browser_js("/pos/ui/tests?mod=web", "", "", login="admin", timeout=1800) diff --git a/addons/web/tests/test_js.py b/addons/web/tests/test_js.py index 4e8ffe71736..5bc383a3884 100644 --- a/addons/web/tests/test_js.py +++ b/addons/web/tests/test_js.py @@ -12,7 +12,7 @@ class WebSuite(odoo.tests.HttpCase): def test_js(self): # webclient desktop test suite - self.browser_js('/web/tests?mod=web&failfast', "", "", login='admin', timeout=1800) + self.browser_js('/web/tests?mod=web', "", "", login='admin', timeout=1800) def test_check_suite(self): # verify no js test is using `QUnit.only` as it forbid any other test to be executed @@ -43,4 +43,4 @@ class MobileWebSuite(odoo.tests.HttpCase): def test_mobile_js(self): # webclient mobile test suite - self.browser_js('/web/tests/mobile?mod=web&failfast', "", "", login='admin', timeout=1800) + self.browser_js('/web/tests/mobile?mod=web', "", "", login='admin', timeout=1800)