Phar distribution #2
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: borNfreee <b0rn@list.ru>
Signed-off-by: borNfreee <b0rn@list.ru>
Signed-off-by: borNfreee <b0rn@list.ru>
Codecov Report
@@ Coverage Diff @@
## master #2 +/- ##
============================================
- Coverage 36.16% 33.81% -2.36%
- Complexity 638 666 +28
============================================
Files 112 113 +1
Lines 1941 2076 +135
============================================
Hits 702 702
- Misses 1239 1374 +135
Continue to review full report at Codecov.
|
Closed
sanmai
referenced
this pull request
in sanmai/infection
May 10, 2018
Parameter #2 $env of method Symfony\Component\Process\Process::start() expects array, array|null given.
maks-rafalko
pushed a commit
that referenced
this pull request
May 10, 2018
…t process (#329) * Got back PhpExecutableFinderTest * Use a default Xdebug-free environment for all subprocesses * Extended test suite for Exec_Path e2e test * Added docblock comments * Use setTimeout() instead of a long list of null'ed constructor's options * setupEnvironment() renamed into setupXdebugFreeEnvironment() * A comment about restoring environment * ProjectCodeTest should only consider our own classes We should consider only properties belonging to our classes, but not to foreign classes we're exteding from. E.g. we can't change Symfony\Component\Process\Process to not have a public propery it has. * PhpExecutableFinderTest tagges as internal/final * Fix for Process::start() API incompatibility with version 4 Parameter #2 $env of method Symfony\Component\Process\Process::start() expects array, array|null given.
maks-rafalko
added a commit
that referenced
this pull request
Dec 9, 2020
* create mutators * avaible only string param * unit tests * cs-fix * mutatesNode rule * coallesce on mutators * regex only removes carret and dolar * code format * riddiculus pretty fix * Add mutators to schema.json, add more tests, add psalm annotations, add support for all possible delimiters * Add psalm issues to baseline because thecodingmachine/safe does not support psalm annotations yet thecodingmachine/safe#141 * Add test with an unpacked array * Simplify code by using ?? operator Co-authored-by: bernard.van.der.esch <bernard.van.der.esch@orba.co> Co-authored-by: Bernard van der Esch <b.vanderesch@gmail.com>
maks-rafalko
added a commit
that referenced
this pull request
Aug 7, 2021
…andom ordering of the tests since we need them to be sorted (fastest - first) When Infection build Mutant's `phpunit.xml` file, we add `<file>` tags, sorted by tests execution time, so that the fastest tests runs first. But when the original `phpunit.xml` contains `executionOrder="default"`, it was inherited to Mutant's `phpunit.xml` and broke the order, running the tests in a random order each time `infection` was executed. This also explains why I got different execution time of Infection for the project with *functional* tests. When Mutant is covered by 1000 functional tests, and tests are executed by random, it can be killed by the first test in run #1 and can be killed by 1000-th test in run #2, if the killing test is executed the last. With this update, all the tests will be executed in the same orders (fastest - first) in all the infection runs.
maks-rafalko
added a commit
that referenced
this pull request
Aug 7, 2021
…andom ordering of the tests since we need them to be sorted (fastest - first) (#1547) When Infection build Mutant's `phpunit.xml` file, we add `<file>` tags, sorted by tests execution time, so that the fastest tests runs first. But when the original `phpunit.xml` contains `executionOrder="default"`, it was inherited to Mutant's `phpunit.xml` and broke the order, running the tests in a random order each time `infection` was executed. This also explains why I got different execution time of Infection for the project with *functional* tests. When Mutant is covered by 1000 functional tests, and tests are executed by random, it can be killed by the first test in run #1 and can be killed by 1000-th test in run #2, if the killing test is executed the last. With this update, all the tests will be executed in the same orders (fastest - first) in all the infection runs.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The first implementation of Phar distribution and self-update command