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

Pcov for faster code coverage #183

Closed
dkarlovi opened this issue Dec 6, 2019 · 4 comments · Fixed by #201
Closed

Pcov for faster code coverage #183

dkarlovi opened this issue Dec 6, 2019 · 4 comments · Fixed by #201

Comments

@dkarlovi
Copy link
Contributor

dkarlovi commented Dec 6, 2019

https://github.com/krakjoe/pcov

@Guite
Copy link

Guite commented Dec 11, 2019

👍
Possible setup in alpine: pecl install pcov && docker-php-ext-enable pcov

@jakzal
Copy link
Owner

jakzal commented Dec 11, 2019

How much faster? :D

@Guite
Copy link

Guite commented Dec 11, 2019

@maks-rafalko
Copy link

maks-rafalko commented Feb 10, 2020

I second this. Today we had one more weird bug with Infection + phpdbg. It just doesn't work for some unknown reason, while works with PHP itself (including xdebug & pcov).

Here is a quote from Joe Watkins (author of phpdbg) about another issue that I spend a week of debugging with:

infection/infection#622 (comment)

I did try to reproduce the issue, I wrote the core of phpdbg and am intimately familiar with it, I'm also aware that aside from mistakes in coverage (which I did not write) it has bugs that may also be effecting you and are stopping me from even doing a full run locally, there's some problem with too many open file handles, class not found and such, this all seems to be connected to your current headache ...

Joe Watkins (krakjoe)

Let's discourage people from using phpdbg. For coverage, we should (must) use pcov from the same author, which is extremely faster.

To give you some numbers, PHPUnit run takes 1.3 minute with pcov vs 4 minutes with Xdebug with xdebug-filter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants