Skip to content

test(repro): skip repro-flow-execute when local session file is absent#37

Merged
PhucHai123 merged 1 commit into
mainfrom
fix/skip-repro-flow-execute-without-fixture
Jun 30, 2026
Merged

test(repro): skip repro-flow-execute when local session file is absent#37
PhucHai123 merged 1 commit into
mainfrom
fix/skip-repro-flow-execute-without-fixture

Conversation

@PhucHai123

Copy link
Copy Markdown
Collaborator

Problem

tests/repro-flow-execute.test.ts relies on a hardcoded session fixture path that only exists on one developer's machine:

/Users/__blitzzz/.pi/agent/sessions/--Users-__blitzzz-Documents-GitHub-pi-agent-flow--/2026-06-19T03-30-34-480Z_019eddee-2d70-71a0-9a21-cdc21b692771.jsonl

When CI (or anyone else) runs npm test, the file is missing and the suite fails with:

Error: ENOENT: no such file or directory, open '/Users/__blitzzz/.pi/agent/sessions/...'

Fix

Make the repro test conditional on the fixture being present:

it.skipIf(!fs.existsSync(SESSION), "bare scout (no dispatch) does not throw 'length'")(async () => {

The test still runs locally when PI_REPRO_SESSION points at an existing file, but it no longer breaks CI or other contributors' machines.

Scope

This is a CI hygiene fix. It is split out from #36 so that PR can remain focused on the withFileMutationQueue install fix.

Suggested merge order: merge this PR first, then #36 will pass CI cleanly.

@PhucHai123 PhucHai123 merged commit 1944630 into main Jun 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant