mirror of
https://github.com/simstudioai/sim.git
synced 2026-01-22 05:18:08 -05:00
* progress * improvement(execution): update execution for passing base64 strings * fix types * cleanup comments * path security vuln * reject promise correctly * fix redirect case * remove proxy routes * fix tests * use ipaddr
169 lines
5.3 KiB
TypeScript
169 lines
5.3 KiB
TypeScript
import { loggerMock } from '@sim/testing'
|
|
import { beforeEach, describe, expect, it, type Mock, vi } from 'vitest'
|
|
import { DEFAULT_EXECUTION_TIMEOUT_MS } from '@/lib/execution/constants'
|
|
import { BlockType } from '@/executor/constants'
|
|
import { FunctionBlockHandler } from '@/executor/handlers/function/function-handler'
|
|
import type { ExecutionContext } from '@/executor/types'
|
|
import type { SerializedBlock } from '@/serializer/types'
|
|
import { executeTool } from '@/tools'
|
|
|
|
vi.mock('@sim/logger', () => loggerMock)
|
|
|
|
vi.mock('@/tools', () => ({
|
|
executeTool: vi.fn(),
|
|
}))
|
|
|
|
const mockExecuteTool = executeTool as Mock
|
|
|
|
describe('FunctionBlockHandler', () => {
|
|
let handler: FunctionBlockHandler
|
|
let mockBlock: SerializedBlock
|
|
let mockContext: ExecutionContext
|
|
|
|
beforeEach(() => {
|
|
handler = new FunctionBlockHandler()
|
|
|
|
mockBlock = {
|
|
id: 'func-block-1',
|
|
metadata: { id: BlockType.FUNCTION, name: 'Test Function' },
|
|
position: { x: 30, y: 30 },
|
|
config: { tool: BlockType.FUNCTION, params: {} },
|
|
inputs: { code: 'string', timeout: 'number' }, // Using ParamType strings
|
|
outputs: {},
|
|
enabled: true,
|
|
}
|
|
|
|
mockContext = {
|
|
workflowId: 'test-workflow-id',
|
|
blockStates: new Map(),
|
|
blockLogs: [],
|
|
metadata: { duration: 0 },
|
|
environmentVariables: {},
|
|
decisions: { router: new Map(), condition: new Map() },
|
|
loopExecutions: new Map(),
|
|
executedBlocks: new Set(),
|
|
activeExecutionPath: new Set(),
|
|
completedLoops: new Set(),
|
|
}
|
|
|
|
// Reset mocks using vi
|
|
vi.clearAllMocks()
|
|
|
|
// Default mock implementation for executeTool
|
|
mockExecuteTool.mockResolvedValue({ success: true, output: { result: 'Success' } })
|
|
})
|
|
|
|
it('should handle function blocks', () => {
|
|
expect(handler.canHandle(mockBlock)).toBe(true)
|
|
const nonFuncBlock: SerializedBlock = { ...mockBlock, metadata: { id: 'other' } }
|
|
expect(handler.canHandle(nonFuncBlock)).toBe(false)
|
|
})
|
|
|
|
it('should execute function block with string code', async () => {
|
|
const inputs = {
|
|
code: 'console.log("Hello"); return 1 + 1;',
|
|
timeout: 10000,
|
|
envVars: {},
|
|
isCustomTool: false,
|
|
workflowId: undefined,
|
|
}
|
|
const expectedToolParams = {
|
|
code: inputs.code,
|
|
language: 'javascript',
|
|
timeout: inputs.timeout,
|
|
envVars: {},
|
|
workflowVariables: {},
|
|
blockData: {},
|
|
blockNameMapping: {},
|
|
_context: { workflowId: mockContext.workflowId, workspaceId: mockContext.workspaceId },
|
|
}
|
|
const expectedOutput: any = { result: 'Success' }
|
|
|
|
const result = await handler.execute(mockContext, mockBlock, inputs)
|
|
|
|
expect(mockExecuteTool).toHaveBeenCalledWith(
|
|
'function_execute',
|
|
expectedToolParams,
|
|
false, // skipPostProcess
|
|
mockContext // execution context
|
|
)
|
|
expect(result).toEqual(expectedOutput)
|
|
})
|
|
|
|
it('should execute function block with array code', async () => {
|
|
const inputs = {
|
|
code: [{ content: 'const x = 5;' }, { content: 'return x * 2;' }],
|
|
timeout: 5000,
|
|
envVars: {},
|
|
isCustomTool: false,
|
|
workflowId: undefined,
|
|
}
|
|
const expectedCode = 'const x = 5;\nreturn x * 2;'
|
|
const expectedToolParams = {
|
|
code: expectedCode,
|
|
language: 'javascript',
|
|
timeout: inputs.timeout,
|
|
envVars: {},
|
|
workflowVariables: {},
|
|
blockData: {},
|
|
blockNameMapping: {},
|
|
_context: { workflowId: mockContext.workflowId, workspaceId: mockContext.workspaceId },
|
|
}
|
|
const expectedOutput: any = { result: 'Success' }
|
|
|
|
const result = await handler.execute(mockContext, mockBlock, inputs)
|
|
|
|
expect(mockExecuteTool).toHaveBeenCalledWith(
|
|
'function_execute',
|
|
expectedToolParams,
|
|
false, // skipPostProcess
|
|
mockContext // execution context
|
|
)
|
|
expect(result).toEqual(expectedOutput)
|
|
})
|
|
|
|
it('should use default timeout if not provided', async () => {
|
|
const inputs = { code: 'return true;' }
|
|
const expectedToolParams = {
|
|
code: inputs.code,
|
|
language: 'javascript',
|
|
timeout: DEFAULT_EXECUTION_TIMEOUT_MS,
|
|
envVars: {},
|
|
workflowVariables: {},
|
|
blockData: {},
|
|
blockNameMapping: {},
|
|
_context: { workflowId: mockContext.workflowId, workspaceId: mockContext.workspaceId },
|
|
}
|
|
|
|
await handler.execute(mockContext, mockBlock, inputs)
|
|
|
|
expect(mockExecuteTool).toHaveBeenCalledWith(
|
|
'function_execute',
|
|
expectedToolParams,
|
|
false, // skipPostProcess
|
|
mockContext // execution context
|
|
)
|
|
})
|
|
|
|
it('should handle execution errors from the tool', async () => {
|
|
const inputs = { code: 'throw new Error("Code failed");' }
|
|
const errorResult = { success: false, error: 'Function execution failed: Code failed' }
|
|
mockExecuteTool.mockResolvedValue(errorResult)
|
|
|
|
await expect(handler.execute(mockContext, mockBlock, inputs)).rejects.toThrow(
|
|
'Function execution failed: Code failed'
|
|
)
|
|
expect(mockExecuteTool).toHaveBeenCalled()
|
|
})
|
|
|
|
it('should handle tool error with no specific message', async () => {
|
|
const inputs = { code: 'some code' }
|
|
const errorResult = { success: false }
|
|
mockExecuteTool.mockResolvedValue(errorResult)
|
|
|
|
await expect(handler.execute(mockContext, mockBlock, inputs)).rejects.toThrow(
|
|
'Function execution failed'
|
|
)
|
|
})
|
|
})
|