test: merge autogenerated vendor tests (google#1007)

* test: merge autogenerated vendor tests

continues google#987

* test: optimize test flow

Author: antongolub

.github/workflows/test.yml

@@ -67,33 +67,27 @@ jobs:
     runs-on: ubuntu-latest
     env:
       FORCE_COLOR: 3
-    strategy:
-      matrix:
-        node-version: [16.x, 20.x, 22.x]
+
     steps:
       - uses: actions/checkout@v4
 
-      - name: Use Node.js ${{ matrix.node-version }}
+      - name: Use Node.js 22.x
         uses: actions/setup-node@v4
         with:
-          node-version: ${{ matrix.node-version }}
+          node-version: 22.x
 
       - uses: actions/download-artifact@v4
         with:
           name: build
       - run: npm ci
 
-      - name: Unit & types tests
-        if: matrix.node-version == '20.x'
-        run: |
-          npm run test:coverage
-          npm run test:types
-
       - name: Unit tests
-        if: matrix.node-version != '20.x'
-        run: npm run test:unit
+        run: npm run test:coverage
         timeout-minutes: 1
 
+      - name: Type tests
+        run: npm run test:types
+
   smoke-win32-node16:
     runs-on: windows-latest
     needs: build

scripts/build-tests.mjs

@@ -18,34 +18,37 @@ import fs from 'node:fs'
 import path from 'node:path'
 import * as vendor from '../build/vendor.js'
 
-const __dirname = path.dirname(new URL(import.meta.url).pathname)
-const root = path.resolve(__dirname, '..')
+const root = path.resolve(new URL(import.meta.url).pathname, '../..')
 const apis = ['chalk', 'depseek', 'fs', 'minimist', 'ps', 'which', 'YAML']
-
 const copyright = await fs.readFileSync(
   path.resolve(root, 'test/fixtures/copyright.txt'),
   'utf8'
 )
 
-// prettier-ignore
+const filePath = path.resolve(root, `test/vendor-export.test.js`)
+let fileContents = `${copyright.replace('YEAR', new Date().getFullYear())}
+import assert from 'node:assert'
+import { test, describe } from 'node:test'
+import {
+${apis.map((v) => '  ' + v).join(',\n')},
+} from '../build/vendor.js'
+`
+
 apis.forEach((name) => {
   const api = vendor[name]
   const methods = Object.entries(api)
-  const formatAssert = (k, v, prefix = '    ') => `${prefix}assert.equal(typeof ${name}.${k}, '${typeof v}')`
+  const formatAssert = (k, v, prefix = '    ') =>
+    `${prefix}assert.equal(typeof ${name}.${k}, '${typeof v}', '${name}.${k}')`
   const methodChecks = methods.length
     ? '\n' + methods.map(([k, v]) => formatAssert(k, v)).join('\n')
     : ''
-  const filePath = path.resolve(root, `test/vendor-${name.toLowerCase()}.test.js`)
-  const fileContents = `${copyright.replace('YEAR', new Date().getFullYear())}
-import assert from 'node:assert'
-import { test, describe } from 'node:test'
-import { ${name} } from '../build/vendor.js'
-
+  fileContents += `
 describe('vendor ${name} API ', () => {
   test('exports', () => {
     assert.equal(typeof ${name}, '${typeof api}')${methodChecks}
   })
 })
 `
-  fs.writeFileSync(filePath, fileContents)
 })
+
+fs.writeFileSync(filePath, fileContents)

test/all.test.js

@@ -20,4 +20,5 @@ import './goods.test.js'
 import './index.test.js'
 import './package.test.js'
 import './util.test.js'
-import './vendor.test.js'
+import './vendor-yaml.test.js'
+import './vendor-export.test.js'