fix: windows hah

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": "AWS_ACCESS_KEY_ID=\"blah\" AWS_SECRET_ACCESS_KEY=\"blah\" node --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": "AWS_ACCESS_KEY_ID=\"blah\" AWS_SECRET_ACCESS_KEY=\"blah\" node test/integration/static/index-test.js && AWS_ACCESS_KEY_ID=\"blah\" AWS_SECRET_ACCESS_KEY=\"blah\" node test/integration/static/publish/index-test.js && AWS_ACCESS_KEY_ID=\"blah\" AWS_SECRET_ACCESS_KEY=\"blah\" node --test --test-reporter=spec 'test/integration/macros-n-plugins-test.js'",
-    "coverage": "mkdir -p coverage && AWS_ACCESS_KEY_ID=\"blah\" AWS_SECRET_ACCESS_KEY=\"blah\" node --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/integration/static/index-test.js

@@ -1,5 +1,10 @@
 #!/usr/bin/env node
 // Custom test runner to avoid Node.js test runner serialization issues
+
+// Set environment variables for AWS (required for tests)
+process.env.AWS_ACCESS_KEY_ID = 'blah'
+process.env.AWS_SECRET_ACCESS_KEY = 'blah'
+
 const { join } = require('path')
 const { mkdtempSync, mkdirSync, writeFileSync, rmSync } = require('fs')
 const { tmpdir } = require('os')

test/integration/static/publish/index-test.js

@@ -1,5 +1,10 @@
 #!/usr/bin/env node
 // Custom test runner to avoid Node.js test runner serialization issues
+
+// Set environment variables for AWS (required for tests)
+process.env.AWS_ACCESS_KEY_ID = 'blah'
+process.env.AWS_SECRET_ACCESS_KEY = 'blah'
+
 const awsLite = require('@aws-lite/client')
 const { join, dirname } = require('path')
 const { mkdtempSync, mkdirSync, writeFileSync, rmSync } = require('fs')

test/run-with-env.js

@@ -0,0 +1,19 @@
+#!/usr/bin/env node
+// Cross-platform script to run tests with environment variables
+process.env.AWS_ACCESS_KEY_ID = 'blah'
+process.env.AWS_SECRET_ACCESS_KEY = 'blah'
+
+const { spawn } = require('child_process')
+const args = process.argv.slice(2)
+
+// On Windows, we need to use shell to expand globs
+const isWindows = process.platform === 'win32'
+
+const child = spawn('node', args, {
+  stdio: 'inherit',
+  shell: isWindows,
+})
+
+child.on('exit', (code) => {
+  process.exit(code || 0)
+})