From 5801ff87cf033253fc46a80daa01bbc13c141df7 Mon Sep 17 00:00:00 2001
From: FrancoisGe <fge@odoo.com>
Date: Thu, 2 Dec 2021 13:45:56 +0000
Subject: [PATCH] [FIX] web: MockServer search

The mockSearch function should not return the list of records for a
domain and a model but the list of ids.

The path "/web/dataset/search" does not exist.

closes odoo/odoo#80835

X-original-commit: 5f6d72e43eaec803eea76927433706dfd5e2be73
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
---
 addons/web/static/tests/helpers/mock_server.js | 4 +---
 addons/web/static/tests/mock_server_tests.js   | 6 ++----
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git a/addons/web/static/tests/helpers/mock_server.js b/addons/web/static/tests/helpers/mock_server.js
index 8ed615e7f83..0d51d2cea02 100644
--- a/addons/web/static/tests/helpers/mock_server.js
+++ b/addons/web/static/tests/helpers/mock_server.js
@@ -406,8 +406,6 @@ export class MockServer {
                 return this.mockLoadAction(args);
             case "/web/dataset/search_read":
                 return this.mockSearchReadController(args);
-            case "/web/dataset/search":
-                return this.mockSearchController(args);
         }
         if (
             route.indexOf("/web/image") >= 0 ||
@@ -1530,7 +1528,7 @@ export class MockServer {
             sort: kwargs.order || args[4],
             context: kwargs.context,
         });
-        return result.records;
+        return result.records.map((r) => r.id);
     }
 
     /**
diff --git a/addons/web/static/tests/mock_server_tests.js b/addons/web/static/tests/mock_server_tests.js
index d024183e2c1..c4eefadc5ef 100644
--- a/addons/web/static/tests/mock_server_tests.js
+++ b/addons/web/static/tests/mock_server_tests.js
@@ -87,8 +87,7 @@ QUnit.test("performRPC: search with active_test=false", async function (assert)
             context: { active_test: false },
         },
     });
-    const ids = result.map((record) => record.id);
-    assert.deepEqual(ids, [1, 2]);
+    assert.deepEqual(result, [1, 2]);
 });
 
 QUnit.test("performRPC: search with active_test=true", async function (assert) {
@@ -102,8 +101,7 @@ QUnit.test("performRPC: search with active_test=true", async function (assert) {
             context: { active_test: true },
         },
     });
-    const ids = result.map((record) => record.id);
-    assert.deepEqual(ids, [1]);
+    assert.deepEqual(result, [1]);
 });
 
 QUnit.test("performRPC: search_read with active_test=false", async function (assert) {