From d9d0b841523d281b2088b559488026c01e02ec22 Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Tue, 18 Jul 2023 09:26:54 +0100
Subject: [PATCH] Revert "Do not filter users post search (#9556)" (#11288)

---
 cypress/e2e/spotlight/spotlight.spec.ts       | 17 ++++++++++-----
 .../dialogs/spotlight/SpotlightDialog.tsx     |  3 +--
 .../views/dialogs/SpotlightDialog-test.tsx    | 21 -------------------
 3 files changed, 13 insertions(+), 28 deletions(-)

diff --git a/cypress/e2e/spotlight/spotlight.spec.ts b/cypress/e2e/spotlight/spotlight.spec.ts
index 2274fe75dd..507fc2d75f 100644
--- a/cypress/e2e/spotlight/spotlight.spec.ts
+++ b/cypress/e2e/spotlight/spotlight.spec.ts
@@ -345,7 +345,8 @@ describe("Spotlight", () => {
                 cy.spotlightFilter(Filter.People);
                 cy.spotlightSearch().clear().type(bot2Name);
                 cy.spotlightResults().should("have.length", 1);
-                cy.spotlightResults().contains(bot2Name).click();
+                cy.spotlightResults().eq(0).should("contain", bot2Name);
+                cy.spotlightResults().eq(0).click();
             })
             .then(() => {
                 cy.roomHeaderName().should("contain", bot2Name);
@@ -360,7 +361,9 @@ describe("Spotlight", () => {
         cy.openSpotlightDialog().within(() => {
             cy.spotlightFilter(Filter.People);
             cy.spotlightSearch().clear().type(bot2Name);
-            cy.spotlightResults().contains(bot2Name).click();
+            cy.spotlightResults().should("have.length", 1);
+            cy.spotlightResults().eq(0).should("contain", bot2Name);
+            cy.spotlightResults().eq(0).click();
         });
 
         // Send first message to actually start DM
@@ -387,10 +390,11 @@ describe("Spotlight", () => {
                     cy.spotlightFilter(Filter.People);
                     cy.spotlightSearch().clear().type(bot1.getUserId());
                     cy.wait(1000); // wait for the dialog code to settle
+                    cy.spotlightResults().should("have.length", 2);
                     cy.contains(
                         ".mx_SpotlightDialog_section.mx_SpotlightDialog_results .mx_SpotlightDialog_option",
                         groupDm.name,
-                    ).should("exist");
+                    );
                 });
 
                 // Search for ByteBot by id, should return group DM and user
@@ -398,10 +402,11 @@ describe("Spotlight", () => {
                     cy.spotlightFilter(Filter.People);
                     cy.spotlightSearch().clear().type(bot2.getUserId());
                     cy.wait(1000); // wait for the dialog code to settle
+                    cy.spotlightResults().should("have.length", 2);
                     cy.contains(
                         ".mx_SpotlightDialog_section.mx_SpotlightDialog_results .mx_SpotlightDialog_option",
                         groupDm.name,
-                    ).should("exist");
+                    );
                 });
             });
     });
@@ -431,7 +436,9 @@ describe("Spotlight", () => {
             .within(() => {
                 cy.spotlightFilter(Filter.People);
                 cy.spotlightSearch().clear().type(bot2Name);
-                cy.spotlightResults().should("contain", bot2Name);
+                cy.wait(3000); // wait for the dialog code to settle
+                cy.spotlightResults().should("have.length", 1);
+                cy.spotlightResults().eq(0).should("contain", bot2Name);
                 cy.get(".mx_SpotlightDialog_startGroupChat").should("contain", "Start a group chat");
                 cy.get(".mx_SpotlightDialog_startGroupChat").click();
             })
diff --git a/src/components/views/dialogs/spotlight/SpotlightDialog.tsx b/src/components/views/dialogs/spotlight/SpotlightDialog.tsx
index 6d3cc56f33..b5baa4e253 100644
--- a/src/components/views/dialogs/spotlight/SpotlightDialog.tsx
+++ b/src/components/views/dialogs/spotlight/SpotlightDialog.tsx
@@ -399,8 +399,7 @@ const SpotlightDialog: React.FC<IProps> = ({ initialText = "", initialFilter = n
                     )
                         return; // bail, does not match query
                 } else if (isMemberResult(entry)) {
-                    // Do not filter users as we rely on the server to filter them for us.
-                    // The server may filter based on fields we do not have access to, e.g. e-mail addresses.
+                    if (!entry.query?.some((q) => q.includes(lcQuery))) return; // bail, does not match query
                 } else if (isPublicRoomResult(entry)) {
                     if (!entry.query?.some((q) => q.includes(lcQuery))) return; // bail, does not match query
                 } else {
diff --git a/test/components/views/dialogs/SpotlightDialog-test.tsx b/test/components/views/dialogs/SpotlightDialog-test.tsx
index d842ecb727..2857225309 100644
--- a/test/components/views/dialogs/SpotlightDialog-test.tsx
+++ b/test/components/views/dialogs/SpotlightDialog-test.tsx
@@ -338,27 +338,6 @@ describe("Spotlight Dialog", () => {
         });
     });
 
-    it("should not filter out users sent by the server", async () => {
-        mocked(mockedClient.searchUserDirectory).mockResolvedValue({
-            results: [
-                { user_id: "@user1:server", display_name: "User Alpha", avatar_url: "mxc://1/avatar" },
-                { user_id: "@user2:server", display_name: "User Beta", avatar_url: "mxc://2/avatar" },
-            ],
-            limited: false,
-        });
-
-        render(<SpotlightDialog initialFilter={Filter.People} initialText="Alpha" onFinished={() => null} />);
-        // search is debounced
-        jest.advanceTimersByTime(200);
-        await flushPromisesWithFakeTimers();
-
-        const content = document.querySelector("#mx_SpotlightDialog_content")!;
-        const options = content.querySelectorAll("li.mx_SpotlightDialog_option");
-        expect(options.length).toBeGreaterThanOrEqual(2);
-        expect(options[0]).toHaveTextContent("User Alpha");
-        expect(options[1]).toHaveTextContent("User Beta");
-    });
-
     it("should start a DM when clicking a person", async () => {
         render(
             <SpotlightDialog