diff --git a/addons/web/static/src/js/views/kanban/kanban_column.js b/addons/web/static/src/js/views/kanban/kanban_column.js index 31e2e94ca1b..66fb538872c 100644 --- a/addons/web/static/src/js/views/kanban/kanban_column.js +++ b/addons/web/static/src/js/views/kanban/kanban_column.js @@ -251,7 +251,7 @@ var KanbanColumn = Widget.extend({ * @private */ _onCancelQuickCreate: function () { - this.cancelQuickCreate(); + this._cancelQuickCreate(); }, /** * @private diff --git a/addons/web/static/test/views/kanban_tests.js b/addons/web/static/test/views/kanban_tests.js index 2d837d8e541..2b882cd7aa6 100644 --- a/addons/web/static/test/views/kanban_tests.js +++ b/addons/web/static/test/views/kanban_tests.js @@ -319,7 +319,7 @@ QUnit.module('Views', { }); QUnit.test('quick create in grouped mode', function (assert) { - assert.expect(3); + assert.expect(4); var kanban = createView({ View: KanbanView, @@ -333,19 +333,27 @@ QUnit.module('Views', { groupBy: ['bar'], }); + // click to add an element and cancel the quick creation kanban.$('.o_kanban_header .o_kanban_quick_add i').first().click(); var $quickCreate = kanban.$('.o_kanban_quick_create'); assert.strictEqual($quickCreate.length, 1, "should have a quick create element"); + $quickCreate.find('input').trigger($.Event('keydown', {keyCode: $.ui.keyCode.ESCAPE})); + assert.strictEqual(kanban.$('.o_kanban_quick_create').length, 0, + "should have destroyed the quick create element"); + + // click to really add an element + kanban.$('.o_kanban_header .o_kanban_quick_add i').first().click(); + $quickCreate = kanban.$('.o_kanban_quick_create'); $quickCreate.find('input').val('new partner'); $quickCreate.find('button.o_kanban_add').click(); assert.strictEqual(this.data.partner.records.length, 5, "should have created a partner"); - assert.strictEqual(_.last(this.data.partner.records).name, "new partner", "should have correct name"); + kanban.destroy(); });