Skip to content

test: move rspack test cases to tests/rspack-test#11617

Merged
LingyuCoder merged 4 commits intomainfrom
test/move-tests
Sep 9, 2025
Merged

test: move rspack test cases to tests/rspack-test#11617
LingyuCoder merged 4 commits intomainfrom
test/move-tests

Conversation

@LingyuCoder
Copy link
Contributor

@LingyuCoder LingyuCoder commented Sep 9, 2025

Summary

Just move test cases from packages/rspack-test-tools/tests to tests/rspack-test, and will merge webpack-test into it in near future.

Related links

Checklist

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

@netlify
Copy link

netlify bot commented Sep 9, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 42d5680
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/68bff968970f0800087bd0e0

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2025

📦 Binary Size-limit

Comparing 42d5680 to refactor: remove wasm plugin useless code (#11615) by jinrui

🙈 Size remains the same at 47.23MB

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 9, 2025

CodSpeed Performance Report

Merging #11617 will not alter performance

Comparing test/move-tests (42d5680) with main (233d358)

🎉 Hooray! codspeed-rust just leveled up to 2.7.2!

A heads-up, this is a breaking change and it might affect your current performance baseline a bit. But here's the exciting part - it's packed with new, cool features and promises improved result stability 🥳!
Curious about what's new? Visit our releases page to delve into all the awesome details about this new version.

Summary

✅ 17 untouched benchmarks

@LingyuCoder LingyuCoder marked this pull request as ready for review September 9, 2025 10:29
Copilot AI review requested due to automatic review settings September 9, 2025 10:29
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

Move rspack test cases from packages/rspack-test-tools/tests to tests/rspack-test to reorganize the test structure. This consolidates test cases in a dedicated tests directory and updates all related imports and path references.

  • Move all test files from packages/rspack-test-tools/tests to tests/rspack-test
  • Update import statements to use @rspack/test-tools instead of relative imports
  • Modify snapshot files to reflect new path structure

Reviewed Changes

Copilot reviewed 106 out of 7696 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/rspack-test/*.test.js Updated imports to use @rspack/test-tools package
tests/rspack-test/snapshots/*.snap Updated path references in snapshot files
packages/rspack-test-tools/src/**/*.ts Updated internal paths to point to new test location
pnpm-workspace.yaml Added tests/rspack-test to workspace packages
scripts/test/add-config-type.cjs Updated paths for config type injection
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 enabled auto-merge (squash) September 9, 2025 10:33
@LingyuCoder LingyuCoder merged commit 7c2d40c into main Sep 9, 2025
93 of 98 checks passed
@LingyuCoder LingyuCoder deleted the test/move-tests branch September 9, 2025 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants