test(filesystem): assert admission cancellation provenance
Keep the explicit checkpoint wrapper and verify its exact caller reason alongside the retained cleanup owner. Refs #4610
This commit is contained in:
1 parent
0c7b6d834a
commit
3bd9aa3484
1 file changed
+8
-2
@@ -51,7 +51,10 @@ vi.mock("node:fs/promises", async (importOriginal) => {
|
||||
|
||||
import { parseBinaryTarget } from "../../src/application/BinaryTargetResolver.js";
|
||||
import { projectAnalysisError } from "../../src/domain/analysisErrorProjection.js";
|
||||
import { RegularFileAdmissionFailure } from "../../src/filesystem/RegularFile.js";
|
||||
import {
|
||||
RegularFileAdmissionFailure,
|
||||
RegularFileOperationCancelled,
|
||||
} from "../../src/filesystem/RegularFile.js";
|
||||
import { createApplicationMcpHarness } from "../fixtures/applicationMcpHarness.js";
|
||||
import { createTestBinarySession } from "../fixtures/binarySession.js";
|
||||
import { advertisedMcpToolSchemas } from "../fixtures/advertisedMcpTool.js";
|
||||
@@ -155,7 +158,10 @@ it("retains the cleanup failure and its file owner when cancelled admission cann
|
||||
}).catch((cause: unknown) => cause);
|
||||
expect(failure).toBeInstanceOf(RegularFileAdmissionFailure);
|
||||
if (!(failure instanceof RegularFileAdmissionFailure)) throw failure;
|
||||
expect(failure.cause).toBe(reason);
|
||||
expect(failure.cause).toBeInstanceOf(RegularFileOperationCancelled);
|
||||
if (!(failure.cause instanceof RegularFileOperationCancelled))
|
||||
throw failure.cause;
|
||||
expect(failure.cause.reason).toBe(reason);
|
||||
expect(failure.cleanupCause).toBe(cleanup);
|
||||
expect(failure.owner.handle).toBe(admission.handles[0]);
|
||||
expect(failure.owner.handle.fd).toBeGreaterThanOrEqual(0);
|
||||
|
||||
Reference in new issue
Block a user