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
Refactor test #2632
Refactor test #2632
Conversation
b02a8e9
to
2ce5f17
Compare
Codecov Report
@@ Coverage Diff @@
## master #2632 +/- ##
==========================================
- Coverage 95.21% 95.14% -0.07%
==========================================
Files 30 30
Lines 1505 1505
Branches 430 430
==========================================
- Hits 1433 1432 -1
- Misses 72 73 +1
Continue to review full report at Codecov.
|
825f2d9
to
f37cca2
Compare
We need search way to say |
can you clarify? |
https://github.com/tj/commander.js/blob/master/index.js#L577 in our tests on some os we have |
1099f9c
to
464fcc4
Compare
@@ -1023,7 +1023,7 @@ class WebpackCLI { | |||
}, | |||
formatHelp: (command, helper) => { | |||
const termWidth = helper.padWidth(command, helper); | |||
const helpWidth = helper.helpWidth || 80; | |||
const helpWidth = helper.helpWidth || process.env.WEBPACK_CLI_HELP_WIDTH || 80; |
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.
For testing purpose and for edge case where developer can set width of TTY
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.
Finally, I'll do a couple PRs for tests, improve them |
What kind of change does this PR introduce?
refactor/tests
Did you add tests for your changes?
it is tests
If relevant, did you update the documentation?
No need
Summary
Part of #2573 to avoid big changes
Does this PR introduce a breaking change?
No
Other information
In future we should migrate on stdout and stderr snapshot tests everywhere, so feel free to send PRs with improve it, better do it by command, for
build
command by group of options,serve
no need touch right now WIP on this