fix: windoooooows

Author: brianleroux

package.json

@@ -9,10 +9,10 @@
   "scripts": {
     "test": "npm run lint && npm run test:integration && npm run coverage",
     "test:nolint": "npm run test:integration && npm run coverage",
-    "test:unit": "node test/run-with-env.js --test --test-reporter=spec 'test/unit/**/*-test.js'",
+    "test:unit": "node test/run-with-env.js --test --test-reporter=spec \"test/unit/**/*-test.js\"",
     "test:slow": "node --test --test-reporter=spec 'test/slow/**/*-test.js'",
-    "test:integration": "node test/integration/static/index-test.js && node test/integration/static/publish/index-test.js && node test/run-with-env.js --test --test-reporter=spec 'test/integration/macros-n-plugins-test.js'",
-    "coverage": "mkdir -p coverage && node test/run-with-env.js --test --experimental-test-coverage --test-reporter=spec --test-reporter-destination=stdout --test-reporter=lcov --test-reporter-destination=coverage/lcov.info 'test/unit/**/*-test.js'",
+    "test:integration": "node test/integration/static/index-test.js && node test/integration/static/publish/index-test.js && node test/run-with-env.js --test --test-reporter=spec test/integration/macros-n-plugins-test.js",
+    "coverage": "mkdir -p coverage && node test/run-with-env.js --test --experimental-test-coverage --test-reporter=spec --test-reporter-destination=stdout --test-reporter=lcov --test-reporter-destination=coverage/lcov.info \"test/unit/**/*-test.js\"",
     "lint": "eslint . --fix",
     "rc": "npm version prerelease --preid RC"
   },

test/run-with-env.js

@@ -4,14 +4,68 @@ process.env.AWS_ACCESS_KEY_ID = 'blah'
 process.env.AWS_SECRET_ACCESS_KEY = 'blah'
 
 const { spawn } = require('child_process')
+const { readdirSync, statSync } = require('fs')
+const { join } = require('path')
 const args = process.argv.slice(2)
 
-// On Windows, we need to use shell to expand globs
-const isWindows = process.platform === 'win32'
+// Simple glob expansion for **/*-test.js pattern
+function expandGlob (pattern) {
+  if (!pattern.includes('*')) {
+    return [ pattern ]
+  }
 
-const child = spawn('node', args, {
+  // Handle test/unit/**/*-test.js pattern
+  const match = pattern.match(/^(.+?)\/\*\*\/\*(.+)$/)
+  if (match) {
+    const baseDir = match[1]
+    const suffix = match[2]
+    const files = []
+
+    function walk (dir) {
+      try {
+        const entries = readdirSync(dir)
+        for (const entry of entries) {
+          const fullPath = join(dir, entry)
+          try {
+            const stat = statSync(fullPath)
+            if (stat.isDirectory()) {
+              walk(fullPath)
+            }
+            else if (entry.endsWith(suffix)) {
+              files.push(fullPath)
+            }
+          }
+          catch {
+            // Skip files we can't stat
+          }
+        }
+      }
+      catch {
+        // Skip directories we can't read
+      }
+    }
+
+    walk(baseDir)
+    return files
+  }
+
+  return [ pattern ]
+}
+
+// Expand any glob patterns in args
+const expandedArgs = []
+for (const arg of args) {
+  if (arg.includes('*')) {
+    expandedArgs.push(...expandGlob(arg))
+  }
+  else {
+    expandedArgs.push(arg)
+  }
+}
+
+const child = spawn('node', expandedArgs, {
   stdio: 'inherit',
-  shell: isWindows,
+  shell: false,
 })
 
 child.on('exit', (code) => {