test(mcp): align mocks with scoped execution

Author: TheodoreSpeaks

apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/prompt-editor/prompt-editor.test.tsx

@@ -7,7 +7,7 @@ import { createRoot, type Root } from 'react-dom/client'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 vi.mock('@/hooks/queries/skills', () => ({ useSkills: () => ({ data: [] }) }))
-vi.mock('@/hooks/queries/mcp', () => ({ useMcpServers: () => ({ data: [] }) }))
+vi.mock('@/hooks/queries/mcp', () => ({ useMcpToolServers: () => ({ data: [] }) }))
 vi.mock('@/blocks/integration-matcher', () => ({
   getIntegrationMatcher: () => ({ regex: null, byName: new Map() }),
 }))

apps/sim/app/workspace/[workspaceId]/home/components/user-input/components/prompt-editor/use-prompt-editor.test.tsx

@@ -6,7 +6,7 @@ import { createRoot, type Root } from 'react-dom/client'
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 
 vi.mock('@/hooks/queries/skills', () => ({ useSkills: () => ({ data: [] }) }))
-vi.mock('@/hooks/queries/mcp', () => ({ useMcpServers: () => ({ data: [] }) }))
+vi.mock('@/hooks/queries/mcp', () => ({ useMcpToolServers: () => ({ data: [] }) }))
 vi.mock('@/blocks/integration-matcher', () => ({
   getIntegrationMatcher: () => ({ regex: null, byName: new Map() }),
 }))

apps/sim/lib/credential-groups/mcp-oauth-state.test.ts

@@ -18,6 +18,9 @@ const { mockRedis, values } = vi.hoisted(() => {
         values.delete(key)
         return value
       }),
+      sadd: vi.fn(async () => 1),
+      srem: vi.fn(async () => 1),
+      pexpire: vi.fn(async () => 1),
     },
   }
 })

apps/sim/lib/credentials/application/credential-crud.test.ts

@@ -37,6 +37,7 @@ vi.mock('@/lib/credentials/queries', () => ({
 vi.mock('@/lib/credentials/access', () => ({
   getCredentialActorContext: mocks.getActor,
   canUseCredential: () => true,
+  requireOrdinaryCredentialType: (type: string) => type,
 }))
 vi.mock('@/lib/credentials/orchestration', () => ({
   updateCredentialRecord: mocks.updateRecord,

apps/sim/lib/internal/mcp/execute-tool.test.ts

@@ -93,6 +93,7 @@ describe('executeMcpTool', () => {
     expect(mocks.createPrincipal).toHaveBeenCalledWith({
       context: CONTEXT,
       audience: 'sim:mcp-servers',
+      resourceScope: { mcpServerId: 'mcp-server' },
     })
     expect(mocks.executeUseCase).toHaveBeenCalledWith({
       principal: PRINCIPAL,