refactor(rollup-plugin-import-meta-assets): migrate tests to node:test#3081
refactor(rollup-plugin-import-meta-assets): migrate tests to node:test#3081bennypowers wants to merge 1 commit into
Conversation
Replace mocha/chai/hanbi with node:test, node:assert/strict, and node:test mock API. Update test:node script to use `node --test`. - `expect(x).to.equal(y)` -> `assert.equal(x, y)` - `expect(x).to.deep.equal(y)` -> `assert.deepEqual(x, y)` - `expect(x).to.have.lengthOf(n)` -> `assert.equal(x.length, n)` - `expect(x).to.match(re)` -> `assert.match(x, re)` - `hanbi.stubMethod(console, 'warn')` -> `mock.method(console, 'warn')` - `hanbi.restore()` -> `mock.restoreAll()` - `consoleStub.callCount` -> `consoleStub.mock.callCount()` - `consoleStub.getCall(n).args` -> `consoleStub.mock.calls[n].arguments` - Removed `test:update-snapshots` script (no longer applicable) - Add `--test-force-exit` to prevent test hangs Assisted-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
| const path = require('path'); | ||
| const { rollup } = require('rollup'); | ||
| const { expect } = require('chai'); | ||
| const hanbi = require('hanbi'); |
There was a problem hiding this comment.
just double checking: you are going to remove hanbi and other dependencies which will not be used after all refactorings, in a separate PR once all are complete?
| expect(chunks['app.js']).to.equal(js` | ||
| assert.equal( | ||
| chunks['app.js'], | ||
| js` |
There was a problem hiding this comment.
the format is a bit off in here and I think in all other similar cases
looks like prettier doesn't like this as a one-liner
assert.equal(chunks['app.js'], js`
but then it should change the indentation of code lines which doesn't happen either
can you please check this? if you do this with copilot, maybe you can ask it to fix such indentation issues
looks like in some other PRs the indentation was refactored correctly in similar cases, e.g. in #3097
2 participants
Summary
@web/rollup-plugin-import-meta-assetstests from mocha/chai/hanbi tonode:test+node:assert/strict+node:testmock APItest:nodescript from mocha tonode --test --test-force-exitmock.method/mock.restoreAlltest:update-snapshotsscript (no longer applicable with node:test)Test plan