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

Add infection test tool #1306

Merged
merged 4 commits into from May 15, 2024
Merged

Add infection test tool #1306

merged 4 commits into from May 15, 2024

Conversation

brettmc
Copy link
Collaborator

@brettmc brettmc commented May 7, 2024

  • run unit tests in random order
    running in random order highlights some interference between tests (mostly logging being enabled), and is a requirement for
    mutation testing (ie, all tests must pass when executed in a random order). Fix the failing tests so that everything consistently passes when tests are run in random order.
  • adding infection mutation testing
    this adds the ability to run infection. It generates a lot of output, which is an exercise for another PR...

running in random order highlights some interference between tests (mostly logging being enabled), and is a requirement for
mutation testing (ie, all tests must pass when executed in a random order).
this adds the ability to run infection. It generates a lot of output, which is an exercise for another PR...
@brettmc brettmc requested a review from a team as a code owner May 7, 2024 09:58
Copy link

codecov bot commented May 7, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 74.08%. Comparing base (500f6ee) to head (06b4349).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1306      +/-   ##
============================================
+ Coverage     74.03%   74.08%   +0.04%     
- Complexity     2385     2386       +1     
============================================
  Files           353      353              
  Lines          7133     7135       +2     
============================================
+ Hits           5281     5286       +5     
+ Misses         1852     1849       -3     
Flag Coverage Δ
8.1 74.00% <0.00%> (-0.04%) ⬇️
8.2 74.00% <0.00%> (-0.04%) ⬇️
8.3 74.01% <0.00%> (-0.03%) ⬇️
8.4 74.00% <0.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/API/LoggerHolder.php 100.00% <ø> (ø)
src/API/Behavior/Internal/Logging.php 0.00% <0.00%> (ø)

... and 6 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 500f6ee...06b4349. Read the comment docs.

@brettmc brettmc mentioned this pull request May 9, 2024
Makefile Outdated Show resolved Hide resolved
@brettmc brettmc merged commit 4f32817 into open-telemetry:main May 15, 2024
10 checks passed
@brettmc brettmc deleted the infection branch May 15, 2024 10:47
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.

None yet

3 participants