[FIX] mail, *: fix opens and redirects, adapt messaging menu close

task-2244279
task-2262862
task-2274993
task-2277537
task-2282238

closes odoo/odoo#56745

X-original-commit: 109f202
Related: odoo/enterprise#12761
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>

2 files changed, 3 insertions, 14 deletions

diff --git a/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer.js b/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer.js
index 4118b5228ef..9b06fbbaa1c 100644
--- a/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer.js
+++ b/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer.js
@@ -12,9 +12,6 @@ registerInstancePatchModel('mail.messaging_initializer', 'mail_bot/static/src/mo
      * @private
      */
     async _initializeOdooBot() {
-        await this.async(() => new Promise(resolve =>
-            this.env.browser.setTimeout(resolve, 2 * 60 * 1000)
-        ));
         const data = await this.async(() => this.env.services.rpc({
             model: 'mail.channel',
             method: 'init_odoobot',
diff --git a/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer_tests.js b/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer_tests.js
index 4121cc5a277..bfb97067e90 100644
--- a/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer_tests.js
+++ b/addons/mail_bot/static/src/models/messaging_initializer/messaging_initializer_tests.js
@@ -24,10 +24,10 @@ QUnit.module('messaging_initializer_tests.js', {
 });
 
 
-QUnit.test('OdooBot initialized after 2 minutes', async function (assert) {
+QUnit.test('OdooBot initialized at init', async function (assert) {
     // TODO this test should be completed in combination with
     // implementing _mockMailChannelInitOdooBot task-2300480
-    assert.expect(3);
+    assert.expect(2);
 
     await this.start({
         env: {
@@ -35,7 +35,6 @@ QUnit.test('OdooBot initialized after 2 minutes', async function (assert) {
                 odoobot_initialized: false,
             },
         },
-        hasTimeControl: true,
         async mockRPC(route, args) {
             if (args.method === 'init_odoobot') {
                 assert.step('init_odoobot');
@@ -44,16 +43,9 @@ QUnit.test('OdooBot initialized after 2 minutes', async function (assert) {
         },
     });
 
-    await this.env.testUtils.advanceTime(119 * 1000);
-    assert.verifySteps(
-        [],
-        "should not have initialized OdooBot after 1 minute 59 seconds"
-    );
-
-    await this.env.testUtils.advanceTime(1 * 1000);
     assert.verifySteps(
         ['init_odoobot'],
-        "should have initialized OdooBot after 2 minutes"
+        "should have initialized OdooBot at init"
     );
 });