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

Monitor mutation coverage score of test suite #1797

Closed
bdsl opened this issue Jun 16, 2019 · 4 comments
Closed

Monitor mutation coverage score of test suite #1797

bdsl opened this issue Jun 16, 2019 · 4 comments

Comments

@bdsl
Copy link
Contributor

bdsl commented Jun 16, 2019

Broken out from #1788.

As discussed, this isn't a priority, but my unsuccessful attempt at doing this using Infection is now merged in. It isn't doing any harm beyond taking up a few minutes of build time for each commit, but obviously having a useless build step is not ideal.

This is the command used to run Infection in the current master:

command: php -d memory_limit=4G ~/.composer/vendor/bin/infection --coverage=build/phpunit --only-covered --threads=2 || echo 'Temporarily ignoring unexplained infection failure'
. The || echo is there to suppress the non-zero return code.

The output is visible at https://circleci.com/gh/vimeo/psalm/39 and looks like:


#!/bin/bash -eo pipefail
php -d memory_limit=4G ~/.composer/vendor/bin/infection --coverage=build/phpunit --only-covered --threads=2 || echo 'Temporarily ignoring unexplained infection failure'

     ____      ____          __  _
    /  _/___  / __/__  _____/ /_(_)___  ____
    / // __ \/ /_/ _ \/ ___/ __/ / __ \/ __ \
  _/ // / / / __/  __/ /__/ /_/ / /_/ / / / /
 /___/_/ /_/_/  \___/\___/\__/_/\____/_/ /_/

Running initial test suite...

PHPUnit version: 8.2.2

Generate mutants...

Processing source code files: 543
/bin/bash: line 1:   184 Killed                  php -d memory_limit=4G ~/.composer/vendor/bin/infection --coverage=build/phpunit --only-covered --threads=2
Temporarily ignoring unexplained infection failure

I never managed to work out why Infection is apparently crashing after three minutes.

@maks-rafalko
Copy link
Contributor

For reference infection/infection#705

@mundia521
Copy link

Still working out

@bdsl
Copy link
Contributor Author

bdsl commented Mar 11, 2020

It may be possible to resolve this following the next release of Infection - I've been testing infection's master branch on Psalm. See latest comments on infection/infection#705 .

@muglug
Copy link
Collaborator

muglug commented Apr 26, 2020

I think this is not worth pursuing

@muglug muglug closed this as completed Apr 26, 2020
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

No branches or pull requests

4 participants