[IMP] web, *: improve mock server and replace most mail mockRPC

Improve mock server:

- add support for mocked `fetch`
- add support for `active_test`
- add support for x2m `in` in domains
- add support for default values computed from a function
- implement a more natural "next id" compute
- allow initial data without ids
- ensure write and x2m commands integrity
- improve bad data/bad commands error messages
- always warn for failing RPC, not only in debug mode
- fix all existing tests that had inconsistency data

Other changes done in mail (or dependents) that are not just related to tests:

- remove `direct_partner` from formatter result
  ->`correspondent` can be computed from other keys, especially `members`

- fix `livechat_visitor` convertData
  -> only process if there is value

- add `current_partner` and `current_user_id` as `init_messaging` result
  -> easier to mock than session

- remove usage of `need_moderation`
  -> that was just a search indirection to `moderation_status`

- adapt `partner_id` -> `res_partner_id` key in `_notification_format`
  -> to be consistent with field name

- add name in result of `mail_partner_format`
  -> sometimes display_name is not the same

- remove usage of `is_moderator`
  -> that was just an indirection to `moderation_channel_ids`

Enterprise counterpart: https://github.com/odoo/enterprise/pull/11523
task-2287171

closes odoo/odoo#55854

X-original-commit: 7ba3fecb3377a720d1eb70e7515a0c45da73836d
Related: odoo/enterprise#12391
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>

1 files changed, 7 insertions, 11 deletions

diff --git a/addons/mail/static/src/utils/timer/timer_tests.js b/addons/mail/static/src/utils/timer/timer_tests.js
index 9b464ac9396..e2d33e91585 100644
--- a/addons/mail/static/src/utils/timer/timer_tests.js
+++ b/addons/mail/static/src/utils/timer/timer_tests.js
@@ -1,13 +1,8 @@
 odoo.define('mail/static/src/utils/timer/timer_tests.js', function (require) {
 'use strict';
 
-const {
-    afterEach: utilsAfterEach,
-    beforeEach: utilsBeforeEach,
-    start,
-} = require('mail/static/src/utils/test_utils.js');
+const { afterEach, beforeEach, nextTick, start } = require('mail/static/src/utils/test_utils.js');
 const Timer = require('mail/static/src/utils/timer/timer.js');
-const { nextTick } = require('mail/static/src/utils/test_utils.js');
 
 const { TimerClearedError } = Timer;
 
@@ -16,23 +11,24 @@ QUnit.module('utils', {}, function () {
 QUnit.module('timer', {}, function () {
 QUnit.module('timer_tests.js', {
     beforeEach() {
-        utilsBeforeEach(this);
+        beforeEach(this);
         this.timers = [];
 
-        this.start = async (...args) => {
-            const { env, widget } = await start(...args);
+        this.start = async (params) => {
+            const { env, widget } = await start(Object.assign({}, params, {
+                data: this.data,
+            }));
             this.env = env;
             this.widget = widget;
         };
     },
     afterEach() {
-        utilsAfterEach(this);
         // Important: tests should cleanly intercept cancelation errors that
         // may result from this teardown.
-        this.widget.destroy();
         for (const timer of this.timers) {
             timer.clear();
         }
+        afterEach(this);
     },
 });