From 97241ea19c08d4602d37c079d6a75c4d2f9ad3ce Mon Sep 17 00:00:00 2001 From: Lex Christopherson Date: Sun, 22 Mar 2026 09:13:31 -0600 Subject: [PATCH] fix(tests): use cross-platform path split in run-manager timestamp test The test split on "/" which fails on Windows where paths use "\". Co-Authored-By: Claude Opus 4.6 (1M context) --- src/resources/extensions/gsd/tests/run-manager.test.ts | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/resources/extensions/gsd/tests/run-manager.test.ts b/src/resources/extensions/gsd/tests/run-manager.test.ts index 0bd67f4c8..d4eb02ddd 100644 --- a/src/resources/extensions/gsd/tests/run-manager.test.ts +++ b/src/resources/extensions/gsd/tests/run-manager.test.ts @@ -156,9 +156,8 @@ describe("createRun", () => { const runDir = createRun(base, "test-workflow"); - // Extract the timestamp directory name - const parts = runDir.split("/"); - const timestamp = parts[parts.length - 1]; + // Extract the timestamp directory name (use path.sep for cross-platform) + const timestamp = runDir.split(/[/\\]/).pop()!; // Should not contain colons (filesystem-unsafe on Windows) assert.ok(!timestamp.includes(":"), `timestamp should not contain colons: ${timestamp}`);