Skip to content

Commit 96a209a

Browse files
authored
Merge pull request #1 from audiohacking/copilot/update-tests-resolve-model-file
fix: cross-platform path assertions in resolveModelFile tests
2 parents 4569570 + 6418ed3 commit 96a209a

1 file changed

Lines changed: 15 additions & 5 deletions

File tree

test/paths.test.ts

Lines changed: 15 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,25 @@ describe("resolveModelFile", () => {
2727
expect(resolveModelFile("model.gguf")).toBe("model.gguf");
2828
});
2929

30-
test("joins models dir for bare filename", () => {
30+
test("joins models dir for bare filename (normalized)", () => {
3131
process.env.ACESTEP_MODELS_DIR = "/data/models";
32-
const expected = path.join("/data/models", "dit.gguf");
33-
expect(path.normalize(resolveModelFile("dit.gguf"))).toBe(path.normalize(expected));
32+
const actual = resolveModelFile("dit.gguf");
33+
if (process.platform === "win32") {
34+
expect(actual.toLowerCase()).toContain("\\data\\models\\dit.gguf");
35+
} else {
36+
const expected = path.join("/data/models", "dit.gguf");
37+
expect(path.normalize(actual)).toBe(path.normalize(expected));
38+
}
3439
});
35-
40+
3641
test("joins models dir for bare filename", () => {
3742
process.env.ACESTEP_MODELS_DIR = "/data/models";
38-
expect(resolveModelFile("dit.gguf")).toBe("/data/models/dit.gguf");
43+
const actual = resolveModelFile("dit.gguf");
44+
if (process.platform === "win32") {
45+
expect(actual.toLowerCase()).toContain("\\data\\models\\dit.gguf");
46+
} else {
47+
expect(actual).toBe("/data/models/dit.gguf");
48+
}
3949
});
4050
});
4151

0 commit comments

Comments
 (0)