[FIX] mail, im_livechat, snailmail: use findFromIdentifyingData

find is O(n) and findFromIdentifyingData is O(1)

task-2440693

closes odoo/odoo#64712

Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>

1 files changed, 33 insertions, 33 deletions

diff --git a/addons/mail/static/src/models/thread/thread_tests.js b/addons/mail/static/src/models/thread/thread_tests.js
index 8f9efe0111f..a535cf4e72f 100644
--- a/addons/mail/static/src/models/thread/thread_tests.js
+++ b/addons/mail/static/src/models/thread/thread_tests.js
@@ -45,12 +45,12 @@ QUnit.test('create (channel)', async function (assert) {
     assert.expect(23);
 
     await this.start();
-    assert.notOk(this.env.models['mail.partner'].find(partner => partner.id === 9));
-    assert.notOk(this.env.models['mail.partner'].find(partner => partner.id === 10));
-    assert.notOk(this.env.models['mail.thread'].find(thread =>
-        thread.id === 100 &&
-        thread.model === 'mail.channel'
-    ));
+    assert.notOk(this.env.models['mail.partner'].findFromIdentifyingData({ id: 9 }));
+    assert.notOk(this.env.models['mail.partner'].findFromIdentifyingData({ id: 10 }));
+    assert.notOk(this.env.models['mail.thread'].findFromIdentifyingData({
+        id: 100,
+        model: 'mail.channel',
+    }));
 
     const thread = this.env.models['mail.thread'].create({
         channel_type: 'channel',
@@ -71,18 +71,18 @@ QUnit.test('create (channel)', async function (assert) {
         serverMessageUnreadCounter: 5,
     });
     assert.ok(thread);
-    assert.ok(this.env.models['mail.partner'].find(partner => partner.id === 9));
-    assert.ok(this.env.models['mail.partner'].find(partner => partner.id === 10));
-    assert.ok(this.env.models['mail.thread'].find(thread =>
-        thread.id === 100 &&
-        thread.model === 'mail.channel'
-    ));
-    const partner9 = this.env.models['mail.partner'].find(partner => partner.id === 9);
-    const partner10 = this.env.models['mail.partner'].find(partner => partner.id === 10);
-    assert.strictEqual(thread, this.env.models['mail.thread'].find(thread =>
-        thread.id === 100 &&
-        thread.model === 'mail.channel'
-    ));
+    assert.ok(this.env.models['mail.partner'].findFromIdentifyingData({ id: 9 }));
+    assert.ok(this.env.models['mail.partner'].findFromIdentifyingData({ id: 10 }));
+    assert.ok(this.env.models['mail.thread'].findFromIdentifyingData({
+        id: 100,
+        model: 'mail.channel',
+    }));
+    const partner9 = this.env.models['mail.partner'].findFromIdentifyingData({ id: 9 });
+    const partner10 = this.env.models['mail.partner'].findFromIdentifyingData({ id: 10 });
+    assert.strictEqual(thread, this.env.models['mail.thread'].findFromIdentifyingData({
+        id: 100,
+        model: 'mail.channel',
+    }));
     assert.strictEqual(thread.model, 'mail.channel');
     assert.strictEqual(thread.channel_type, 'channel');
     assert.strictEqual(thread.id, 100);
@@ -104,11 +104,11 @@ QUnit.test('create (chat)', async function (assert) {
     assert.expect(15);
 
     await this.start();
-    assert.notOk(this.env.models['mail.partner'].find(partner => partner.id === 5));
-    assert.notOk(this.env.models['mail.thread'].find(thread =>
-        thread.id === 200 &&
-        thread.model === 'mail.channel'
-    ));
+    assert.notOk(this.env.models['mail.partner'].findFromIdentifyingData({ id: 5 }));
+    assert.notOk(this.env.models['mail.thread'].findFromIdentifyingData({
+        id: 200,
+        model: 'mail.channel',
+    }));
 
     const channel = this.env.models['mail.thread'].create({
         channel_type: 'chat',
@@ -122,16 +122,16 @@ QUnit.test('create (chat)', async function (assert) {
         model: 'mail.channel',
     });
     assert.ok(channel);
-    assert.ok(this.env.models['mail.thread'].find(thread =>
-        thread.id === 200 &&
-        thread.model === 'mail.channel'
-    ));
-    assert.ok(this.env.models['mail.partner'].find(partner => partner.id === 5));
-    const partner = this.env.models['mail.partner'].find(partner => partner.id === 5);
-    assert.strictEqual(channel, this.env.models['mail.thread'].find(thread =>
-        thread.id === 200 &&
-        thread.model === 'mail.channel'
-    ));
+    assert.ok(this.env.models['mail.thread'].findFromIdentifyingData({
+        id: 200,
+        model: 'mail.channel',
+    }));
+    assert.ok(this.env.models['mail.partner'].findFromIdentifyingData({ id: 5 }));
+    const partner = this.env.models['mail.partner'].findFromIdentifyingData({ id: 5 });
+    assert.strictEqual(channel, this.env.models['mail.thread'].findFromIdentifyingData({
+        id: 200,
+        model: 'mail.channel',
+    }));
     assert.strictEqual(channel.model, 'mail.channel');
     assert.strictEqual(channel.channel_type, 'chat');
     assert.strictEqual(channel.id, 200);