diff options
| author | Aaron Bohy <aab@odoo.com> | 2019-09-06 10:56:46 +0000 |
|---|---|---|
| committer | Aaron Bohy <aab@odoo.com> | 2019-09-06 11:43:40 +0000 |
| commit | 707882497b397ce908c49fe0b9ed0e1911f3e6a4 (patch) | |
| tree | 3134a31c6b5a20e7e138d07e1ed74e2fc3468265 /addons/web/static/tests/fields/basic_fields_tests.js | |
| parent | 7210ff4e19801ef85330e606d14d9caca6c01fb1 (diff) | |
[FIX] web: tests: load template before FieldColor tests
The ColorPickerDialog lazyloads its template. In the test
environment, nothing was done (except for a single nextTick) to
ensure that the template has been loaded before interacting with
the dialog. This led to non deterministic failing runbot builds.
Also correctly load the signature template before running its tests
instead of doing two chelou nextTick();
closes odoo/odoo#36511
Signed-off-by: David Monjoie (dmo) <dmo@odoo.com>
Diffstat (limited to 'addons/web/static/tests/fields/basic_fields_tests.js')
| -rw-r--r-- | addons/web/static/tests/fields/basic_fields_tests.js | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/addons/web/static/tests/fields/basic_fields_tests.js b/addons/web/static/tests/fields/basic_fields_tests.js index 112a1c74b7b..a5620212500 100644 --- a/addons/web/static/tests/fields/basic_fields_tests.js +++ b/addons/web/static/tests/fields/basic_fields_tests.js @@ -1,6 +1,7 @@ odoo.define('web.basic_fields_tests', function (require) { "use strict"; +var ajax = require('web.ajax'); var basicFields = require('web.basic_fields'); var concurrency = require('web.concurrency'); var config = require('web.config'); @@ -13,7 +14,6 @@ var testUtils = require('web.test_utils'); var field_registry = require('web.field_registry'); var createView = testUtils.createView; -var createView = testUtils.createView; var DebouncedField = basicFields.DebouncedField; var JournalDashboardGraph = basicFields.JournalDashboardGraph; var _t = core._t; @@ -5817,7 +5817,11 @@ QUnit.module('basic_fields', { form.destroy(); }); - QUnit.module('FieldColor'); + QUnit.module('FieldColor', { + before: function () { + return ajax.loadXML('/web/static/src/xml/colorpicker_dialog.xml', core.qweb); + }, + }); QUnit.test('Field Color: default widget state', async function (assert) { assert.expect(3); @@ -5837,7 +5841,6 @@ QUnit.module('basic_fields', { }); await testUtils.dom.click(form.$('.o_field_color')); - await testUtils.nextTick(); assert.containsOnce($, '.modal'); assert.containsNone($('.modal'), '.o_opacity_slider', "Opacity slider should not be present"); |
