-
-
Notifications
You must be signed in to change notification settings - Fork 591
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tests: improve --node-args test cases #1546
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this error comes from Node.js, would it be a good idea to hardcode it like this?
Actually it's hardcoded for --unknown flag case as well, webpack-cli/test/node/node.test.js Line 65 in 6a8dd32
So it should be changed as well. Let's wait for other opinions as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
@evilebottnawi Thanks for your update. I labeled the Pull Request so reviewers will review it again. @anshumanv Please review the new changes. |
Good to merge? |
What kind of change does this PR introduce?
tests
Did you add tests for your changes?
Yes
If relevant, did you update the documentation?
NA
Summary
Improved test cases to check for the specific error messages.
Does this PR introduce a breaking change?
No
Other information