polendina 3.3.3 → 3.3.4

package/AGENTS.md

@@ -65,6 +65,9 @@ Real usage, in increasing order of involvement. Each shape must keep working:
 - `polendina --runner=bare-async test/test.js` paired with
   `polendina-node bare-async test/test.js`. The same files run in Node.js and
   the browser with no test framework at all.
+- `import hex from '!!raw-loader!./fixtures/block.hex'` inside a test to read a
+  fixture as a string. Binary and hex fixture suites do this, declaring no
+  loader of their own.
 - `polendina --cleanup --webpack-config test/webpack.config.js test/test-*.js`
   where the config is `export default { externals: { standard: 'null' } }`.
   `js-ipld-schema` does this to stop webpack bundling a Node-only
@@ -149,8 +152,13 @@ Things that look wrong and are not. Each exists because something broke.
   `polendina/node_modules` directory.
 - **`import 'mocha/mocha.js'` for side effect, then `globalThis.mocha`.**
   Mocha 11 ships UMD, 12 ships ESM. The global is the common denominator.
-  Both are supported at once. The `color()` / `useColors()` branch is the
-  same idea for older mochas.
+  Both are supported at once.
+- **`raw-loader` is a dependency that no polendina source references.**
+  Consumers load fixture files as text with
+  `import hex from '!!raw-loader!./fixtures/block.hex'` and declare nothing; the
+  loader resolves from polendina's tree through `resolveLoader.modules`. The
+  mocha fixtures assert it. Removing it or the second `resolveLoader` entry
+  breaks those consumers.
 - **`exprContextCritical: false`.** Mocha's bundle has a dynamic
   `require.resolve`; without this every run prints a webpack warning.
 - **`resolve.tsconfig: false`.** Webpack 5.109+ on Node 22.6+ follows a
@@ -160,9 +168,8 @@ Things that look wrong and are not. Each exists because something broke.
   forward-slashed.** Tests in subdirectories and Windows paths both broke the
   cwd-relative version. Three separate fixes landed here. Paths in
   `wrap-loader.cjs` are the most common regression site in the repo.
-- **`puppeteer.launch({ args: ['--no-sandbox'], headless: 'new' })`.**
-  `--no-sandbox` for root in Docker and CI. `'new'` was a Puppeteer 20
-  deprecation fix and is itself a candidate for removal on current Puppeteer.
+- **`puppeteer.launch({ args: ['--no-sandbox'] })`.** `--no-sandbox` for
+  root in Docker and CI. Headless is Puppeteer's default and is not named.
 - **`st({ cache: false })`.** A stale cached bundle between page, worker and
   serviceworker runs on Windows.
 - **`setTimeout(...).unref()` for `--timeout`.** A clean finish must not be
@@ -316,11 +323,16 @@ Good work, roughly in order of value:
 
 - **Keep the matrix green.** Pick up a red dependabot PR, find the fallout,
   land the fix with the bump. This is the single most useful thing.
