From cc89c28f03d1bacea5fa843cffbbfa7e98146c33 Mon Sep 17 00:00:00 2001
From: Jennie Robinson Faber <jennie@jenniefaber.com>
Date: Thu, 21 May 2026 17:52:06 +0100
Subject: [PATCH] test(events): update list mock chain after removing .select()
 projection

GET /api/events now does find().sort().lean() (no .select()) since the
handler strips registrations in the map step. The mock chain in
members-only-visibility.test.js still expected .select(), causing all
list tests to throw "lean is not a function".
---
 tests/server/api/events/members-only-visibility.test.js | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/tests/server/api/events/members-only-visibility.test.js b/tests/server/api/events/members-only-visibility.test.js
index f357c31..b5a214e 100644
--- a/tests/server/api/events/members-only-visibility.test.js
+++ b/tests/server/api/events/members-only-visibility.test.js
@@ -5,14 +5,12 @@ import { createMockEvent } from '../../helpers/createMockEvent.js'
 const {
   mockFind,
   mockSort,
-  mockSelect,
   mockLean,
   mockGetOptionalMember,
   mockLoadPublicEvent
 } = vi.hoisted(() => ({
   mockFind: vi.fn(),
   mockSort: vi.fn(),
-  mockSelect: vi.fn(),
   mockLean: vi.fn(),
   mockGetOptionalMember: vi.fn(),
   mockLoadPublicEvent: vi.fn()
@@ -36,8 +34,7 @@ vi.mock('../../../../server/utils/loadEvent.js', () => ({
 
 function setupFindChain(result = []) {
   mockLean.mockResolvedValue(result)
-  mockSelect.mockReturnValue({ lean: mockLean })
-  mockSort.mockReturnValue({ select: mockSelect })
+  mockSort.mockReturnValue({ lean: mockLean })
   mockFind.mockReturnValue({ sort: mockSort })
 }