fix: idk

Author: brianleroux

package.json

@@ -11,7 +11,7 @@
     "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:slow": "node --test --test-reporter=spec 'test/slow/**/*-test.js'",
-    "test:integration": "AWS_ACCESS_KEY_ID=\"blah\" AWS_SECRET_ACCESS_KEY=\"blah\" node --test --test-reporter=spec 'test/integration/**/*-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 --test-reporter=spec 'test/integration/static/publish/*-test.js' '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'",
     "lint": "eslint . --fix",
     "rc": "npm version prerelease --preid RC"

test/integration/static/index-test.js

@@ -1,12 +1,14 @@
-const { test, after } = require('node:test')
-const assert = require('node:assert/strict')
+#!/usr/bin/env node
+// Custom test runner to avoid Node.js test runner serialization issues
 const { join } = require('path')
 const { mkdtempSync, mkdirSync, writeFileSync, rmSync } = require('fs')
 const { tmpdir } = require('os')
 const inventory = require('@architect/inventory')
 const { updater } = require('@architect/utils')
 const staticDeployMod = require(join(process.cwd(), 'src', 'static', 'index.js'))
 
+let passed = 0
+let failed = 0
 let tmpDirs = []
 
 function createTmpDir (structure) {
@@ -33,7 +35,7 @@ function createTmpDir (structure) {
   return tmpDir
 }
 
-after(() => {
+function cleanup () {
   tmpDirs.forEach(dir => {
     try {
       rmSync(dir, { recursive: true, force: true })
@@ -42,7 +44,7 @@ after(() => {
       // Ignore cleanup errors
     }
   })
-})
+}
 
 function staticDeploy (cwd, testParams, callback) {
   inventory({ cwd }, function (err, result) {
@@ -66,57 +68,85 @@ function staticDeploy (cwd, testParams, callback) {
   })
 }
 
-test('Set up env', () => {
-  assert.ok(staticDeployMod, 'Static asset deployment module is present')
-})
-
-test('Skip static deploy if @static is not defined', async () => {
-  let arc = '@app\n an-app'
-  let cwd = createTmpDir({ 'app.arc': arc })
-  await new Promise((resolve, reject) => {
-    staticDeploy(cwd, {}, err => {
-      if (err) reject(err)
-      else resolve()
-    })
+async function test (name, fn) {
+  try {
+    await fn()
+    console.log(`✔ ${name}`)
+    passed++
+  }
+  catch (err) {
+    console.error(`✖ ${name}`)
+    console.error(err)
+    failed++
+  }
+}
+
+async function main () {
+  await test('Set up env', async () => {
+    if (!staticDeployMod) throw new Error('Static asset deployment module is not present')
   })
-})
-
-test('Static deploy exits gracefully if @http is defined but public folder is not present', async () => {
-  let arc = '@app\n an-app\n @http'
-  let cwd = createTmpDir({ 'app.arc': arc })
-  await new Promise((resolve, reject) => {
-    staticDeploy(cwd, {}, err => {
-      if (err) reject(err)
-      else resolve()
+
+  await test('Skip static deploy if @static is not defined', async () => {
+    let arc = '@app\n an-app'
+    let cwd = createTmpDir({ 'app.arc': arc })
+    await new Promise((resolve, reject) => {
+      staticDeploy(cwd, {}, err => {
+        if (err) reject(err)
+        else resolve()
+      })
     })
   })
-})
 
-test('Static deploy skips when isDryRun is true', async () => {
-  let arc = '@app\n an-app\n @static'
-  let cwd = createTmpDir({
-    'app.arc': arc,
-    'public': {},
+  await test('Static deploy exits gracefully if @http is defined but public folder is not present', async () => {
+    let arc = '@app\n an-app\n @http'
+    let cwd = createTmpDir({ 'app.arc': arc })
+    await new Promise((resolve, reject) => {
+      staticDeploy(cwd, {}, err => {
+        if (err) reject(err)
+        else resolve()
+      })
+    })
   })
-  await new Promise((resolve, reject) => {
-    staticDeploy(cwd, { isDryRun: true, bucket: 'test-bucket' }, err => {
-      if (err) reject(err)
-      else resolve()
+
+  await test('Static deploy skips when isDryRun is true', async () => {
+    let arc = '@app\n an-app\n @static'
+    let cwd = createTmpDir({
+      'app.arc': arc,
+      'public': {},
+    })
+    await new Promise((resolve, reject) => {
+      staticDeploy(cwd, { isDryRun: true, bucket: 'test-bucket' }, err => {
+        if (err) reject(err)
+        else resolve()
+      })
     })
   })
-})
-
-test('Static deploy skips when @http is defined and public folder is not present', async () => {
-  let arc = '@app\n an-app\n @http'
-  let cwd = createTmpDir({ 'app.arc': arc })
-  await new Promise((resolve, reject) => {
-    staticDeploy(cwd, { bucket: 'test-bucket' }, err => {
-      if (err) reject(err)
-      else resolve()
+
+  await test('Static deploy skips when @http is defined and public folder is not present', async () => {
+    let arc = '@app\n an-app\n @http'
+    let cwd = createTmpDir({ 'app.arc': arc })
+    await new Promise((resolve, reject) => {
+      staticDeploy(cwd, { bucket: 'test-bucket' }, err => {
+        if (err) reject(err)
+        else resolve()
+      })
     })
   })
-})
 
-test('Teardown', () => {
-  assert.ok(true, 'Done')
-})
+  await test('Teardown', async () => {
+    // Cleanup complete
+  })
+
+  cleanup()
+
+  console.log(`\nℹ tests ${passed + failed}`)
+  console.log(`ℹ pass ${passed}`)
+  console.log(`ℹ fail ${failed}`)
+
+  process.exit(failed > 0 ? 1 : 0)
+}
+
+// Only run if executed directly
+if (require.main === module) {
+  main()
+}