polendina 3.3.1 → 3.3.3

package/test/common.js

@@ -1,19 +1,37 @@
 import path from 'path'
 import assert from 'assert'
 import { execFile } from 'child_process'
-import { fileURLToPath } from 'url'
 
-const __dirname = fileURLToPath(path.dirname(import.meta.url))
-const cli = path.join(__dirname, '../polendina-cli.js')
+// consumers installed by setup.js, one per runner, named by its variants table
+export function consumers () {
+  return JSON.parse(process.env.POLENDINA_CONSUMERS)
+}
+
+export function consumer (name) {
+  const dir = consumers()[name]
+  if (!dir) {
+    throw new Error(`no consumer named ${name}`)
+  }
+  return dir
+}
+
+// runs the installed polendina CLI from inside a copied fixture directory;
+// resolved at call time because consumers exist only once setup.js has run
+export async function runCli (consumerName, fixture, args = []) {
+  const dir = consumer(consumerName)
+  const cli = path.join(dir, 'node_modules/polendina/polendina-cli.js')
+  return runCommand(cli, ['test*.js', 'test/test*.js', '--cleanup', ...args], path.join(dir, 'fixtures', fixture))
+}
 
-export async function runCli (cwd, args) {
-  return runCommand(`${cli} test*.js test/test*.js --cleanup ${args || ''}`, cwd)
+export async function runNodeCli (consumerName, fixture, mode) {
+  const dir = consumer(consumerName)
+  const cli = path.join(dir, 'node_modules/polendina/polendina-node-cli.js')
+  return runCommand(cli, [mode, 'test*.js'], path.join(dir, 'fixtures', fixture))
 }
 
-export async function runCommand (command, cwd) {
+export async function runCommand (script, args, cwd) {
   return new Promise((resolve, reject) => {
-    const args = command.split(' ').filter(Boolean)
-    execFile(process.execPath, args, { cwd }, (err, stdout, stderr) => {
+    execFile(process.execPath, [script, ...args], { cwd }, (err, stdout, stderr) => {
       const code = err ? err.code : 0
       if (!code) {
         try {
@@ -22,8 +40,7 @@ export async function runCommand (command, cwd) {
           return reject(e)
         }
       }
-      const out = { stdout: stdout.toString(), stderr: stderr.toString(), code }
-      resolve(out)
+      resolve({ stdout: stdout.toString(), stderr: stderr.toString(), code })
     })
   })
 }

package/test/consumer/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "polendina-test-consumer",
+  "version": "1.0.0",
+  "private": true,
+  "description": "Version ranges for the test frameworks that test/setup.js installs, one runner per temporary consumer, alongside a packed polendina",
+  "devDependencies": {
+    "chai": "^6.0.1",
+    "mocha": "^12.0.2",
+    "tape": "^5.6.3"
+  }
+}

package/test/fixtures/bare-async/package.json

@@ -1,18 +1,5 @@
 {
   "name": "fixture-bare-async",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true
 }

package/test/fixtures/bare-async-esm/package.json

@@ -1,19 +1,6 @@
 {
   "name": "fixture-bare-async",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/bare-async-failure/package.json

@@ -1,18 +1,5 @@
 {
   "name": "fixture-bare-async-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true
 }

package/test/fixtures/bare-async-failure-esm/package.json

@@ -1,19 +1,6 @@
 {
   "name": "fixture-bare-async-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/bare-sync/package.json

@@ -1,18 +1,5 @@
 {
   "name": "fixture-bare-sync",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true
 }

package/test/fixtures/bare-sync-esm/package.json

@@ -1,19 +1,6 @@
 {
   "name": "fixture-bare-sync",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/bare-sync-failure/package.json

@@ -1,18 +1,5 @@
 {
   "name": "fixture-bare-sync-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true
 }

package/test/fixtures/bare-sync-failure-esm/package.json

@@ -1,19 +1,6 @@
 {
   "name": "fixture-bare-sync-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "node test-1.js && node test-2.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/mocha/package.json

@@ -1,19 +1,5 @@
 {
   "name": "fixture-mocha",
   "version": "1.0.0",
-  "description": "test-fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "mocha test-*.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "chai": "^6.0.1",
-    "mocha": "^12.0.2"
-  }
+  "private": true
 }

package/test/fixtures/mocha-esm/package.json

@@ -1,20 +1,6 @@
 {
   "name": "fixture-mocha",
   "version": "1.0.0",
-  "description": "test-fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "mocha test-*.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "chai": "^6.0.1",
-    "mocha": "^12.0.2"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/mocha-failure/package.json

@@ -1,19 +1,5 @@
 {
   "name": "fixture-mocha-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "mocha test-*.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "chai": "^6.0.1",
-    "mocha": "^12.0.2"
-  }
+  "private": true
 }

package/test/fixtures/mocha-failure-esm/package.json

@@ -1,20 +1,6 @@
 {
   "name": "fixture-mocha-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "mocha test-*.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "chai": "^6.0.1",
-    "mocha": "^12.0.2"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/tape/package.json

@@ -1,18 +1,5 @@
 {
   "name": "fixture-tape",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "tape test.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "tape": "^5.6.3"
-  }
+  "private": true
 }

package/test/fixtures/tape-esm/package.json

@@ -1,19 +1,6 @@
 {
   "name": "fixture-tape",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "tape test.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "tape": "^5.6.3"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/tape-failure/package.json

@@ -1,18 +1,5 @@
 {
   "name": "fixture-tape-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "tape test.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "tape": "^5.6.3"
-  }
+  "private": true
 }

package/test/fixtures/tape-failure-esm/package.json

@@ -1,19 +1,6 @@
 {
   "name": "fixture-tape-failure",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "type": "module",
-  "scripts": {
-    "test": "tape test.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "devDependencies": {
-    "tape": "^5.6.3"
-  }
+  "private": true,
+  "type": "module"
 }

package/test/fixtures/webpack-merge/package.json

@@ -1,18 +1,5 @@
 {
   "name": "fixture-webpack-merge",
   "version": "1.0.0",
-  "description": "test fixture",
-  "main": "index.js",
-  "scripts": {
-    "test": "node test.js"
-  },
-  "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/rvagg/polendina.git"
-  },
-  "license": "Apache-2.0",
-  "devDependencies": {
-    "chai": "^6.0.1"
-  }
+  "private": true
 }

package/test/setup.js

@@ -0,0 +1,96 @@
+// Mocha global fixtures. Packs polendina and installs it into temporary
+// consumer directories outside the repository, one per test runner, each with
+// only the frameworks that runner's fixtures import at the versions in
+// test/consumer/package.json, then copies test/fixtures in. npm hoists polendina's dependencies into each consumer's node_modules,
+// the layout a real install has and one the fixtures cannot produce from
+// inside this tree. Keeping the runners apart means no runner's dependency
+// tree can shadow or nest another's.
+//
+// Tests find the consumers through POLENDINA_CONSUMERS, a JSON object of
+// name to directory. Set POLENDINA_KEEP_CONSUMERS to leave them behind.
+
+import fs from 'fs'
+import os from 'os'
+import path from 'path'
+import { exec } from 'child_process'
+import { fileURLToPath } from 'url'
+
+const __dirname = fileURLToPath(path.dirname(import.meta.url))
+const root = path.join(__dirname, '..')
+const puppeteerVersion = JSON.parse(fs.readFileSync(path.join(root, 'node_modules/puppeteer/package.json'))).version
+const template = JSON.parse(fs.readFileSync(path.join(__dirname, 'consumer/package.json'))).devDependencies
+
+// consumer name to the packages it installs alongside polendina; a bare name
+// takes its version range from test/consumer/package.json, name@range pins one
+const variants = {
+  mocha: ['mocha', 'chai'],
+  mocha11: ['mocha@11', 'chai'],
+  tape: ['tape'],
+  bare: ['chai']
+}
+
+const consumers = {}
+
+function dependency (pkg) {
+  const at = pkg.indexOf('@', 1)
+  if (at > 0) {
+    return [pkg.slice(0, at), pkg.slice(at + 1)]
+  }
+  if (!template[pkg]) {
+    throw new Error(`${pkg} is not in test/consumer/package.json`)
+  }
+  return [pkg, template[pkg]]
+}
+
+function sh (command, cwd) {
+  return new Promise((resolve, reject) => {
+    const env = { ...process.env, PUPPETEER_SKIP_DOWNLOAD: '1' }
+    exec(command, { cwd, env }, (err, stdout, stderr) => {
+      if (err) {
+        return reject(new Error(`${command}\n${stderr}`))
+      }
+      resolve(stdout.toString())
+    })
+  })
+}
+
+async function installConsumer (name, tarball, packages) {
+  const dir = await fs.promises.mkdtemp(path.join(os.tmpdir(), `polendina-consumer-${name}-`))
+  const pkg = {
+    name: `polendina-consumer-${name}`,
+    private: true,
+    devDependencies: {
+      polendina: `file:${tarball}`,
+      puppeteer: puppeteerVersion,
+      ...Object.fromEntries(packages.map(dependency))
+    }
+  }
+  await fs.promises.writeFile(path.join(dir, 'package.json'), JSON.stringify(pkg, null, 2))
+  await fs.promises.cp(path.join(__dirname, 'fixtures'), path.join(dir, 'fixtures'), {
+    recursive: true,
+    filter: (src) => path.basename(src) !== 'node_modules'
+  })
+  await sh('npm install --no-audit --no-fund --no-package-lock --prefer-offline', dir)
+  return dir
+}
+
+export async function mochaGlobalSetup () {
+  const packDir = await fs.promises.mkdtemp(path.join(os.tmpdir(), 'polendina-pack-'))
+  const [{ filename }] = JSON.parse(await sh(`npm pack --json --pack-destination "${packDir}"`, root))
+  const tarball = path.join(packDir, filename)
+  await Promise.all(Object.entries(variants).map(async ([name, packages]) => {
+    consumers[name] = await installConsumer(name, tarball, packages)
+  }))
+  await fs.promises.rm(packDir, { recursive: true, force: true })
+  process.env.POLENDINA_CONSUMERS = JSON.stringify(consumers)
+}
+
+export async function mochaGlobalTeardown () {
+  if (process.env.POLENDINA_KEEP_CONSUMERS) {
+    console.log('Keeping consumers:', consumers)
+    return
+  }
+  for (const dir of Object.values(consumers)) {
+    await fs.promises.rm(dir, { recursive: true, force: true })
+  }
+}

package/test/test-bare-async.js

@@ -1,15 +1,11 @@
 /* globals describe it */
 
 import assert from 'assert'
-import path from 'path'
-import { fileURLToPath } from 'url'
 import { runCli } from './common.js'
 
-const __dirname = fileURLToPath(path.dirname(import.meta.url))
-
 for (const type of ['cjs', 'esm']) {
-  const bareAsyncFixture = path.join(__dirname, `fixtures/bare-async${type === 'esm' ? '-esm' : ''}`)
-  const bareAsyncFailureFixture = path.join(__dirname, `fixtures/bare-async-failure${type === 'esm' ? '-esm' : ''}`)
+  const bareAsyncFixture = `bare-async${type === 'esm' ? '-esm' : ''}`
+  const bareAsyncFailureFixture = `bare-async-failure${type === 'esm' ? '-esm' : ''}`
 
   describe(`basic bare-async (${type})`, function () {
     this.timeout(60000)
@@ -27,7 +23,7 @@ testing bare fixture subdir
 `
 
     it('should run in page', async () => {
-      const { stdout, stderr, code } = await runCli(bareAsyncFixture, '--runner=bare-async')
+      const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -41,7 +37,7 @@ testing bare fixture subdir
     })
 
     it('should run in worker', async () => {
-      const { stdout, stderr, code } = await runCli(bareAsyncFixture, '--runner=bare-async --worker --page=false')
+      const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async', '--worker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -55,7 +51,7 @@ testing bare fixture subdir
     })
 
     it('should run in serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(bareAsyncFixture, '--runner=bare-async --serviceworker --page=false')
+      const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async', '--serviceworker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -69,7 +65,7 @@ testing bare fixture subdir
     })
 
     it('should run in page, worker and serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(bareAsyncFixture, '--runner=bare-async --worker --serviceworker')
+      const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async', '--worker', '--serviceworker'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -111,7 +107,7 @@ testing bare fixture
     const expectedStderr = 'AssertionError: faily mcfailface'
 
     it('should fail in page', async () => {
-      let { stdout, stderr, code } = await runCli(bareAsyncFailureFixture, '--runner=bare-async')
+      let { stdout, stderr, code } = await runCli('bare', bareAsyncFailureFixture, ['--runner=bare-async'])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = stdout.replace(/^ +at .*\n/gm, '') // stack traces
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')
@@ -124,7 +120,7 @@ testing bare fixture
     })
 
     it('should fail in worker', async () => {
-      let { stdout, stderr, code } = await runCli(bareAsyncFailureFixture, '--runner=bare-async --worker --page=false')
+      let { stdout, stderr, code } = await runCli('bare', bareAsyncFailureFixture, ['--runner=bare-async', '--worker', '--page=false'])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = stdout.replace(/^ +at .*\n/gm, '') // stack traces
       const expected = expectedTemplate.replace(/WORKER/, 'in worker')
@@ -137,7 +133,7 @@ testing bare fixture
     })
 
     it('should fail in serviceworker', async () => {
-      let { stdout, stderr, code } = await runCli(bareAsyncFailureFixture, '--runner=bare-async --serviceworker --page=false')
+      let { stdout, stderr, code } = await runCli('bare', bareAsyncFailureFixture, ['--runner=bare-async', '--serviceworker', '--page=false'])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = stdout.replace(/^ +at .*\n/gm, '') // stack traces
       const expected = expectedTemplate.replace(/WORKER/, 'in serviceworker')
@@ -150,7 +146,7 @@ testing bare fixture
     })
 
     it('should fail in page and not run in worker', async () => {
-      let { stdout, stderr, code } = await runCli(bareAsyncFailureFixture, '--runner=bare-async --worker')
+      let { stdout, stderr, code } = await runCli('bare', bareAsyncFailureFixture, ['--runner=bare-async', '--worker'])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = stdout.replace(/^ +at .*\n/gm, '') // stack traces
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')