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) <noreply@anthropic.com>
This commit is contained in:
Lex Christopherson 2026-03-22 09:13:31 -06:00
parent 2aa01b8a60
commit 97241ea19c

View file

@ -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}`);