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

Speedup #433

Merged
merged 3 commits into from
May 4, 2016
Merged

Speedup #433

merged 3 commits into from
May 4, 2016

Conversation

MajorCaiger
Copy link
Contributor

Re-submitted after resolving conflicts as requested.

Rob Caiger added 2 commits May 3, 2016 19:33
- Init data by running xdebug with 'unused' and 'dead code' flags on all whitelist files
- Run xdebug without flags for each test method
- Cache num lines check for each file, to prevent superflous checks
- (SB) Fix CS/WS issues
@@ -39,7 +39,7 @@
/**
* Start collection of code coverage information.
*/
public function start();
public function start($determineUnusedAndDead = true);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you forgot to update the phpdoc

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point, I've added these now

@GrahamCampbell
Copy link
Contributor

This is a major breaking change due to the modification of an interface. Existing implementations will cause php to fatal error.

@GrahamCampbell
Copy link
Contributor

Oh, I see the master is 4.x. That's fine then. ;)

@sebastianbergmann sebastianbergmann merged commit 44b5f9d into sebastianbergmann:master May 4, 2016
@sebastianbergmann
Copy link
Owner

Thanks, @MajorCaiger. Did you do a benchmark and can share some numbers?

@MajorCaiger MajorCaiger mentioned this pull request Jul 27, 2016
@MajorCaiger
Copy link
Contributor Author

The original benchmarks can be found here #384 I think the implementation changed slightly but the improvements should be similar.

@liorharbater
Copy link

Hi @sebastianbergmann
Using this Speedup enhancement would greatly assist us in reducing our coverage tests cycle time!
Do you plan to release it as part of an official phpunit 4.X release any time soon?
We are using php 5.3 and upgrading to any, more advanced, php version will be too complex at this stage, so we are limited to the 4.X phpunit.

Thanks,
Lior

@sebastianbergmann
Copy link
Owner

There will be no major changes to anything related to PHPUnit 4.8, sorry.

@liorharbater
Copy link

Unfortunate, but understandable :)
Any plans to release this Speedup enhancement as part of PHPUnit 5.X, any time soon ?

@GrahamCampbell
Copy link
Contributor

This is already in PHPUnit 5.x is it not?

@willemstuursma
Copy link

We've seen a speed increase from 1h48m to 13m. I just want to say I really appreciate this change.

@MajorCaiger
Copy link
Contributor Author

@willemstuursma That's a fantastic improvement!

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 this pull request may close these issues.

5 participants