fix progress on line wrap (trim line to stderr.columns)

[kuceb]

fix #9221

fixed it by trimming lines to process.stderr.columns if it exists

What kind of change does this PR introduce?
bugfix

Did you add tests for your changes?
yes

also added a test utility file so I didn't have to clutter up the progressPlugin.test.js, but it can be moved back in. I just needed a utility for capturing stderr and promisified compiler.run

[webpack-bot]

For maintainers only:

• This need to be documented (issue in webpack/webpack.js.org will be filed when merged)

[webpack-bot]

Thank you for your pull request! The most important CI builds succeeded, we’ll review the pull request soon.

[webpack-bot]

@bkucera Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@sokra Please review the new changes.

[kuceb]

friendly bump

[sokra]

Thanks

[kinke]

With --progress, I now have an output of thousands of lines (instead of just the progress line being constantly updated).

[kuceb]

@kinke please open an issue if you think this caused a bug, closed PRs are not good for visibility. Also please mention the webpack version you're on.

[kinke]

Well it's not a bug IMO, just a major inconvenience - I downgraded from v4.35.2 to v4.34 because of this.

[kuceb]

@kinke ok, well please open an issue or comment on this issue #9221, preferably with a reproducible example.

also it would be nice to log process.stderr.columns and post that with the issue. it may be a problem when that's not set

[sodatea]

IMO falling back to Infinity is not a good idea here.
It breaks use cases like the Vue CLI UI, where the stderr stream is piped thus not having the columns property vuejs/vue-cli#4557.
Such problems won't occur with the old 40 max length.