Skip to content

test: run webpack config test cases with rspack test tools#11645

Merged
LingyuCoder merged 1 commit intomainfrom
test/merge-webpack-config-tests
Sep 15, 2025
Merged

test: run webpack config test cases with rspack test tools#11645
LingyuCoder merged 1 commit intomainfrom
test/merge-webpack-config-tests

Conversation

@LingyuCoder
Copy link
Contributor

Summary

run webpack config test cases with rspack test tools

Related links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@LingyuCoder LingyuCoder requested review from Copilot and removed request for Copilot September 11, 2025 07:43
@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Sep 11, 2025
@netlify
Copy link

netlify bot commented Sep 11, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 44610b6
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/68c58ba53135a1000717ad1a

@github-actions
Copy link
Contributor

github-actions bot commented Sep 11, 2025

📦 Binary Size-limit

Comparing 44610b6 to chore(deps): update pnpm to v10.15.1 (#11605) by renovate[bot]

🙈 Size remains the same at 47.45MB

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 11, 2025

CodSpeed Performance Report

Merging #11645 will not alter performance

Comparing test/merge-webpack-config-tests (44610b6) with main (5549377)

Summary

✅ 17 untouched

Copilot AI review requested due to automatic review settings September 11, 2025 09:31
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates webpack configuration test cases to run with rspack test tools, ensuring compatibility between webpack and rspack testing infrastructure.

  • Update module scope and execution hook signatures to include compiler options
  • Add support for additional globals and statistics indices in test runners
  • Extend helper functions and utilities to support broader test scenarios

Reviewed Changes

Copilot reviewed 31 out of 3906 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tests/rspack-test/configCases/async-module/environment-not-support-async-warning/test.filter.js Update import path for supportsWebAssembly helper
tests/rspack-test/configCases/asset-modules/only-entry/test.js Add console logging for debugging
tests/rspack-test/configCases/asset-modules/entry-with-runtimeChunk/test.js Add console logging for debugging
tests/rspack-test/configCases/asset-emitted/normal/webpack.config.js Update inline snapshot format
tests/rspack-test/snapshots/Config.test.js.snap Add new test snapshots
packages/rspack-test-tools/src/type.ts Update type signatures for module scope and execution hooks
packages/rspack-test-tools/src/runner/* Update runners with expanded module scope signatures
packages/rspack-test-tools/src/processor/* Update processors with new execution hook signatures
packages/rspack-test-tools/src/helper/legacy/* Add various helper utilities
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@LingyuCoder LingyuCoder force-pushed the test/merge-webpack-config-tests branch from b55d743 to 96fad84 Compare September 11, 2025 13:38
@LingyuCoder LingyuCoder force-pushed the test/merge-webpack-config-tests branch from 34ac4f0 to 44610b6 Compare September 13, 2025 15:19
@LingyuCoder LingyuCoder enabled auto-merge (squash) September 15, 2025 03:14
@LingyuCoder LingyuCoder merged commit e759459 into main Sep 15, 2025
68 of 70 checks passed
@LingyuCoder LingyuCoder deleted the test/merge-webpack-config-tests branch September 15, 2025 03:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants