Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: avoid config check in webpack-test && add external test #3312

Merged
merged 3 commits into from
May 26, 2023

Conversation

hardfist
Copy link
Contributor

@hardfist hardfist commented May 26, 2023

Related issue (if exists)

Summary

馃 Generated by Copilot at 4552108

This pull request enables and updates some test cases for the externals feature of rspack, and adds a setup file to configure the rspack validation mode for the tests. It modifies test.filter.js files in externals/global and externals/this config cases, and adds setupFiles to package.json and setupEnv.js to webpack-test.

Walkthrough

馃 Generated by Copilot at 4552108

  • Enable testing global and this externals features of rspack by changing test.filter.js files to return true (link, link)
  • Add a setupFiles property to package.json to specify a setupEnv.js file for jest tests (link)
  • Set RSPACK_CONFIG_VALIDATE environment variable to loose-silent in setupEnv.js to avoid validation errors or warnings in test cases (link)

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label May 26, 2023
IWANABETHATGUY
IWANABETHATGUY previously approved these changes May 26, 2023
@hardfist hardfist force-pushed the webpack-test-external1 branch 2 times, most recently from 6b1f207 to 9241cf1 Compare May 26, 2023 07:37
Boshen
Boshen previously approved these changes May 26, 2023
IWANABETHATGUY
IWANABETHATGUY previously approved these changes May 26, 2023
@Boshen Boshen added this pull request to the merge queue May 26, 2023
Merged via the queue into main with commit 2526cd7 May 26, 2023
14 checks passed
@Boshen Boshen deleted the webpack-test-external1 branch May 26, 2023 09:12
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.

None yet

3 participants