fix: ci for webpack@beta.30

[snitin315]

What kind of change does this PR introduce?
tests

Did you add tests for your changes?
yes
If relevant, did you update the documentation?

Summary
fixed failing tests for webpack@beta.30

Does this PR introduce a breaking change?
no

Other information

[snitin315]

/cc @webpack/cli-team , help output is not working anymore. It looks like issue in chalk or command-line-usage.

[rishabh3112]

Working fine for me at next.

Working fine on your branch as well

Check your local setup once :).

[snitin315]

Failing in CI as well, for webpack@next.

[snitin315]

Check your local setup once :).

@rishabh3112 Can you check with webpack 5.0.0-beta.30?

[alexander-akait]

@snitin315 weird, we migrate from chalk 😕

[snitin315]

But command-line-usage is still using chalk as a dep. There are two possible solutions that I can think of right now -

• We can use the commander.js default help output for webpack@next. (Quick fix).
• Our custom utility for help output.

[rishabh3112]

Can you check with webpack 5.0.0-beta.30?

Yeah, I did. I reproduced the error too. It is after latest pre-release of webpack v5 as previous version of v5 is working fine. Please check for changes in it where flag description was either added or edited.

[snitin315]

Can you check with webpack 5.0.0-beta.30?

Yeah, I did. I reproduced the error too. It is after latest pre-release of webpack v5 as previous version of v5 is working fine. Please check for changes in it where flag description was either added or edited.

Ah yes! Looking into it.

[snitin315]

It looks like I found the culprits.

https://github.com/webpack/webpack/blob/8237bb442917d37380e7a53b367f0a07011f6a28/test/__snapshots__/Cli.test.js.snap#L2217
https://github.com/webpack/webpack/blob/8237bb442917d37380e7a53b367f0a07011f6a28/test/__snapshots__/Cli.test.js.snap#L2223
https://github.com/webpack/webpack/blob/8237bb442917d37380e7a53b367f0a07011f6a28/test/__snapshots__/Cli.test.js.snap#L2282
https://github.com/webpack/webpack/blob/8237bb442917d37380e7a53b367f0a07011f6a28/test/__snapshots__/Cli.test.js.snap#L2288

[snitin315]

Yes, help output is working after I remove these flags. Should I create a PR to update these descriptions at the core?

[rishabh3112]

Yes @snitin315, go ahead!

[alexander-akait]

@snitin315 I think we need escape some characters

[snitin315]

Yes, I did some more research and found a similar issue Polymer/tools#595 and PR Polymer/tools#612

[snitin315]

Not sure why serve tests are failing on CI, they are passing locally

[snitin315]

/cc @webpack/cli-team ready for review 👍🏻