test: config優先順位のE2E追加とE2Eドキュメント更新
This commit is contained in:
nrslib
parent
67ae3e8ae5
commit
2a6b9f4ad0
@ -168,3 +168,53 @@ E2Eテストを追加・変更した場合は、このドキュメントも更
|
||||
- 出力に `ファイルをデプロイしました` を含むことを確認する。
|
||||
- `$HOME/.claude/skills/takt/SKILL.md` が存在することを確認する。
|
||||
- `$HOME/.claude/skills/takt/pieces/` および `$HOME/.claude/skills/takt/personas/` ディレクトリが存在し、それぞれ少なくとも1ファイルを含むことを確認する。
|
||||
|
||||
## 追記シナリオ（2026-02-19）
|
||||
過去にドキュメント未反映だったシナリオを以下に追記する。
|
||||
|
||||
- Config priority（`e2e/specs/config-priority.e2e.ts`）
|
||||
- 目的: `piece` と `auto_pr` の優先順位（config/env/CLI）を検証。
|
||||
- 手順（要約）:
|
||||
- `--pipeline` で `--piece` 未指定時に設定値の `piece` が使われることを確認。
|
||||
- `auto_pr` 未設定時は確認デフォルト `true` が反映されることを確認。
|
||||
- `config` と `TAKT_AUTO_PR` の優先を確認。
|
||||
- Pipeline --skip-git on local/non-git directories（`e2e/specs/pipeline-local-repo.e2e.ts`）
|
||||
- 目的: ローカルGitリポジトリおよび非Gitディレクトリで `--pipeline --skip-git` が動作することを確認。
|
||||
- Task content_file reference（`e2e/specs/task-content-file.e2e.ts`）
|
||||
- 目的: `tasks.yaml` の `content_file` 参照が解決されること、および不正参照時エラーを確認。
|
||||
- Task status persistence（`e2e/specs/task-status-persistence.e2e.ts`）
|
||||
- 目的: 成功時/失敗時の `tasks.yaml` 状態遷移（完了消込・失敗記録）を確認。
|
||||
- Run multiple tasks（`e2e/specs/run-multiple-tasks.e2e.ts`）
|
||||
- 目的: 複数pendingタスクの連続実行、途中失敗時継続、タスク空時の終了挙動を確認。
|
||||
- Session NDJSON log output（`e2e/specs/session-log.e2e.ts`）
|
||||
- 目的: NDJSONログの主要イベント（`piece_complete` / `piece_abort` 等）出力を確認。
|
||||
- Structured output rule matching（`e2e/specs/structured-output.e2e.ts`）
|
||||
- 目的: structured output によるルール判定（Phase 3）を確認。
|
||||
- Piece error handling（`e2e/specs/piece-error-handling.e2e.ts`）
|
||||
- 目的: エージェントエラー、最大反復到達、前回応答受け渡しの挙動を確認。
|
||||
- Multi-step with parallel movements（`e2e/specs/multi-step-parallel.e2e.ts`）
|
||||
- 目的: 並列ムーブメントを含む複数ステップ遷移を確認。
|
||||
- Sequential multi-step session log transitions（`e2e/specs/multi-step-sequential.e2e.ts`）
|
||||
- 目的: 逐次ステップでのセッションログ遷移を確認。
|
||||
- Cycle detection via loop_monitors（`e2e/specs/cycle-detection.e2e.ts`）
|
||||
- 目的: ループ監視設定による abort/continue の境界を確認。
|
||||
- Provider error handling（`e2e/specs/provider-error.e2e.ts`）
|
||||
- 目的: provider上書き、mockシナリオ不足時の挙動、シナリオ不在時エラーを確認。
|
||||
- Model override（`e2e/specs/model-override.e2e.ts`）
|
||||
- 目的: `--model` オプションが通常実行/`--pipeline --skip-git` で反映されることを確認。
|
||||
- Error handling edge cases（`e2e/specs/error-handling.e2e.ts`）
|
||||
- 目的: 不正引数・存在しないpiece・不正YAMLなど代表エラーケースを確認。
|
||||
- Quiet mode（`e2e/specs/quiet-mode.e2e.ts`）
|
||||
- 目的: `--quiet` でAIストリーム出力が抑制されることを確認。
|
||||
- Catalog command（`e2e/specs/cli-catalog.e2e.ts`）
|
||||
- 目的: `takt catalog` の一覧表示・型指定・不正型エラーを確認。
|
||||
- Prompt preview command（`e2e/specs/cli-prompt.e2e.ts`）
|
||||
- 目的: `takt prompt` のプレビュー出力と不正piece時エラーを確認。
|
||||
- Switch piece command（`e2e/specs/cli-switch.e2e.ts`）
|
||||
- 目的: `takt switch` の切替成功・不正piece時エラーを確認。
|
||||
- Clear sessions command（`e2e/specs/cli-clear.e2e.ts`）
|
||||
- 目的: `takt clear` でセッション情報が削除されることを確認。
|
||||
- Help command（`e2e/specs/cli-help.e2e.ts`）
|
||||
- 目的: `takt --help` と `takt run --help` の表示内容を確認。
|
||||
- Eject builtin pieces（`e2e/specs/eject.e2e.ts`）
|
||||
- 目的: `takt eject` のproject/global出力、既存時スキップ、facet個別ejectを確認。
|
||||
|
||||
152
e2e/specs/config-priority.e2e.ts
Normal file
152
e2e/specs/config-priority.e2e.ts
Normal file
@ -0,0 +1,152 @@
|
||||
import { describe, it, expect, beforeEach, afterEach } from 'vitest';
|
||||
import { dirname, join, resolve } from 'node:path';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { mkdirSync, readFileSync, writeFileSync } from 'node:fs';
|
||||
import { parse as parseYaml } from 'yaml';
|
||||
import { createIsolatedEnv, updateIsolatedConfig, type IsolatedEnv } from '../helpers/isolated-env';
|
||||
import { createTestRepo, type TestRepo } from '../helpers/test-repo';
|
||||
import { runTakt } from '../helpers/takt-runner';
|
||||
|
||||
const __filename = fileURLToPath(import.meta.url);
|
||||
const __dirname = dirname(__filename);
|
||||
|
||||
function readFirstTask(repoPath: string): Record<string, unknown> {
|
||||
const tasksPath = join(repoPath, '.takt', 'tasks.yaml');
|
||||
const raw = readFileSync(tasksPath, 'utf-8');
|
||||
const parsed = parseYaml(raw) as { tasks?: Array<Record<string, unknown>> } | null;
|
||||
const first = parsed?.tasks?.[0];
|
||||
if (!first) {
|
||||
throw new Error(`No task record found in ${tasksPath}`);
|
||||
}
|
||||
return first;
|
||||
}
|
||||
|
||||
// E2E更新時は docs/testing/e2e.md も更新すること
|
||||
describe('E2E: Config priority (piece / autoPr)', () => {
|
||||
let isolatedEnv: IsolatedEnv;
|
||||
let testRepo: TestRepo;
|
||||
|
||||
beforeEach(() => {
|
||||
isolatedEnv = createIsolatedEnv();
|
||||
testRepo = createTestRepo();
|
||||
});
|
||||
|
||||
afterEach(() => {
|
||||
try {
|
||||
testRepo.cleanup();
|
||||
} catch {
|
||||
// best-effort
|
||||
}
|
||||
try {
|
||||
isolatedEnv.cleanup();
|
||||
} catch {
|
||||
// best-effort
|
||||
}
|
||||
});
|
||||
|
||||
it('should use configured piece in pipeline when --piece is omitted', () => {
|
||||
const configuredPiecePath = resolve(__dirname, '../fixtures/pieces/mock-single-step.yaml');
|
||||
const scenarioPath = resolve(__dirname, '../fixtures/scenarios/execute-done.json');
|
||||
const projectConfigDir = join(testRepo.path, '.takt');
|
||||
mkdirSync(projectConfigDir, { recursive: true });
|
||||
writeFileSync(
|
||||
join(projectConfigDir, 'config.yaml'),
|
||||
`piece: ${JSON.stringify(configuredPiecePath)}\n`,
|
||||
'utf-8',
|
||||
);
|
||||
|
||||
const result = runTakt({
|
||||
args: [
|
||||
'--pipeline',
|
||||
'--task', 'Pipeline run should resolve piece from config',
|
||||
'--skip-git',
|
||||
'--provider', 'mock',
|
||||
],
|
||||
cwd: testRepo.path,
|
||||
env: {
|
||||
...isolatedEnv.env,
|
||||
TAKT_MOCK_SCENARIO: scenarioPath,
|
||||
},
|
||||
timeout: 240_000,
|
||||
});
|
||||
|
||||
expect(result.exitCode).toBe(0);
|
||||
expect(result.stdout).toContain(`Running piece: ${configuredPiecePath}`);
|
||||
expect(result.stdout).toContain(`Piece '${configuredPiecePath}' completed`);
|
||||
}, 240_000);
|
||||
|
||||
it('should default auto_pr to true when unset in config/env', () => {
|
||||
const piecePath = resolve(__dirname, '../fixtures/pieces/mock-single-step.yaml');
|
||||
const scenarioPath = resolve(__dirname, '../fixtures/scenarios/execute-done.json');
|
||||
|
||||
const result = runTakt({
|
||||
args: [
|
||||
'--task', 'Auto PR default behavior',
|
||||
'--piece', piecePath,
|
||||
'--create-worktree', 'yes',
|
||||
'--provider', 'mock',
|
||||
],
|
||||
cwd: testRepo.path,
|
||||
env: {
|
||||
...isolatedEnv.env,
|
||||
TAKT_MOCK_SCENARIO: scenarioPath,
|
||||
},
|
||||
timeout: 240_000,
|
||||
});
|
||||
|
||||
expect(result.exitCode).toBe(0);
|
||||
const task = readFirstTask(testRepo.path);
|
||||
expect(task['auto_pr']).toBe(true);
|
||||
}, 240_000);
|
||||
|
||||
it('should use auto_pr from config when set', () => {
|
||||
const piecePath = resolve(__dirname, '../fixtures/pieces/mock-single-step.yaml');
|
||||
const scenarioPath = resolve(__dirname, '../fixtures/scenarios/execute-done.json');
|
||||
updateIsolatedConfig(isolatedEnv.taktDir, { auto_pr: false });
|
||||
|
||||
const result = runTakt({
|
||||
args: [
|
||||
'--task', 'Auto PR from config',
|
||||
'--piece', piecePath,
|
||||
'--create-worktree', 'yes',
|
||||
'--provider', 'mock',
|
||||
],
|
||||
cwd: testRepo.path,
|
||||
env: {
|
||||
...isolatedEnv.env,
|
||||
TAKT_MOCK_SCENARIO: scenarioPath,
|
||||
},
|
||||
timeout: 240_000,
|
||||
});
|
||||
|
||||
expect(result.exitCode).toBe(0);
|
||||
const task = readFirstTask(testRepo.path);
|
||||
expect(task['auto_pr']).toBe(false);
|
||||
}, 240_000);
|
||||
|
||||
it('should prioritize env auto_pr over config', () => {
|
||||
const piecePath = resolve(__dirname, '../fixtures/pieces/mock-single-step.yaml');
|
||||
const scenarioPath = resolve(__dirname, '../fixtures/scenarios/execute-done.json');
|
||||
updateIsolatedConfig(isolatedEnv.taktDir, { auto_pr: false });
|
||||
|
||||
const result = runTakt({
|
||||
args: [
|
||||
'--task', 'Auto PR from env override',
|
||||
'--piece', piecePath,
|
||||
'--create-worktree', 'yes',
|
||||
'--provider', 'mock',
|
||||
],
|
||||
cwd: testRepo.path,
|
||||
env: {
|
||||
...isolatedEnv.env,
|
||||
TAKT_AUTO_PR: 'true',
|
||||
TAKT_MOCK_SCENARIO: scenarioPath,
|
||||
},
|
||||
timeout: 240_000,
|
||||
});
|
||||
|
||||
expect(result.exitCode).toBe(0);
|
||||
const task = readFirstTask(testRepo.path);
|
||||
expect(task['auto_pr']).toBe(true);
|
||||
}, 240_000);
|
||||
});
|
||||
@ -35,6 +35,7 @@ export default defineConfig({
|
||||
'e2e/specs/eject.e2e.ts',
|
||||
'e2e/specs/quiet-mode.e2e.ts',
|
||||
'e2e/specs/task-content-file.e2e.ts',
|
||||
'e2e/specs/config-priority.e2e.ts',
|
||||
],
|
||||
},
|
||||
});
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user