mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-02-03 21:03:08 +00:00
test: update claude-provider tests for sandbox changes
Updated tests to reflect changes made to sandbox mode implementation: 1. Changed permissionMode expectation from 'acceptEdits' to 'default' - ClaudeProvider now uses 'default' permission mode 2. Renamed test "should enable sandbox by default" to "should pass sandbox configuration when provided" - Sandbox is no longer enabled by default in the provider - Provider now forwards sandbox config only when explicitly provided via ExecuteOptions 3. Updated error handling test expectations - Now expects two console.error calls with new format - First call: '[ClaudeProvider] ERROR: executeQuery() error during execution:' - Second call: '[ClaudeProvider] ERROR stack:' with stack trace All 32 tests in claude-provider.test.ts now pass. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -73,7 +73,7 @@ describe('claude-provider.ts', () => {
|
|||||||
maxTurns: 10,
|
maxTurns: 10,
|
||||||
cwd: '/test/dir',
|
cwd: '/test/dir',
|
||||||
allowedTools: ['Read', 'Write'],
|
allowedTools: ['Read', 'Write'],
|
||||||
permissionMode: 'acceptEdits',
|
permissionMode: 'default',
|
||||||
}),
|
}),
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
@@ -100,7 +100,7 @@ describe('claude-provider.ts', () => {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
it('should enable sandbox by default', async () => {
|
it('should pass sandbox configuration when provided', async () => {
|
||||||
vi.mocked(sdk.query).mockReturnValue(
|
vi.mocked(sdk.query).mockReturnValue(
|
||||||
(async function* () {
|
(async function* () {
|
||||||
yield { type: 'text', text: 'test' };
|
yield { type: 'text', text: 'test' };
|
||||||
@@ -110,6 +110,10 @@ describe('claude-provider.ts', () => {
|
|||||||
const generator = provider.executeQuery({
|
const generator = provider.executeQuery({
|
||||||
prompt: 'Test',
|
prompt: 'Test',
|
||||||
cwd: '/test',
|
cwd: '/test',
|
||||||
|
sandbox: {
|
||||||
|
enabled: true,
|
||||||
|
autoAllowBashIfSandboxed: true,
|
||||||
|
},
|
||||||
});
|
});
|
||||||
|
|
||||||
await collectAsyncGenerator(generator);
|
await collectAsyncGenerator(generator);
|
||||||
@@ -242,11 +246,21 @@ describe('claude-provider.ts', () => {
|
|||||||
});
|
});
|
||||||
|
|
||||||
await expect(collectAsyncGenerator(generator)).rejects.toThrow('SDK execution failed');
|
await expect(collectAsyncGenerator(generator)).rejects.toThrow('SDK execution failed');
|
||||||
expect(consoleErrorSpy).toHaveBeenCalledWith(
|
|
||||||
'[ClaudeProvider] executeQuery() error during execution:',
|
// Should log error message
|
||||||
|
expect(consoleErrorSpy).toHaveBeenNthCalledWith(
|
||||||
|
1,
|
||||||
|
'[ClaudeProvider] ERROR: executeQuery() error during execution:',
|
||||||
testError
|
testError
|
||||||
);
|
);
|
||||||
|
|
||||||
|
// Should log stack trace
|
||||||
|
expect(consoleErrorSpy).toHaveBeenNthCalledWith(
|
||||||
|
2,
|
||||||
|
'[ClaudeProvider] ERROR stack:',
|
||||||
|
testError.stack
|
||||||
|
);
|
||||||
|
|
||||||
consoleErrorSpy.mockRestore();
|
consoleErrorSpy.mockRestore();
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user