Skip to content
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

[Bug/Feature Request]: Coverage class printing #1337

Open
schonhoff opened this issue Jan 8, 2025 · 0 comments
Open

[Bug/Feature Request]: Coverage class printing #1337

schonhoff opened this issue Jan 8, 2025 · 0 comments
Labels

Comments

@schonhoff
Copy link

What Happened

Hello,

is it possible remove the output for every class on the coverage option if you add a --no-class-output option to the coverage command or maybe the other coverage options (for example --coverage-html) removes the terminal printing on its own because the output will be printed in another file (html in the example case).

The printing for a huge test coverage (20000 tests with almost 2000 classes) is using a lot of time and sometimes the terminal crashes because of the buffered output.

I would provide a PR but I couldn't figure out how PEST is using the options :-(
I would add here:

if ($useOtherOutput) {
    return $exitCode;
}

I don't know if the coverage result printing (without single classes) uses a lot of time. If not maybe this can only be printed instead of every single class and its coverage.

A similar request was discussed for type-coverage here:
#1183

How to Reproduce

It is quite hard to reproduce because you need a huge test suite.

Sample Repository

No response

Pest Version

3.7.1

PHP Version

8.3.15

Operation System

Windows

Notes

I'm using Powershell 7 and the new PHPStorm terminal option.

@schonhoff schonhoff added the bug label Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant