test(core): windows-portable path expectations; normalize inputStamp joins

First run of the windows CI lane caught POSIX-literal assumptions: expectations
now go through upath.resolve/normalize (drive-letter anchoring) and the
file:// fixture is built with pathToFileURL. collectInputFiles joins via upath
so its output is separator-consistent on Windows too (byte-identical on POSIX,
existing stamps stay valid).

Author: bugra9

cppjs-core/cpp.js/src/utils/inputStamp.js

@@ -1,4 +1,5 @@
 import fs from 'node:fs';
+import upath from 'upath';
 import findFiles from './findFiles.js';
 import { getContentHash, getFileHash } from './hash.js';
 
@@ -13,9 +14,10 @@ export function collectInputFiles(roots, exts, extraFiles = []) {
             cwd: root,
             ignore: EXCLUDED_DIRS.map((dir) => `**/${dir}/**`),
         }).forEach((file) => files.add(file));
-        if (fs.existsSync(`${root}/package.json`)) files.add(`${root}/package.json`);
+        const packageJson = upath.join(root, 'package.json');
+        if (fs.existsSync(packageJson)) files.add(packageJson);
     });
-    extraFiles.forEach((file) => files.add(file));
+    extraFiles.forEach((file) => files.add(upath.normalize(file)));
     return [...files].filter((file) => fs.existsSync(file)).sort();
 }
 

cppjs-core/cpp.js/test/getAbsolutePath.test.js

@@ -11,12 +11,14 @@ describe('getAbsolutePath', () => {
         expect(getAbsolutePath('/some/project', undefined)).toBeNull();
     });
 
+    // upath.resolve anchors POSIX-style absolutes to the current drive on Windows,
+    // so expectations resolve the same way instead of hardcoding the POSIX literal.
     test('returns the path as-is when it is already absolute', () => {
-        expect(getAbsolutePath('/some/project', '/etc/foo')).toBe('/etc/foo');
+        expect(getAbsolutePath('/some/project', '/etc/foo')).toBe(upath.resolve('/etc/foo'));
     });
 
     test('joins relative path against the projectPath when projectPath is provided', () => {
-        expect(getAbsolutePath('/some/project', 'src/native')).toBe('/some/project/src/native');
+        expect(getAbsolutePath('/some/project', 'src/native')).toBe(upath.resolve('/some/project/src/native'));
     });
 
     test('resolves relative path against cwd when projectPath is missing', () => {
@@ -25,6 +27,6 @@ describe('getAbsolutePath', () => {
     });
 
     test('normalizes ../ in relative paths against the projectPath', () => {
-        expect(getAbsolutePath('/some/project/sub', '../sibling')).toBe('/some/project/sibling');
+        expect(getAbsolutePath('/some/project/sub', '../sibling')).toBe(upath.resolve('/some/project/sibling'));
     });
 });

cppjs-core/cpp.js/test/getParentPath.test.js

@@ -1,10 +1,14 @@
 import { describe, test, expect } from 'vitest';
+import { pathToFileURL } from 'node:url';
+import upath from 'upath';
 import getParentPath from '../src/utils/getParentPath.js';
 
 describe('getParentPath', () => {
+    // Built with pathToFileURL so the URL is valid on Windows too (drive letter);
+    // a hardcoded POSIX file:// URL is rejected by fileURLToPath there.
     test('strips the file segment from a file:// URL', () => {
-        const input = 'file:///Users/me/project/cppjs-core/cpp.js/src/index.js';
-        expect(getParentPath(input)).toBe('/Users/me/project/cppjs-core/cpp.js/src');
+        const input = pathToFileURL(upath.resolve('/Users/me/project/cppjs-core/cpp.js/src/index.js')).href;
+        expect(getParentPath(input)).toBe(upath.resolve('/Users/me/project/cppjs-core/cpp.js/src'));
     });
 
     test('strips the file segment from a plain absolute path', () => {

cppjs-core/cpp.js/test/inputStamp.test.js

@@ -2,16 +2,19 @@ import { describe, test, expect, beforeEach, afterEach } from 'vitest';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
+import upath from 'upath';
 import { collectInputFiles, computeInputStamp } from '../src/utils/inputStamp.js';
 
 describe('inputStamp', () => {
     let root;
 
+    // collectInputFiles returns upath-normalized paths (glob posix output), so
+    // expectations normalize the same way for Windows.
     const write = (rel, content = '') => {
         const file = path.join(root, rel);
         fs.mkdirSync(path.dirname(file), { recursive: true });
         fs.writeFileSync(file, content);
-        return file;
+        return upath.normalize(file);
     };
 
     beforeEach(() => {

cppjs-core/cpp.js/test/loadConfig.test.js

@@ -2,6 +2,7 @@ import { describe, test, expect, beforeEach, afterEach } from 'vitest';
 import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
+import upath from 'upath';
 import loadConfig, { getFilledConfig } from '../src/state/loadConfig.js';
 
 describe('excludedDependencies', () => {
@@ -156,7 +157,7 @@ describe('dependency replacement', () => {
         expect(dep.general.name).toBe('gdal');
         expect(dep.general.alias.package).toBe('@cpp.js/package-gdal');
         expect(dep.export.libName).toEqual(['gdal']);
-        expect(dep.paths.project).toBe('/new/aaagdal');
+        expect(dep.paths.project).toBe(upath.resolve('/new/aaagdal'));
     });
 
     test('leaves a dependency untouched when no replace entry matches', () => {
@@ -166,7 +167,7 @@ describe('dependency replacement', () => {
             dependencies: [{ general: { name: 'gdal' }, paths: { project: '/old/gdal' } }],
         };
         const dep = fill(app, { proj: { replace: {} } }).allDependencies[0];
-        expect(dep.paths.project).toBe('/old/gdal');
+        expect(dep.paths.project).toBe(upath.resolve('/old/gdal'));
     });
 });
 
@@ -192,6 +193,6 @@ describe('raw config isolation', () => {
         first.paths.output = '/app/.cppjs/deps/z/dist';
 
         expect(rawDep.paths.project).toBe('/pkg/zlib-wasm');
-        expect(fill(app).allDependencies[0].paths.project).toBe('/pkg/zlib-wasm');
+        expect(fill(app).allDependencies[0].paths.project).toBe(upath.resolve('/pkg/zlib-wasm'));
     });
 });