polendina 3.3.1 → 3.3.3

package/test/test-bare-sync.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 bareSyncFixture = path.join(__dirname, `fixtures/bare-sync${type === 'esm' ? '-esm' : ''}`)
-  const bareSyncFailureFixture = path.join(__dirname, `fixtures/bare-sync-failure${type === 'esm' ? '-esm' : ''}`)
+  const bareSyncFixture = `bare-sync${type === 'esm' ? '-esm' : ''}`
+  const bareSyncFailureFixture = `bare-sync-failure${type === 'esm' ? '-esm' : ''}`
 
   describe(`basic bare-sync (${type})`, function () {
     this.timeout(60000)
@@ -23,7 +19,7 @@ testing bare fixture subdir
 `
 
     it('should run in page', async () => {
-      const { stdout, stderr, code } = await runCli(bareSyncFixture, '--runner=bare-sync')
+      const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -37,7 +33,7 @@ testing bare fixture subdir
     })
 
     it('should run in worker', async () => {
-      const { stdout, stderr, code } = await runCli(bareSyncFixture, '--runner=bare-sync --worker --page=false')
+      const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync', '--worker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -51,7 +47,7 @@ testing bare fixture subdir
     })
 
     it('should run in serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(bareSyncFixture, '--runner=bare-sync --serviceworker --page=false')
+      const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync', '--serviceworker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -65,7 +61,7 @@ testing bare fixture subdir
     })
 
     it('should run in page, worker and serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(bareSyncFixture, '--runner=bare-sync --worker --serviceworker')
+      const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync', '--worker', '--serviceworker'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -105,7 +101,7 @@ testing bare fixture
     const expectedStderr = 'AssertionError: faily mcfailface'
 
     it('should fail in page', async () => {
-      let { stdout, stderr, code } = await runCli(bareSyncFailureFixture, '--runner=bare-sync')
+      let { stdout, stderr, code } = await runCli('bare', bareSyncFailureFixture, ['--runner=bare-sync'])
       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')
@@ -118,7 +114,7 @@ testing bare fixture
     })
 
     it('should fail in worker', async () => {
-      let { stdout, stderr, code } = await runCli(bareSyncFailureFixture, '--runner=bare-sync --worker --page=false')
+      let { stdout, stderr, code } = await runCli('bare', bareSyncFailureFixture, ['--runner=bare-sync', '--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')
@@ -131,7 +127,7 @@ testing bare fixture
     })
 
     it('should fail in serviceworker', async () => {
-      let { stdout, stderr, code } = await runCli(bareSyncFailureFixture, '--runner=bare-sync --serviceworker --page=false')
+      let { stdout, stderr, code } = await runCli('bare', bareSyncFailureFixture, ['--runner=bare-sync', '--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')
@@ -144,7 +140,7 @@ testing bare fixture
     })
 
     it('should fail in page and not run in worker', async () => {
-      let { stdout, stderr, code } = await runCli(bareSyncFailureFixture, '--runner=bare-sync --worker')
+      let { stdout, stderr, code } = await runCli('bare', bareSyncFailureFixture, ['--runner=bare-sync', '--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')

package/test/test-mocha.js

@@ -1,37 +1,37 @@
 /* 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))
-
 function removeTimings (stdout) {
   return stdout.replace(/\(\d+ms\)/g, '(Xms)')
 }
 
-for (const type of ['cjs', 'esm']) {
-  const mochaFixture = path.join(__dirname, `fixtures/mocha${type === 'esm' ? '-esm' : ''}`)
-  const mochaFailureFixture = path.join(__dirname, `fixtures/mocha-failure${type === 'esm' ? '-esm' : ''}`)
+// the consumers to run against, with the spec reporter's pass symbol, which
+// differs by major in the browser bundle
+const ticks = { mocha: '\u2714', mocha11: '\u2705' }
+
+for (const [consumerName, type] of Object.keys(ticks).flatMap((c) => ['cjs', 'esm'].map((t) => [c, t]))) {
+  const mochaFixture = `mocha${type === 'esm' ? '-esm' : ''}`
+  const mochaFailureFixture = `mocha-failure${type === 'esm' ? '-esm' : ''}`
 
-  describe(`basic mocha (${type})`, function () {
+  describe(`basic mocha (${type}, ${consumerName})`, function () {
     this.timeout(60000)
     const expectedTemplate = `
   test suite 1
-    ✔ test case 1
-    ✔ test case 2
+    ${ticks[consumerName]} test case 1
+    ${ticks[consumerName]} test case 2
 
   test suite 2 - worker
-    ✔ is WORKER
+    ${ticks[consumerName]} is WORKER
 
   test suite 3
-    ✔ test case 1
-    ✔ test case 2
+    ${ticks[consumerName]} test case 1
+    ${ticks[consumerName]} test case 2
 `
 
     it('should run in page', async () => {
-      const { stdout, stderr, code } = await runCli(mochaFixture)
+      const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, [])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -45,7 +45,7 @@ for (const type of ['cjs', 'esm']) {
     })
 
     it('should run in worker', async () => {
-      const { stdout, stderr, code } = await runCli(mochaFixture, '--worker --page=false')
+      const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, ['--worker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -59,7 +59,7 @@ for (const type of ['cjs', 'esm']) {
     })
 
     it('should run in serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(mochaFixture, '--serviceworker --page=false')
+      const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, ['--serviceworker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -73,7 +73,7 @@ for (const type of ['cjs', 'esm']) {
     })
 
     it('should run in page, worker and serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(mochaFixture, '--worker --serviceworker')
+      const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, ['--worker', '--serviceworker'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -102,11 +102,11 @@ for (const type of ['cjs', 'esm']) {
     })
   })
 
-  describe(`failing mocha (${type})`, function () {
+  describe(`failing mocha (${type}, ${consumerName})`, function () {
     this.timeout(60000)
     const expectedTemplate = `
   test suite 1 - worker
-    ✔ is WORKER
+    ${ticks[consumerName]} is WORKER
 
   test suite 2 - failing
     1) should fail
@@ -121,7 +121,7 @@ for (const type of ['cjs', 'esm']) {
 `
 
     it('should fail in page', async () => {
-      let { stdout, code } = await runCli(mochaFailureFixture)
+      let { stdout, code } = await runCli(consumerName, mochaFailureFixture, [])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = removeTimings(stdout)
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')
@@ -133,7 +133,7 @@ for (const type of ['cjs', 'esm']) {
     })
 
     it('should fail in worker', async () => {
-      let { stdout, code } = await runCli(mochaFailureFixture, '--worker --page=false')
+      let { stdout, code } = await runCli(consumerName, mochaFailureFixture, ['--worker', '--page=false'])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = removeTimings(stdout)
       const expected = expectedTemplate.replace(/WORKER/, 'in worker')
@@ -145,7 +145,7 @@ for (const type of ['cjs', 'esm']) {
     })
 
     it('should fail in serviceworker', async () => {
-      let { stdout, code } = await runCli(mochaFailureFixture, '--serviceworker --page=false')
+      let { stdout, code } = await runCli(consumerName, mochaFailureFixture, ['--serviceworker', '--page=false'])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = removeTimings(stdout)
       const expected = expectedTemplate.replace(/WORKER/, 'in serviceworker')
@@ -157,7 +157,7 @@ for (const type of ['cjs', 'esm']) {
     })
 
     it('should fail in page and not run in worker', async () => {
-      let { stdout, code } = await runCli(mochaFailureFixture, '--worker')
+      let { stdout, code } = await runCli(consumerName, mochaFailureFixture, ['--worker'])
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       stdout = removeTimings(stdout)
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')

package/test/test-packed-install.js

@@ -0,0 +1,47 @@
+/* globals describe it */
+
+// setup.js installs polendina from a packed tarball into consumers outside
+// this repository; check that layout is the hoisted one a real install has
+// and that each consumer carries only its own runner
+
+import assert from 'assert'
+import fs from 'fs'
+import path from 'path'
+import { consumers } from './common.js'
+
+const present = (dir, ...segments) => fs.promises.access(path.join(dir, ...segments))
+const absent = (dir, ...segments) => assert.rejects(present(dir, ...segments))
+
+describe('packed install', function () {
+  it('hoists polendina dependencies into every consumer tree', async () => {
+    for (const dir of Object.values(consumers())) {
+      await absent(dir, 'node_modules/polendina/node_modules/stream-browserify')
+      await present(dir, 'node_modules/stream-browserify')
+    }
+  })
+
+  it('installs each runner only in its own consumer', async () => {
+    const { mocha, mocha11, tape, bare } = consumers()
+    await present(mocha, 'node_modules/mocha')
+    await present(mocha11, 'node_modules/mocha')
+    await present(tape, 'node_modules/tape')
+    await absent(mocha, 'node_modules/tape')
+    await absent(tape, 'node_modules/mocha')
+    await absent(bare, 'node_modules/mocha')
+    await absent(bare, 'node_modules/tape')
+  })
+
+  it('installs the previous mocha major in its consumer', async () => {
+    const { mocha, mocha11 } = consumers()
+    const major = (dir) => JSON.parse(fs.readFileSync(path.join(dir, 'node_modules/mocha/package.json'))).version.split('.')[0]
+    assert.strictEqual(major(mocha11), '11')
+    assert.notStrictEqual(major(mocha), '11')
+  })
+
+  it('does not install mocha or tape in polendina itself', async () => {
+    for (const dir of Object.values(consumers())) {
+      await absent(dir, 'node_modules/polendina/node_modules/mocha')
+      await absent(dir, 'node_modules/polendina/node_modules/tape')
+    }
+  })
+})

package/test/test-polendina-node.js

@@ -1,27 +1,17 @@
 /* globals describe it */
 
 import assert from 'assert'
-import path from 'path'
-import { fileURLToPath } from 'url'
-import { runCommand } from './common.js'
-
-const __dirname = fileURLToPath(path.dirname(import.meta.url))
-
-const cli = path.join(__dirname, '../polendina-node-cli.js')
-
-function runCli (mode, cwd) {
-  return runCommand(`${cli} ${mode} test*.js`, cwd)
-}
+import { runNodeCli } from './common.js'
 
 function removeTiming (stdout) {
   return stdout.replace(/Took [\d.]+ seconds/, 'Took X seconds')
 }
 
 for (const type of ['cjs', 'esm']) {
-  const bareSyncFixture = path.join(__dirname, `fixtures/bare-sync${type === 'esm' ? '-esm' : ''}`)
-  const bareSyncFailureFixture = path.join(__dirname, `fixtures/bare-sync-failure${type === 'esm' ? '-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 bareSyncFixture = `bare-sync${type === 'esm' ? '-esm' : ''}`
+  const bareSyncFailureFixture = `bare-sync-failure${type === 'esm' ? '-esm' : ''}`
+  const bareAsyncFixture = `bare-async${type === 'esm' ? '-esm' : ''}`
+  const bareAsyncFailureFixture = `bare-async-failure${type === 'esm' ? '-esm' : ''}`
 
   describe(`polendina-node bare-sync (${type})`, function () {
     this.timeout(60000)
@@ -33,7 +23,7 @@ testing bare fixture
   ✔ test-2.js
 Took X seconds
 `
-      const { stdout, stderr, code } = await runCli('bare-sync', bareSyncFixture)
+      const { stdout, stderr, code } = await runNodeCli('bare', bareSyncFixture, 'bare-sync')
       if (code !== 0) {
         console.error(stderr)
       }
@@ -49,7 +39,7 @@ testing bare fixture
   ✘ test-2.js
 Took X seconds
 `
-      const { stdout, stderr, code } = await runCli('bare-sync', bareSyncFailureFixture)
+      const { stdout, stderr, code } = await runNodeCli('bare', bareSyncFailureFixture, 'bare-sync')
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       assert.strictEqual(removeTiming(stdout), expected)
       assert.ok(stderr.includes('AssertionError: faily mcfailface'), 'stderr contains expected output')
@@ -68,7 +58,7 @@ testing bare fixture
     ✔ test2
 Took X seconds
 `
-      const { stdout, stderr, code } = await runCli('bare-async', bareAsyncFixture)
+      const { stdout, stderr, code } = await runNodeCli('bare', bareAsyncFixture, 'bare-async')
       if (code !== 0) {
         console.error(stderr)
       }
@@ -86,7 +76,7 @@ testing bare fixture
     ✘ test2
 Took X seconds
 `
-      const { stdout, stderr, code } = await runCli('bare-async', bareAsyncFailureFixture)
+      const { stdout, stderr, code } = await runNodeCli('bare', bareAsyncFailureFixture, 'bare-async')
       assert.strictEqual(code, 1, 'exited with non-zero exit code')
       assert.strictEqual(removeTiming(stdout), expected)
       assert.ok(stderr.includes('AssertionError: faily mcfailface'), 'stderr contains expected output')

package/test/test-tape.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 tapeFixture = path.join(__dirname, `fixtures/tape${type === 'esm' ? '-esm' : ''}`)
-  const tapeFailureFixture = path.join(__dirname, `fixtures/tape-failure${type === 'esm' ? '-esm' : ''}`)
+  const tapeFixture = `tape${type === 'esm' ? '-esm' : ''}`
+  const tapeFailureFixture = `tape-failure${type === 'esm' ? '-esm' : ''}`
 
   describe(`basic tape (${type})`, function () {
     this.timeout(60000)
@@ -37,7 +33,7 @@ ok 5 all good
 `
 
     it('should run in page', async () => {
-      const { stdout, stderr, code } = await runCli(tapeFixture, '--runner=tape')
+      const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -51,7 +47,7 @@ ok 5 all good
     })
 
     it('should run in worker', async () => {
-      const { stdout, stderr, code } = await runCli(tapeFixture, '--runner=tape --worker --page=false')
+      const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape', '--worker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -65,7 +61,7 @@ ok 5 all good
     })
 
     it('should run in serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(tapeFixture, '--runner=tape --serviceworker --page=false')
+      const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape', '--serviceworker', '--page=false'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -79,7 +75,7 @@ ok 5 all good
     })
 
     it('should run in page, worker and serviceworker', async () => {
-      const { stdout, stderr, code } = await runCli(tapeFixture, '--runner=tape --worker --serviceworker')
+      const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape', '--worker', '--serviceworker'])
       if (code !== 0) {
         console.error(stderr)
       }
@@ -131,7 +127,7 @@ not ok 2 bork
 `
 
     it('should fail in page', async () => {
-      let { stdout, code } = await runCli(tapeFailureFixture, '--runner=tape')
+      let { stdout, code } = await runCli('tape', tapeFailureFixture, ['--runner=tape'])
       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')
@@ -143,7 +139,7 @@ not ok 2 bork
     })
 
     it('should fail in worker', async () => {
-      let { stdout, code } = await runCli(tapeFailureFixture, '--runner=tape --worker --page=false')
+      let { stdout, code } = await runCli('tape', tapeFailureFixture, ['--runner=tape', '--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')
@@ -155,7 +151,7 @@ not ok 2 bork
     })
 
     it('should fail in serviceworker', async () => {
-      let { stdout, code } = await runCli(tapeFailureFixture, '--runner=tape --serviceworker --page=false')
+      let { stdout, code } = await runCli('tape', tapeFailureFixture, ['--runner=tape', '--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')
@@ -167,7 +163,7 @@ not ok 2 bork
     })
 
     it('should fail in page and not run in worker', async () => {
-      let { stdout, code } = await runCli(tapeFailureFixture, '--runner=tape --worker')
+      let { stdout, code } = await runCli('tape', tapeFailureFixture, ['--runner=tape', '--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')

package/test/test-webpack-merge.js

@@ -1,12 +1,8 @@
 /* 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))
-const webpackMergeFixture = path.join(__dirname, 'fixtures/webpack-merge')
+const webpackMergeFixture = 'webpack-merge'
 
 describe('basic webpack-merge', function () {
   this.timeout(60000)
@@ -16,7 +12,7 @@ assert.ok() is a function
 WOOP is set
   ✔ test.js
 `
-    const { stdout, stderr, code } = await runCli(webpackMergeFixture, '--runner=bare-sync --webpack-config webpack.config.js')
+    const { stdout, stderr, code } = await runCli('bare', webpackMergeFixture, ['--runner=bare-sync', '--webpack-config', 'webpack.config.js'])
     if (code !== 0) {
       console.error(stderr)
     }
@@ -35,7 +31,7 @@ assert.ok() is a function
 `
     const expectedStderr = 'WOOP is not defined'
 
-    const { stdout, stderr, code } = await runCli(webpackMergeFixture, '--runner=bare-sync')
+    const { stdout, stderr, code } = await runCli('bare', webpackMergeFixture, ['--runner=bare-sync'])
     assert.strictEqual(code, 1, 'exited with zero exit code')
     assert.ok(stdout.includes(expectedStdout), 'stdout contains expected test output')
     assert.ok(stderr.includes(expectedStderr), 'stderr contains expected test output')