mirror of
https://github.com/simstudioai/sim.git
synced 2026-09-24 15:45:35 +08:00
* fix(agent-tool): fix workflow tool in agent to respect user-provided params, added badge for deployment status * ack PR comment * updated gh stars
231 lines
6.2 KiB
TypeScript
231 lines
6.2 KiB
TypeScript
import { describe, expect, it } from 'vitest'
|
|
import { workflowExecutorTool } from '@/tools/workflow/executor'
|
|
|
|
describe('workflowExecutorTool', () => {
|
|
describe('request.body', () => {
|
|
const buildBody = workflowExecutorTool.request.body!
|
|
|
|
it.concurrent('should pass through object inputMapping unchanged (LLM-provided args)', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: { firstName: 'John', lastName: 'Doe', age: 30 },
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: { firstName: 'John', lastName: 'Doe', age: 30 },
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should parse JSON string inputMapping (UI-provided via tool-input)', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: '{"firstName": "John", "lastName": "Doe"}',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: { firstName: 'John', lastName: 'Doe' },
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should handle nested objects in JSON string inputMapping', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: '{"user": {"name": "John", "email": "john@example.com"}, "count": 5}',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: { user: { name: 'John', email: 'john@example.com' }, count: 5 },
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should handle arrays in JSON string inputMapping', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: '{"tags": ["a", "b", "c"], "ids": [1, 2, 3]}',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: { tags: ['a', 'b', 'c'], ids: [1, 2, 3] },
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should default to empty object when inputMapping is undefined', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: undefined,
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: {},
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should default to empty object when inputMapping is null', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: null as any,
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: {},
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should fallback to empty object for invalid JSON string', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: 'not valid json {',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: {},
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should fallback to empty object for empty string', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: '',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: {},
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should handle empty object inputMapping', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: {},
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: {},
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should handle empty JSON object string', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: '{}',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: {},
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should preserve special characters in string values', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: '{"message": "Hello\\nWorld", "path": "C:\\\\Users"}',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: { message: 'Hello\nWorld', path: 'C:\\Users' },
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should handle unicode characters in JSON string', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: '{"greeting": "こんにちは", "emoji": "👋"}',
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: { greeting: 'こんにちは', emoji: '👋' },
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
|
|
it.concurrent('should not modify object with string values that look like JSON', () => {
|
|
const params = {
|
|
workflowId: 'test-workflow-id',
|
|
inputMapping: { data: '{"nested": "json"}' },
|
|
}
|
|
|
|
const result = buildBody(params)
|
|
|
|
expect(result).toEqual({
|
|
input: { data: '{"nested": "json"}' },
|
|
triggerType: 'api',
|
|
useDraftState: false,
|
|
})
|
|
})
|
|
})
|
|
|
|
describe('request.url', () => {
|
|
it.concurrent('should build correct URL with workflowId', () => {
|
|
const url = workflowExecutorTool.request.url as (params: any) => string
|
|
|
|
expect(url({ workflowId: 'abc-123' })).toBe('/api/workflows/abc-123/execute')
|
|
expect(url({ workflowId: 'my-workflow' })).toBe('/api/workflows/my-workflow/execute')
|
|
})
|
|
})
|
|
|
|
describe('tool metadata', () => {
|
|
it.concurrent('should have correct id', () => {
|
|
expect(workflowExecutorTool.id).toBe('workflow_executor')
|
|
})
|
|
|
|
it.concurrent('should have required workflowId param', () => {
|
|
expect(workflowExecutorTool.params.workflowId.required).toBe(true)
|
|
})
|
|
|
|
it.concurrent('should have optional inputMapping param', () => {
|
|
expect(workflowExecutorTool.params.inputMapping.required).toBe(false)
|
|
})
|
|
|
|
it.concurrent('should use POST method', () => {
|
|
expect(workflowExecutorTool.request.method).toBe('POST')
|
|
})
|
|
})
|
|
})
|