Use Infinity width for non-TTY streams

[PayBas]

Purpose

Fixes #5919

Approach

trivial

References

#5919

Pre-Merge TODO

• Write tests for your proposed changes
• Make sure that existing tests do not fail

I didn't find any tests that have coverage on get-viewport-width.ts, so I assumed that tests weren't necessary for this. If it makes you feel any better, it's trivially easy to test manually.

[VasilyStrelyaev]

Thank you for your contribution! We will update this pull request once we complete our review.

[AndreyBelym]

Thank you for your contribution. Don't you think that it would be better to just set the viewport width to Infinity if the output stream is not a TTY? This way you won't have to configure your environment variables to get decent reports in CI - since every CI system I know can scroll report output.

While we don't have unit tests for this module, I assume that it was not a decision. I would like to add some test at least for the new functionality.

[PayBas]

@AndreyBelym that would be fine as well. Just let me know what you guys decide and I'll update it.

As for tests: I'm somewhat confused by the test setup here. I'm not a frontend developer, and it's been quite a while since I used any of this stuff. I'd have to familiarize myself with the test setup, and that might take quite a while.

[Aleksey28]

Thank you for your contribution. We are ready to approve of your pull request. Please set Infinity if the output stream is not a TTY. We'll add tests ourselves.

[github-actions]

Release v1.20.0-alpha.1 addresses this.