mirror/rea
1
0
Fork 0
mirror of https://github.com/morluto/rea synced 2026-10-25 22:08:11 -04:00
Code Issues Packages Projects Releases Wiki Activity

ci(windows): run remaining Windows-only path admission cases in the lane (#5807)

Browse Source 
Three test files have cases that run only on Windows
(skipIf(process.platform !== "win32") / describe.runIf(win32)), but the
windows-curated job runs an explicit file list that did not name them,
so CI never executed those cases:

- src/contracts/sessionPathInputs.test.ts (Windows absolute path forms)
- tests/boundary/filesystem/directAnalysisSnapshotReplay.test.ts
  (snapshot replay through a junction and before native admission)
- tests/boundary/mcp/rawBinaryMcp.test.ts (saved reads across a
  junction and a noncanonical Windows path)

Add them to the lane and the package change scope, and guard the
entries in curatedWindowsLane.test.ts.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Oke Iyanuoluwa Enoch
2026-10-21 19:52:11 +01:00
committed by GitHub
parent 07fd426e9a
commit 3dde9a5e1c
3 changed files with 18 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/ci.yml
+3
View File
@@ -856,6 +856,9 @@ jobs:
tests/process-global/filesystemSnapshotFallbackCancellation.test.ts
tests/boundary/filesystem/typescriptCompiler.test.ts
tests/boundary/filesystem/binaryTarget.test.ts
src/contracts/sessionPathInputs.test.ts
tests/boundary/filesystem/directAnalysisSnapshotReplay.test.ts
tests/boundary/mcp/rawBinaryMcp.test.ts
tests/boundary/filesystem/safeOutputTree.test.ts
tests/boundary/providers/native/artifactProvider.archiveSafety.test.ts
tests/boundary/providers/native/artifactProvider.extractionCompleteness.test.ts
scripts/ci/scopes.mjs
+3
View File
@@ -148,6 +148,9 @@ export const SCOPES = {
"src/cli/setupCommands.ts",
"tests/boundary/filesystem/typescriptCompiler.test.ts",
"tests/boundary/filesystem/binaryTarget.test.ts",
"src/contracts/sessionPathInputs.test.ts",
"tests/boundary/filesystem/directAnalysisSnapshotReplay.test.ts",
"tests/boundary/mcp/rawBinaryMcp.test.ts",
"tests/boundary/filesystem/safeOutputTree.test.ts",
"tests/boundary/providers/native/artifactProvider.archiveSafety.test.ts",
"tests/boundary/providers/native/artifactProvider.extractionCompleteness.test.ts",
tests/process-global/curatedWindowsLane.test.ts
+12
View File
@@ -99,6 +99,18 @@ describe("curated Windows test lane", () => {
);
});
it("keeps Windows-only path and snapshot admission cases in the lane", async () => {
// These files have cases that run only on Windows, so outside this lane
// CI never executes them.
expect(await explicitPaths("windows")).toEqual(
expect.arrayContaining([
"src/contracts/sessionPathInputs.test.ts",
"tests/boundary/filesystem/directAnalysisSnapshotReplay.test.ts",
"tests/boundary/mcp/rawBinaryMcp.test.ts",
]),
);
});
it("keeps Windows runner cancellation coverage in the lane", async () => {
expect(await explicitPaths("windows")).toEqual(
expect.arrayContaining([