test: easier way to write hmr test cases #8849
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
before:
after:
This pull request includes several important changes to the
rspack-test-tools
package, focusing on improving hot module replacement (HMR) support, simplifying the codebase, and updating test snapshots.Improvements to HMR support:
packages/rspack-test-tools/src/helper/legacy/fake-update-loader.js
to handle module reexports and runtime updates.TestHotUpdatePlugin
to enhance HMR functionality inHotProcessor
and updated the loader path for hot updates. [1] [2] [3]Codebase simplification:
checkArrayExpectation
calls to use async/await, removing unnecessary promise wrappers in multiple files (basic.ts
,watch.ts
,cache.ts
). [1] [2] [3] [4]HotStepRunnerFactory
andHotRunnerFactory
to use async/await for thenext
function and introducednextHMR
for handling HMR updates. [1] [2] [3] [4] [5]Test snapshot updates:
ensure-chunk-change-to-promise-all
,recovery-cacheable
,recovery
,hash/hot-index
,recover-after-self-error
,dispose-removed-module
,replace-runtime-module
,require-disposed-module-warning
,self-accept-factory
). [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]Checklist