mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-02-05 09:33:07 +00:00
test: fix sandbox mode test assertions
Add comprehensive test coverage for sandbox mode configuration: - Added tests for enableSandboxMode=false for both createChatOptions and createAutoModeOptions - Added tests for enableSandboxMode not provided for both functions - Updated existing tests to pass enableSandboxMode=true where sandbox assertions exist This addresses the broken test assertions identified by coderabbit-ai review. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -179,7 +179,7 @@ describe('sdk-options.ts', () => {
|
|||||||
it('should create options with chat settings', async () => {
|
it('should create options with chat settings', async () => {
|
||||||
const { createChatOptions, TOOL_PRESETS, MAX_TURNS } = await import('@/lib/sdk-options.js');
|
const { createChatOptions, TOOL_PRESETS, MAX_TURNS } = await import('@/lib/sdk-options.js');
|
||||||
|
|
||||||
const options = createChatOptions({ cwd: '/test/path' });
|
const options = createChatOptions({ cwd: '/test/path', enableSandboxMode: true });
|
||||||
|
|
||||||
expect(options.cwd).toBe('/test/path');
|
expect(options.cwd).toBe('/test/path');
|
||||||
expect(options.maxTurns).toBe(MAX_TURNS.standard);
|
expect(options.maxTurns).toBe(MAX_TURNS.standard);
|
||||||
@@ -212,6 +212,27 @@ describe('sdk-options.ts', () => {
|
|||||||
|
|
||||||
expect(options.model).toBe('claude-sonnet-4-20250514');
|
expect(options.model).toBe('claude-sonnet-4-20250514');
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('should not set sandbox when enableSandboxMode is false', async () => {
|
||||||
|
const { createChatOptions } = await import('@/lib/sdk-options.js');
|
||||||
|
|
||||||
|
const options = createChatOptions({
|
||||||
|
cwd: '/test/path',
|
||||||
|
enableSandboxMode: false,
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(options.sandbox).toBeUndefined();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should not set sandbox when enableSandboxMode is not provided', async () => {
|
||||||
|
const { createChatOptions } = await import('@/lib/sdk-options.js');
|
||||||
|
|
||||||
|
const options = createChatOptions({
|
||||||
|
cwd: '/test/path',
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(options.sandbox).toBeUndefined();
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('createAutoModeOptions', () => {
|
describe('createAutoModeOptions', () => {
|
||||||
@@ -219,7 +240,7 @@ describe('sdk-options.ts', () => {
|
|||||||
const { createAutoModeOptions, TOOL_PRESETS, MAX_TURNS } =
|
const { createAutoModeOptions, TOOL_PRESETS, MAX_TURNS } =
|
||||||
await import('@/lib/sdk-options.js');
|
await import('@/lib/sdk-options.js');
|
||||||
|
|
||||||
const options = createAutoModeOptions({ cwd: '/test/path' });
|
const options = createAutoModeOptions({ cwd: '/test/path', enableSandboxMode: true });
|
||||||
|
|
||||||
expect(options.cwd).toBe('/test/path');
|
expect(options.cwd).toBe('/test/path');
|
||||||
expect(options.maxTurns).toBe(MAX_TURNS.maximum);
|
expect(options.maxTurns).toBe(MAX_TURNS.maximum);
|
||||||
@@ -252,6 +273,27 @@ describe('sdk-options.ts', () => {
|
|||||||
|
|
||||||
expect(options.abortController).toBe(abortController);
|
expect(options.abortController).toBe(abortController);
|
||||||
});
|
});
|
||||||
|
|
||||||
|
it('should not set sandbox when enableSandboxMode is false', async () => {
|
||||||
|
const { createAutoModeOptions } = await import('@/lib/sdk-options.js');
|
||||||
|
|
||||||
|
const options = createAutoModeOptions({
|
||||||
|
cwd: '/test/path',
|
||||||
|
enableSandboxMode: false,
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(options.sandbox).toBeUndefined();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should not set sandbox when enableSandboxMode is not provided', async () => {
|
||||||
|
const { createAutoModeOptions } = await import('@/lib/sdk-options.js');
|
||||||
|
|
||||||
|
const options = createAutoModeOptions({
|
||||||
|
cwd: '/test/path',
|
||||||
|
});
|
||||||
|
|
||||||
|
expect(options.sandbox).toBeUndefined();
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
describe('createCustomOptions', () => {
|
describe('createCustomOptions', () => {
|
||||||
|
|||||||
Reference in New Issue
Block a user