From f1a0a32e4680e44a811905aeb7d29576afd1cc7b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexandre=20K=C3=BChn?= Date: Thu, 16 Aug 2018 18:33:19 +0200 Subject: [FIX] web:tests: do not rely on `testUtils.patch` in tests Revision on https://github.com/odoo/odoo/commit/10f43955d81ff3e7b5ab050656b312ec19f927c5 Commit above uses `testUtils.patch` in the test. However, this utility function does not seem to work. That sucks... --- addons/web/static/tests/fields/basic_fields_tests.js | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) (limited to 'addons/web/static/tests/fields/basic_fields_tests.js') diff --git a/addons/web/static/tests/fields/basic_fields_tests.js b/addons/web/static/tests/fields/basic_fields_tests.js index 68643f7b48d..3c942ae3041 100644 --- a/addons/web/static/tests/fields/basic_fields_tests.js +++ b/addons/web/static/tests/fields/basic_fields_tests.js @@ -2018,14 +2018,14 @@ QUnit.module('basic_fields', { var done = assert.async(); assert.expect(6); - testUtils.patch(JournalDashboardGraph, { - on_attach_callback: function () { - assert.step('on_attach_callback'); - }, - on_detach_callback: function () { - assert.step('on_detach_callback'); - }, - }); + var originalAttachCallback = JournalDashboardGraph.prototype.on_attach_callback; + var originalDetachCallback = JournalDashboardGraph.prototype.on_detach_callback; + JournalDashboardGraph.prototype.on_attach_callback = function () { + assert.step('on_attach_callback'); + }; + JournalDashboardGraph.prototype.on_detach_callback = function () { + assert.step('on_detach_callback'); + }; createAsyncView({ View: KanbanView, @@ -2056,7 +2056,8 @@ QUnit.module('basic_fields', { ]); kanban.destroy(); - testUtils.unpatch(JournalDashboardGraph); + JournalDashboardGraph.prototype.on_attach_callback = originalAttachCallback; + JournalDashboardGraph.prototype.on_detach_callback = originalDetachCallback; done(); }); }); -- cgit v1.2.3