[FIX] web: Call on_attach_callback on systray widgets

Currently, the method `on_attach_callback` is never called on any Widget in the
systray.

Calling `on_attach_callback` is required if any systray widget uses an OWL
component via the ComponentWrapper. Without the callback, the component is never
considered mounted to the DOM and no event handler is triggered.

closes odoo/odoo#46566

Related: odoo/enterprise#8887
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>

1 files changed, 47 insertions, 0 deletions

diff --git a/addons/web/static/tests/chrome/menu_tests.js b/addons/web/static/tests/chrome/menu_tests.js
new file mode 100644
index 00000000000..c7ce40b16c2
--- /dev/null
+++ b/addons/web/static/tests/chrome/menu_tests.js
@@ -0,0 +1,47 @@
+odoo.define('web.menu_tests', function (require) {
+    "use strict";
+
+    const testUtils = require('web.test_utils');
+    const Menu = require('web.Menu');
+    const SystrayMenu = require('web.SystrayMenu');
+    const Widget = require('web.Widget');
+
+
+    QUnit.module('chrome', {}, function () {
+        QUnit.module('Menu');
+
+        QUnit.test('Systray on_attach_callback is called', async function (assert) {
+            assert.expect(4);
+
+            const parent = testUtils.createParent({});
+
+            // Add some widgets to the systray
+            const Widget1 = Widget.extend({
+                on_attach_callback: () => assert.step('on_attach_callback widget1')
+            });
+            const Widget2 = Widget.extend({
+                on_attach_callback: () => assert.step('on_attach_callback widget2')
+            });
+            SystrayMenu.Items = [Widget1, Widget2]
+
+            testUtils.mock.patch(SystrayMenu, {
+                on_attach_callback: function () {
+                    assert.step('on_attach_callback systray');
+                    this._super(...arguments);
+                }
+            });
+
+            const menu = new Menu(parent, {children: []});
+            await menu.appendTo($('#qunit-fixture'));
+
+            assert.verifySteps([
+                'on_attach_callback systray',
+                'on_attach_callback widget1',
+                'on_attach_callback widget2',
+            ]);
+            testUtils.mock.unpatch(SystrayMenu);
+            parent.destroy();
+        });
+    });
+
+});