test(execution,providers): fix preflight mock reset + isolate provider client cache in tests

- preprocessing.test: re-establish the checkOrgMemberUsageLimit mock in beforeEach
  (the only gate mock not re-set). In the full suite its implementation was reset
  so the success-path test got undefined -> threw -> 500 -> success:false. Mirrors
  how checkServerSideUsageLimits is handled.
- client-cache: add clearProviderClientCacheForTests; call it in the bedrock and
  vllm test beforeEach so construction assertions always start from a cache miss
  now that those providers memoize their client.

Author: waleedlatif1

apps/sim/lib/execution/preprocessing.test.ts

@@ -44,7 +44,10 @@ vi.mock('@sim/workflow-authz', () => ({
   }),
 }))
 
-import { checkServerSideUsageLimits } from '@/lib/billing/calculations/usage-monitor'
+import {
+  checkOrgMemberUsageLimit,
+  checkServerSideUsageLimits,
+} from '@/lib/billing/calculations/usage-monitor'
 import { getHighestPrioritySubscription } from '@/lib/billing/core/subscription'
 import { preprocessExecution } from './preprocessing'
 
@@ -125,6 +128,7 @@ describe('preprocessExecution logPreprocessingErrors option', () => {
       remaining: 100,
       resetAt: new Date(),
     })
+    vi.mocked(checkOrgMemberUsageLimit).mockResolvedValue({ isExceeded: false } as any)
   })
 
   it('suppresses preprocessing-error logging when logPreprocessingErrors is false', async () => {
@@ -174,6 +178,7 @@ describe('preprocessExecution ban gate', () => {
       currentUsage: 1,
       limit: 10,
     } as any)
+    vi.mocked(checkOrgMemberUsageLimit).mockResolvedValue({ isExceeded: false } as any)
   })
 
   it('blocks execution with 403 when the actor is banned (ban wins over the parallel gates)', async () => {

apps/sim/providers/bedrock/index.test.ts

@@ -50,10 +50,12 @@ vi.mock('@/tools', () => ({
 
 import { BedrockRuntimeClient } from '@aws-sdk/client-bedrock-runtime'
 import { bedrockProvider } from '@/providers/bedrock/index'
+import { clearProviderClientCacheForTests } from '@/providers/client-cache'
 
 describe('bedrockProvider credential handling', () => {
   beforeEach(() => {
     vi.clearAllMocks()
+    clearProviderClientCacheForTests()
     mockSend.mockResolvedValue({
       output: { message: { content: [{ text: 'response' }] } },
       usage: { inputTokens: 10, outputTokens: 5 },

apps/sim/providers/client-cache.ts

@@ -29,3 +29,8 @@ export function getCachedProviderClient<T extends object>(key: string, factory:
   clientCache.set(key, client)
   return client
 }
+
+/** Clears the cache so tests asserting client construction start from a miss. */
+export function clearProviderClientCacheForTests(): void {
+  clientCache.clear()
+}

apps/sim/providers/vllm/index.test.ts

@@ -79,6 +79,7 @@ vi.mock('@/stores/providers', () => ({
   useProvidersStore: { getState: () => ({ setProviderModels: vi.fn() }) },
 }))
 
+import { clearProviderClientCacheForTests } from '@/providers/client-cache'
 import type { ProviderToolConfig } from '@/providers/types'
 import { vllmProvider } from '@/providers/vllm/index'
 
@@ -117,6 +118,7 @@ const createPayload = (callIndex: number) => mockCreate.mock.calls[callIndex][0]
 describe('vllmProvider', () => {
   beforeEach(() => {
     vi.clearAllMocks()
+    clearProviderClientCacheForTests()
     openAIArgs.length = 0
     envState.VLLM_BASE_URL = 'http://localhost:8000'
     envState.VLLM_API_KEY = undefined