From 6feb1ff1f2ed6946f7a559b1546b27b9eca9700f Mon Sep 17 00:00:00 2001 From: "Audric Onockx (auon)" Date: Mon, 22 May 2023 13:23:42 +0000 Subject: [PATCH] [REF] web: adapt uncorrect onchange test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit In previous commit (https://github.com/odoo/odoo/pull/119214), this line was added to make the block onchange and check if the view was rerendered correctly, but having `await nextTick()` inside an RPC doesn't make sense. So we use `makeDeferrer()` which is the tool provided for that matter. closes odoo/odoo#122110 X-original-commit: a1056b1299f1edae736ea1ac00b6895e1c758104 Signed-off-by: Georis François (fge) --- addons/web/static/tests/views/list_view_tests.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/addons/web/static/tests/views/list_view_tests.js b/addons/web/static/tests/views/list_view_tests.js index 467964bf8f1..36e0c9818b9 100644 --- a/addons/web/static/tests/views/list_view_tests.js +++ b/addons/web/static/tests/views/list_view_tests.js @@ -18592,6 +18592,7 @@ QUnit.module("Views", (hooks) => { }); QUnit.test("x2many onchange, check result", async function (assert) { + const def = makeDeferred(); serverData.models.foo.onchanges = { m2m: function () {}, }; @@ -18607,7 +18608,7 @@ QUnit.module("Views", (hooks) => { async mockRPC(route, args) { if (args.method === "onchange") { assert.step("onchange"); - await nextTick(); + await def; return { value: { m2o: [3, "Value 3"] } }; } }, @@ -18626,6 +18627,9 @@ QUnit.module("Views", (hooks) => { assert.verifySteps(["onchange"]); await click(target.querySelector(".o_list_button_save:not(.btn-link)")); + def.resolve(); + await nextTick(); + assert.strictEqual( target.querySelector(".o_data_cell.o_many2many_tags_cell").textContent, "Value 1Value 2Value 3"