-- **Remove dead weight with evidence.** `raw-loader` is a dependency no source
-  references. The `--coverage` flag and the `pti` TODO have never shipped. The
-  commented-out `page.on('console')` block is dead. The `useColors` branch is
-  unreachable on supported mochas. Each removal is one PR with a full test run
-  and a `git log -S` showing where it came from.
+- **Remove dead weight with evidence.** A flag that never shipped, a branch
+  unreachable on supported versions. Each removal is one PR with a full test
+  run and a `git log -S` showing where it came from. A dependency with no
+  reference in this repo is not evidence on its own: `raw-loader` is used
+  only from consumers' inline loader strings. Search the consumers first.
+- **Make dynamic `import()` work in Worker and ServiceWorker modes.** Webpack
+  splits each one into a chunk and loads it through a script tag, which only
+  a page has. `module.parser.javascript.dynamicImportMode: 'eager'` would
+  inline them into the single bundle; the mocha fixtures' raw-loader case
+  would then pass as a dynamic import in all three scopes.
 - **Make failures legible.** When a resolve fails inside a consumer's
   dependency, the webpack error is the user's only clue. Better messages
   around bundle errors and the timeout are cheap wins.

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [3.3.4](https://github.com/rvagg/polendina/compare/v3.3.3...v3.3.4) (2026-10-24)
+
+### Trivial Changes
+
+* remove dead code, document and test raw-loader ([#206](https://github.com/rvagg/polendina/issues/206)) ([3ce94c0](https://github.com/rvagg/polendina/commit/3ce94c02208d72e8c174d6607bc30266e2ad69aa))
+
 ## [3.3.3](https://github.com/rvagg/polendina/compare/v3.3.2...v3.3.3) (2026-10-24)
 
 ### Tests

package/README.md

@@ -20,6 +20,7 @@
   * [bare-sync](#bare-sync)
   * [bare-async](#bare-async)
 * [TypeScript support (and other Webpack extensions)](#typescript-support-and-other-webpack-extensions)
+* [Loading fixture files](#loading-fixture-files)
 * [polendina-node: the minimal Node.js test runner](#polendina-node-the-minimal-nodejs-test-runner)
 * [Minimising Puppeteer's size](#minimising-puppeteers-size)
   * [Global and npx](#global-and-npx)
@@ -187,6 +188,16 @@ module.exports = {
 $ polendina --cleanup test/*.spec.ts --webpack-config webpack.config.ts.js
 ```
 
+## Loading fixture files
+
+polendina bundles [raw-loader](https://ghub.io/raw-loader) so a test can read a fixture file as a string without any configuration, using webpack's inline loader syntax:
+
+```js
+import hex from '!!raw-loader!./fixtures/block.hex'
+```
+
+The leading `!!` stops any other loader from touching the file. In a CommonJS test the string is on `require('!!raw-loader!./fixtures/block.hex').default`. A dynamic `import()` works too, but only in page mode: webpack loads it as a separate chunk through a script tag, which Workers and ServiceWorkers do not have.
+
 
 ## `polendina-node`: the minimal Node.js test runner
 

package/lib/puppeteer.js

@@ -1,15 +1,12 @@
 import puppeteer from 'puppeteer'
-// TODO
-// const pti = require('puppeteer-to-istanbul')
-// const { copyFile } = require('fs').promises
 import { log, error, logRaw, logWrite, flushLogs } from './log.js'
 
 // wrap our convoluted _run() function in a pure Promise that can handle
 // both standard throws and the callback that ends it. _run() needs to handle
 // ends in a few different ways, hence the hack.
-export function run (outputDir, port, timeout, mode, runner, coverage) {
+export function run (outputDir, port, timeout, mode, runner) {
   return new Promise((resolve, reject) => {
-    _run(outputDir, port, timeout, mode, runner, coverage, (err, errors) => {
+    _run(outputDir, port, timeout, mode, runner, (err, errors) => {
       if (err) {
         return reject(err)
       }
@@ -21,10 +18,10 @@ export function run (outputDir, port, timeout, mode, runner, coverage) {
 // this can throw, or it can end via `callback()` which may or may not contain
 // a runtime-error argument and a "number of errors from tests" argument.
 // the callback may be triggered by proper test end or failure, or a timeout.
-async function _run (outputDir, port, timeout, mode, runner, coverage, callback) {
+async function _run (outputDir, port, timeout, mode, runner, callback) {
   let executionQueue = Promise.resolve()
 
-  const browser = await puppeteer.launch({ args: ['--no-sandbox'], headless: 'new' })
+  const browser = await puppeteer.launch({ args: ['--no-sandbox'] })
   const [page] = await browser.pages()
 
   // this works as a debounce exit method, when process.stdout.write() is used
@@ -44,13 +41,6 @@ async function _run (outputDir, port, timeout, mode, runner, coverage, callback)
       executionQueue.then(async () => {
         flushLogs(true)
         executionQueue = null
-        /* TODO
-        if (coverage) {
-          const jsCoverage = await page.coverage.stopJSCoverage()
-          pti.write([...jsCoverage])
-          await copyFile('build/bundle.js.map', '.nyc_output/js/bundle.js.map')
-        }
-        */
         await browser.close()
       }).catch(callback)
         .then(() => { callback(null, errors) })
@@ -64,24 +54,6 @@ async function _run (outputDir, port, timeout, mode, runner, coverage, callback)
     }
   }
 
-  // this should be a rare, or impossible event since we intercept console.log
-  /*
-  page.on('console', (msg) => {
-    if (!executionQueue) {
-      error(`log after end: ${msg.text()}`)
-      return
-    }
-    const args = []
-    executionQueue = executionQueue.then(async () => {
-      logRaw('info', -1, await Promise.all(args))
-    })
-    for (const arg of msg.args()) {
-      args.push(arg.evaluate(n => n))
-    }
-    maybeEnd()
-  })
-  */
-
   page.on('error', (error) => console.error(error))
   page.on('pageerror', (error) => console.error(error))
 
@@ -97,10 +69,6 @@ async function _run (outputDir, port, timeout, mode, runner, coverage, callback)
     maybeEnd()
   })
 
-  if (coverage) {
-    await page.coverage.startJSCoverage()
-  }
-
   const url = `http://localhost:${port}/?mode=${mode}&runner=${runner}`
   log(`Running ${runner} ${mode} tests with Puppeteer via\n                ${url}`)
   if (runner !== 'mocha') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polendina",
-  "version": "3.3.3",
+  "version": "3.3.4",
   "description": "Non-UI browser testing for JavaScript libraries from the command-line",
   "main": "polendina.js",
   "type": "module",

package/polendina-cli.js

@@ -52,12 +52,6 @@ const argv = yargs(hideBin(process.argv))
     describe: 'Number of seconds to wait before auto-failing the test suite',
     default: 30
   })
-  .option('coverage', {
-    type: 'boolean',
-    describe: 'Enable coverage reporting',
-    default: false,
-    hidden: true
-  })
   .option('webpack-config', {
     type: 'string',
     describe: 'Supply a path to a webpack.config.js to merge into Polendina\'s Webpack config (use with caution)',

package/polendina.js

@@ -132,7 +132,7 @@ export class Polendina {
       server.on('error', reject)
       server.listen(() => {
         const port = server.address().port
-        puppeteer(this.outputDir, port, this._options.timeout, mode, this._options.runner, this._options.coverage)
+        puppeteer(this.outputDir, port, this._options.timeout, mode, this._options.runner)
           .then((errors) => {
             server.close(() => {
               if (this._options.runner !== 'mocha') {

package/resources/common-run.js

@@ -26,12 +26,10 @@ export const log = {
     executionQueue(() => globalThis.polendinaLog(['info', logSeq++].concat(args)))
     return executionQueue
   },
-  // TODO
   warn: (...args) => {
     executionQueue(() => globalThis.polendinaLog(['warn', logSeq++].concat(args)))
     return executionQueue
   },
-  // TODO
   error: (...args) => {
     executionQueue(() => globalThis.polendinaLog(['error', logSeq++].concat(args)))
     return executionQueue
@@ -87,7 +85,6 @@ function setupLogging () {
     log.info.apply(null, args)
   }
 
-  // TODO: differentiate
   console.warn = log.warn
   console.error = log.error
 }

package/resources/mocha-run.js

@@ -8,12 +8,7 @@ import { registry, executionQueue, log, setup } from './common-run.js'
 async function runMocha () {
   const mochaLocal = globalThis.mocha
   mochaLocal.setup({ reporter: registry.argv.mochaReporter, ui: 'bdd' })
-  // mocha@7 deprecated useColors()
-  if (typeof mochaLocal.color === 'function') {
-    mochaLocal.color(true)
-  } else if (typeof mochaLocal.useColors === 'function') {
-    mochaLocal.useColors(true)
-  }
+  mochaLocal.color(true)
 
   // the well-behaved reporters, like spec, are easy to intercept
   mochaLocal.constructor.reporters.Base.consoleLog = log.info

package/test/fixtures/mocha/fixture.txt

@@ -0,0 +1 @@
+raw fixture

package/test/fixtures/mocha/test-1.js

@@ -11,4 +11,8 @@ describe('test suite 1', () => {
   it('test case 2', () => {
     assert.ok(true, 'all good')
   })
+
+  it('test case 3', () => {
+    assert.strictEqual(require('!!raw-loader!./fixture.txt').default, 'raw fixture') // eslint-disable-line import/no-webpack-loader-syntax
+  })
 })

package/test/fixtures/mocha-esm/fixture.txt

@@ -0,0 +1 @@
+raw fixture

package/test/fixtures/mocha-esm/test-1.js

@@ -1,6 +1,7 @@
 /* globals describe it */
 
 import fixture from './index.js'
+import raw from '!!raw-loader!./fixture.txt' // eslint-disable-line import/no-webpack-loader-syntax
 import { assert } from 'chai'
 
 describe('test suite 1', () => {
@@ -11,4 +12,8 @@ describe('test suite 1', () => {
   it('test case 2', () => {
     assert.ok(true, 'all good')
   })
+
+  it('test case 3', () => {
+    assert.strictEqual(raw, 'raw fixture')
+  })
 })

package/test/test-bare-async.js

@@ -25,7 +25,7 @@ testing bare fixture subdir
     it('should run in page', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')
@@ -39,7 +39,7 @@ testing bare fixture subdir
     it('should run in worker', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async', '--worker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in worker')
@@ -53,7 +53,7 @@ testing bare fixture subdir
     it('should run in serviceworker', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async', '--serviceworker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in serviceworker')
@@ -67,7 +67,7 @@ testing bare fixture subdir
     it('should run in page, worker and serviceworker', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareAsyncFixture, ['--runner=bare-async', '--worker', '--serviceworker'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
 

package/test/test-bare-sync.js

@@ -21,7 +21,7 @@ testing bare fixture subdir
     it('should run in page', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')
@@ -35,7 +35,7 @@ testing bare fixture subdir
     it('should run in worker', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync', '--worker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in worker')
@@ -49,7 +49,7 @@ testing bare fixture subdir
     it('should run in serviceworker', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync', '--serviceworker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in serviceworker')
@@ -63,7 +63,7 @@ testing bare fixture subdir
     it('should run in page, worker and serviceworker', async () => {
       const { stdout, stderr, code } = await runCli('bare', bareSyncFixture, ['--runner=bare-sync', '--worker', '--serviceworker'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
 

package/test/test-mocha.js

@@ -21,6 +21,7 @@ for (const [consumerName, type] of Object.keys(ticks).flatMap((c) => ['cjs', 'es
   test suite 1
     ${ticks[consumerName]} test case 1
     ${ticks[consumerName]} test case 2
+    ${ticks[consumerName]} test case 3
 
   test suite 2 - worker
     ${ticks[consumerName]} is WORKER
@@ -33,7 +34,7 @@ for (const [consumerName, type] of Object.keys(ticks).flatMap((c) => ['cjs', 'es
     it('should run in page', async () => {
       const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, [])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')
@@ -47,7 +48,7 @@ for (const [consumerName, type] of Object.keys(ticks).flatMap((c) => ['cjs', 'es
     it('should run in worker', async () => {
       const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, ['--worker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in worker')
@@ -61,7 +62,7 @@ for (const [consumerName, type] of Object.keys(ticks).flatMap((c) => ['cjs', 'es
     it('should run in serviceworker', async () => {
       const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, ['--serviceworker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in serviceworker')
@@ -75,7 +76,7 @@ for (const [consumerName, type] of Object.keys(ticks).flatMap((c) => ['cjs', 'es
     it('should run in page, worker and serviceworker', async () => {
       const { stdout, stderr, code } = await runCli(consumerName, mochaFixture, ['--worker', '--serviceworker'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
 

package/test/test-polendina-node.js

@@ -25,7 +25,7 @@ Took X seconds
 `
       const { stdout, stderr, code } = await runNodeCli('bare', bareSyncFixture, 'bare-sync')
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       assert.strictEqual(removeTiming(stdout), expected)
@@ -60,7 +60,7 @@ Took X seconds
 `
       const { stdout, stderr, code } = await runNodeCli('bare', bareAsyncFixture, 'bare-async')
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       assert.strictEqual(removeTiming(stdout), expected)

package/test/test-tape.js

@@ -35,7 +35,7 @@ ok 5 all good
     it('should run in page', async () => {
       const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'not in worker')
@@ -49,7 +49,7 @@ ok 5 all good
     it('should run in worker', async () => {
       const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape', '--worker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in worker')
@@ -63,7 +63,7 @@ ok 5 all good
     it('should run in serviceworker', async () => {
       const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape', '--serviceworker', '--page=false'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
       const expected = expectedTemplate.replace(/WORKER/, 'in serviceworker')
@@ -77,7 +77,7 @@ ok 5 all good
     it('should run in page, worker and serviceworker', async () => {
       const { stdout, stderr, code } = await runCli('tape', tapeFixture, ['--runner=tape', '--worker', '--serviceworker'])
       if (code !== 0) {
-        console.error(stderr)
+        console.error(stdout, stderr)
       }
       assert.strictEqual(code, 0, 'exited with zero exit code')
 

package/test/test-webpack-merge.js

@@ -14,7 +14,7 @@ WOOP is set
 `
     const { stdout, stderr, code } = await runCli('bare', webpackMergeFixture, ['--runner=bare-sync', '--webpack-config', 'webpack.config.js'])
     if (code !== 0) {
-      console.error(stderr)
+      console.error(stdout, stderr)
     }
     assert.strictEqual(code, 0, 'exited with zero exit code')
     if (!stdout.includes(expected)) {