test(memory): use the shared env and outbox mocks for cleanup enqueue

Author: mzxchandra

apps/sim/lib/mothership/memories/cleanup-outbox.test.ts

@@ -1,25 +1,21 @@
 /**
  * @vitest-environment node
  */
+import { resetEnvMock, setEnv } from '@sim/testing/mocks/env.mock'
+import { mothershipAgentUrlMock } from '@sim/testing/mocks/mothership-agent-url.mock'
+import { mothershipGoFetchMock } from '@sim/testing/mocks/mothership-go-fetch.mock'
+import { outboxServiceMock, outboxServiceMockFns } from '@sim/testing/mocks/outbox-service.mock'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 
-const { mockEnv, mockEnqueue } = vi.hoisted(() => ({
-  mockEnv: { COPILOT_API_KEY: undefined as string | undefined } as Record<
-    string,
-    string | undefined
-  >,
-  mockEnqueue: vi.fn(),
-}))
-
-vi.mock('@/lib/core/config/env', () => ({ env: mockEnv }))
-vi.mock('@/lib/core/outbox/service', () => ({ enqueueOutboxEvent: mockEnqueue }))
+vi.mock('@/lib/core/outbox/service', () => outboxServiceMock)
 vi.mock('@/lib/mothership/constants', () => ({ SIM_AGENT_API_URL: 'https://agent.example.test' }))
-vi.mock('@/lib/mothership/request/go/fetch', () => ({ fetchGo: vi.fn() }))
+vi.mock('@/lib/mothership/request/go/fetch', () => mothershipGoFetchMock)
 vi.mock('@/lib/mothership/request/headers', () => ({ mothershipRequestHeaders: () => ({}) }))
-vi.mock('@/lib/mothership/server/agent-url', () => ({ getMothershipSourceEnvHeaders: () => ({}) }))
+vi.mock('@/lib/mothership/server/agent-url', () => mothershipAgentUrlMock)
 
 import { enqueueMemoryCleanup } from '@/lib/mothership/memories/cleanup-outbox'
 
+const { mockEnqueueOutboxEvent } = outboxServiceMockFns
 const executor = { insert: vi.fn() }
 const targets = [
   { kind: 'private_owner' as const, userId: 'user-1' },
@@ -28,20 +24,25 @@ const targets = [
 
 describe('enqueueMemoryCleanup', () => {
   beforeEach(() => {
-    vi.clearAllMocks()
-    mockEnv.COPILOT_API_KEY = undefined
+    resetEnvMock()
+    setEnv({
+      COPILOT_API_KEY: undefined,
+      COPILOT_DEV_URL: undefined,
+      COPILOT_STAGING_URL: undefined,
+      COPILOT_PROD_URL: undefined,
+    })
   })
 
   it('queues nothing when this Sim has no worker credential', async () => {
     await enqueueMemoryCleanup(executor, targets)
-    expect(mockEnqueue).not.toHaveBeenCalled()
+    expect(mockEnqueueOutboxEvent).not.toHaveBeenCalled()
   })
 
   it('queues one credential-bound event per destination and target', async () => {
-    mockEnv.COPILOT_API_KEY = 'configured-key'
+    setEnv({ COPILOT_API_KEY: 'configured-key' })
     await enqueueMemoryCleanup(executor, targets)
-    expect(mockEnqueue).toHaveBeenCalledTimes(targets.length)
-    for (const [, , payload] of mockEnqueue.mock.calls)
+    expect(mockEnqueueOutboxEvent).toHaveBeenCalledTimes(targets.length)
+    for (const [, , payload] of mockEnqueueOutboxEvent.mock.calls)
       expect(payload).toMatchObject({
         baseUrl: 'https://agent.example.test',
         credentialFingerprint: expect.any(String),