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

[runner] Support phpunit coverage generation using pcov #61

Open
wants to merge 1 commit into
base: old
Choose a base branch
from

Conversation

andrewnicols
Copy link
Collaborator

This commit adds support for coverage generation in phpunit tests using
the pcov php module, which is now installed on all of our current
images.

Generation is generated and saved to clover4j and crap4j file formats,
which have minimal file size implications.

It was found during testing that the phpunit formats were simply too
large, as were all other alternatives, with the cobertura output
reaching 7-8Gb.

This change is controlled by the COVERAGE environment option. When set
to the value "pcov", coverage will be generated using pcov.

This commit adds support for coverage generation in phpunit tests using
the `pcov` php module, which is now installed on all of our current
images.

Generation is generated and saved to clover4j and crap4j file formats,
which have minimal file size implications.

It was found during testing that the phpunit formats were simply too
large, as were all other alternatives, with the cobertura output
reaching 7-8Gb.

This change is controlled by the COVERAGE environment option. When set
to the value "pcov", coverage will be generated using pcov.
@stronk7
Copy link
Member

stronk7 commented Oct 15, 2021

This looks ok, only detail I don't understand is which is the purpose of the COVERAGEDIRHASH directory. It's not used by the runner, maybe it's to store reports, results of the posterior coverage analysis?

@stronk7
Copy link
Member

stronk7 commented Oct 15, 2021

Wow, sorry for the dupe comment, github is behaving weirdly from here... have delete it.

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.

2 participants