[IMP] web: dialog triggers custom callback on close

Added an option in the Dialog class to give it a function to run
when it's closed by other ways than by its buttons.
Its default value is the "cancel" callback if given, or none.

Before this commit, there was no way to add any callback to run
when the modal wasn't closed by a button.

Now, when exiting the modal by other means (e.g.: "Escape" key),
it will run a given callback.

1 files changed, 129 insertions, 0 deletions

diff --git a/addons/web/static/tests/core/dialog_tests.js b/addons/web/static/tests/core/dialog_tests.js
new file mode 100644
index 00000000000..ea3a672322f
--- /dev/null
+++ b/addons/web/static/tests/core/dialog_tests.js
@@ -0,0 +1,129 @@
+odoo.define('web.dialog_tests', function (require) {
+"use strict";
+
+var Dialog = require('web.Dialog');
+var testUtils = require('web.test_utils');
+var Widget = require('web.Widget');
+
+var ESCAPE_KEY = $.Event("keyup", { which: 27 });
+
+function createEmptyParent(debug) {
+    var widget = new Widget();
+
+    testUtils.mock.addMockEnvironment(widget, {
+        debug: debug || false,
+    });
+    return widget;
+}
+
+QUnit.module('core', {}, function () {
+
+    QUnit.module('Dialog');
+
+    QUnit.test("Closing custom dialog using buttons calls standard callback", async function (assert) {
+        assert.expect(3);
+
+        var testPromise = testUtils.makeTestPromiseWithAssert(assert, 'custom callback');
+        var parent = createEmptyParent();
+        new Dialog(parent, {
+            buttons: [
+                {
+                    text: "Close",
+                    classes: 'btn-primary',
+                    close: true,
+                    click: testPromise.resolve,
+                },
+            ],
+            $content: $('<main/>'),
+            onForceClose: testPromise.reject,
+        }).open();
+
+        assert.verifySteps([]);
+
+        await testUtils.nextTick();
+        await testUtils.dom.click($('.modal[role="dialog"] .btn-primary'));
+
+        testPromise.then(() => {
+            assert.verifySteps(['ok custom callback']);
+        });
+
+        parent.destroy();
+    });
+
+    QUnit.test("Closing custom dialog without using buttons calls force close callback", async function (assert) {
+        assert.expect(3);
+
+        var testPromise = testUtils.makeTestPromiseWithAssert(assert, 'custom callback');
+        var parent = createEmptyParent();
+        new Dialog(parent, {
+            buttons: [
+                {
+                    text: "Close",
+                    classes: 'btn-primary',
+                    close: true,
+                    click: testPromise.reject,
+                },
+            ],
+            $content: $('<main/>'),
+            onForceClose: testPromise.resolve,
+        }).open();
+
+        assert.verifySteps([]);
+
+        await testUtils.nextTick();
+        await testUtils.dom.triggerEvents($('.modal[role="dialog"]'), [ESCAPE_KEY]);
+
+        testPromise.then(() => {
+            assert.verifySteps(['ok custom callback']);
+        });
+
+        parent.destroy();
+    });
+
+    QUnit.test("Closing confirm dialog without using buttons calls cancel callback", async function (assert) {
+        assert.expect(3);
+
+        var testPromise = testUtils.makeTestPromiseWithAssert(assert, 'confirm callback');
+        var parent = createEmptyParent();
+        var options = {
+            confirm_callback: testPromise.reject,
+            cancel_callback: testPromise.resolve,
+        };
+        Dialog.confirm(parent, "", options);
+
+        assert.verifySteps([]);
+
+        await testUtils.nextTick();
+        await testUtils.dom.triggerEvents($('.modal[role="dialog"]'), [ESCAPE_KEY]);
+
+        testPromise.then(() => {
+            assert.verifySteps(['ok confirm callback']);
+        });
+
+        parent.destroy();
+    });
+
+    QUnit.test("Closing alert dialog without using buttons calls confirm callback", async function (assert) {
+        assert.expect(3);
+
+        var testPromise = testUtils.makeTestPromiseWithAssert(assert, 'alert callback');
+        var parent = createEmptyParent();
+        var options = {
+            confirm_callback: testPromise.resolve,
+        };
+        Dialog.alert(parent, "", options);
+
+        assert.verifySteps([]);
+
+        await testUtils.nextTick();
+        await testUtils.dom.triggerEvents($('.modal[role="dialog"]'), [ESCAPE_KEY]);
+
+        testPromise.then(() => {
+            assert.verifySteps(['ok alert callback']);
+        });
+
+        parent.destroy();
+    });
+});
+
+});