fix: drive letter sensitivity on windows (#50)

Fixes #49

Author: connor4312

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - Fix bailing multiple configs on first failure (fixes [#44](https://github.com/microsoft/vscode-test-cli/issues/44))
 - Fix extension install on Windows on modern Node versions (fixes [#43](https://github.com/microsoft/vscode-test-cli/issues/43))
+- Normalize casing in paths (fixes [#49](https://github.com/microsoft/vscode-test-cli/issues/49))
 
 ## 0.0.9 - 2024-03-04
 

src/runner.cts

@@ -27,7 +27,7 @@ export async function run() {
   const required: { mochaGlobalSetup?: () => unknown; mochaGlobalTeardown?: () => unknown }[] = [
     ...preload,
     ...ensureArray(mochaOpts.require),
-  ].map((f) => require(f));
+  ].map((f) => require(normalizeCasing(f)));
 
   // currently `require` only seems to take effect for parallel runs, but remove
   // the option in case it's supported for serial runs in the future since we're
@@ -39,7 +39,7 @@ export async function run() {
   }
 
   for (const file of files) {
-    mocha.addFile(file);
+    mocha.addFile(normalizeCasing(file));
   }
 
   await new Promise<void>((resolve, reject) =>
@@ -55,5 +55,17 @@ export async function run() {
   }
 }
 
+const normalizeCasing = (path: string) => {
+  // Normalize to lower-case drive letter to avoid path sensitivity in the loader
+  // duplicating imports. VS Code normalizes to lower case drive letters in its
+  // URIs, so do the same here
+  // https://github.com/microsoft/vscode/blob/032c1b75447ade317715c3d2a82c2d9cd3e55dde/src/vs/base/common/uri.ts#L181-L185
+  if (process.platform === 'win32' && path.match(/^[A-Z]:/)) {
+    return path[0].toLowerCase() + path.slice(1);
+  }
+
+  return path;
+};
+
 const ensureArray = <T,>(value: T | T[] | undefined): T[] =>
   value ? (Array.isArray(value) ? value : [value]) : [];