Skip to content

Commit 5e2e02c

Browse files
authored
fix: use path.resolve in Codex adapter test for Windows compatibility (Fission-AI#624)
The test expected path.join('/custom/codex-home', ...) but the implementation uses path.resolve() which adds the drive letter on Windows (e.g. D:\). Align the test expectation with the implementation.
1 parent a3cee3c commit 5e2e02c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

test/core/command-generation/adapters.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ describe('command-generation/adapters', () => {
235235
process.env.CODEX_HOME = '/custom/codex-home';
236236
try {
237237
const filePath = codexAdapter.getFilePath('explore');
238-
expect(filePath).toBe(path.join('/custom/codex-home', 'prompts', 'opsx-explore.md'));
238+
expect(filePath).toBe(path.join(path.resolve('/custom/codex-home'), 'prompts', 'opsx-explore.md'));
239239
} finally {
240240
if (original !== undefined) {
241241
process.env.CODEX_HOME = original;

0 commit comments

Comments
 (0)