From 6d4eb69ef3ddefe4698a338eee7ac946446f6b86 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?G=C3=A9ry=20Debongnie?= Date: Fri, 20 Apr 2018 10:36:18 +0200 Subject: [PATCH] [FIX] barcodes: update tests to new notification service The notification manager was removed and replaced by a notification service. Some tests were forward ported after this change, so they need to be updated to the new system. --- addons/barcodes/static/tests/barcode_tests.js | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/addons/barcodes/static/tests/barcode_tests.js b/addons/barcodes/static/tests/barcode_tests.js index b2a2bf46126..de88e79b9fc 100644 --- a/addons/barcodes/static/tests/barcode_tests.js +++ b/addons/barcodes/static/tests/barcode_tests.js @@ -8,6 +8,7 @@ var fieldRegistry = require('web.field_registry'); var FormController = require('web.FormController'); var FormView = require('web.FormView'); var testUtils = require('web.test_utils'); +var NotificationService = require('web.NotificationService'); var createView = testUtils.createView; var triggerKeypressEvent = testUtils.triggerKeypressEvent; @@ -64,14 +65,16 @@ QUnit.test('Button with barcode_trigger', function (assert) { '' + '', res_id: 2, + services: [NotificationService.extend({ + notify: function (params) { + assert.step(params.type); + } + })], intercepts: { execute_action: function (event) { assert.strictEqual(event.data.action_data.name, 'do_something', "do_something method call verified"); }, - warning: function () { - assert.step('warn'); - }, }, }); @@ -509,11 +512,11 @@ QUnit.test('barcode_scanned only trigger error for active view', function (asser '', }, res_id: 1, - intercepts: { - warning: function (event) { - assert.step(event.name); + services: [NotificationService.extend({ + notify: function (params) { + assert.step(params.type); } - }, + })], viewOptions: { mode: 'edit', },