CI issues

- avoid failing glob; ensure tests rebuild by removing tsconfig.test.tsbuildinfo
- fix YAML parse error by quoting run command with colon/question-mark
- use script for ESM smoke to avoid shell quoting issues across OSes
- skip cross-repo data tests in CI via PUID_SKIP_DATA_TESTS env; use ESM smoke script

Author: dingosky

.github/workflows/ci.yml

@@ -9,6 +9,8 @@ jobs:
   build-test:
     name: build+test (${{ matrix.os }} / node@${{ matrix.node }})
     runs-on: ${{ matrix.os }}
+    env:
+      PUID_SKIP_DATA_TESTS: '1'
     strategy:
       fail-fast: false
       matrix:
@@ -34,7 +36,7 @@ jobs:
         run: yarn build
 
       - name: Node ESM import smoke
-        run: node --input-type=module -e "import { generate, puid } from './build/module/index.mjs'; console.log(typeof generate==='function' && typeof puid==='function' ? 'ok' : 'bad')"
+        run: node scripts/ci/esm-smoke.mjs
 
       - name: Test
         run: yarn test:unit

package.json

@@ -88,7 +88,7 @@
     "prepare-release": "run-s reset-hard test cov:check doc:html version doc:publish",
     "reset-hard": "git clean -dfx && git reset --hard && yarn",
     "test": "run-s build test:*",
-    "test:clean": "rm -rf ./build/*test*",
+    "test:clean": "node -e \"const fs=require('fs');['build/test','build/tsconfig.test.tsbuildinfo'].forEach(p=>{try{fs.rmSync(p,{recursive:true,force:true})}catch{}})\"",
     "test:data": "yarn test:clean && yarn build:test && yarn ava src/data.spec.ts",
     "test:prettier": "prettier \"src/**/*.ts\" --list-different",
     "test:unit": "yarn test:clean && yarn build:test && nyc --silent ava",

scripts/ci/esm-smoke.mjs

@@ -0,0 +1,4 @@
+import { generate, puid } from '../../build/module/index.mjs'
+
+const ok = typeof generate === 'function' && typeof puid === 'function'
+console.log(ok ? 'ok' : 'bad')

src/data.spec.ts

@@ -3,6 +3,9 @@ import test, { ExecutionContext } from 'ava'
 import { default as puid } from './lib/puid'
 import { dataIds, dataParams, fileBytes } from './lib/util'
 
+// Allow skipping data-driven cross-repo tests in CI or when PUID_SKIP_DATA_TESTS is set
+const it = process.env.PUID_SKIP_DATA_TESTS ? test.skip : test
+
 const testData = (t: ExecutionContext, dataName: string) => {
   const params = dataParams(dataName)
   const ids = dataIds(dataName)
@@ -17,14 +20,14 @@ const testData = (t: ExecutionContext, dataName: string) => {
   if (randId) ids.forEach((id) => id !== '' && t.is(id, randId()))
 }
 
-test('alphanum data', (t) => testData(t, 'alphanum'))
+it('alphanum data', (t) => testData(t, 'alphanum'))
 
-test('alpha_10_lower data', (t) => testData(t, 'alpha_10_lower'))
+it('alpha_10_lower data', (t) => testData(t, 'alpha_10_lower'))
 
-test('dingosky data', (t) => testData(t, 'dingosky'))
+it('dingosky data', (t) => testData(t, 'dingosky'))
 
-test('safe32 data', (t) => testData(t, 'safe32'))
+it('safe32 data', (t) => testData(t, 'safe32'))
 
-test('safe_ascii data', (t) => testData(t, 'safe_ascii'))
+it('safe_ascii data', (t) => testData(t, 'safe_ascii'))
 
-test('unicode data', (t) => testData(t, 'unicode'))
+it('unicode data', (t) => testData(t, 'unicode'))