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

Introduce mutation testing #1031

Merged
merged 1 commit into from
Mar 15, 2023
Merged

Introduce mutation testing #1031

merged 1 commit into from
Mar 15, 2023

Conversation

martintmk
Copy link
Contributor

@martintmk martintmk commented Jan 23, 2023

The issue or feature being addressed

This PR adds the mutation testing that verifies the quality of the unit tests written.

See https://stryker-mutator.io/ for more details.

Details on the issue fix or feature implementation

No new tests added, just introdution of mutation gate. The current mutation score is 72, and new PRs that add features cannot fall below this threshold. Over time the score should increase by adding more through tests.

Confirm the following

  • I started this PR by branching from the head of the default branch
  • I have targeted the PR to merge into the default branch
  • I have included unit tests for the issue/feature
  • I have successfully run a local build

@codecov-commenter
Copy link

codecov-commenter commented Jan 23, 2023

Codecov Report

Merging #1031 (0689041) into main (a399bd1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1031   +/-   ##
=======================================
  Coverage   71.45%   71.45%           
=======================================
  Files         137      137           
  Lines        3745     3745           
  Branches      748      748           
=======================================
  Hits         2676     2676           
  Misses        864      864           
  Partials      205      205           
Flag Coverage Δ
linux 71.45% <ø> (ø)
macos 71.45% <ø> (ø)
windows 71.45% <ø> (ø)

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

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@martintmk martintmk marked this pull request as ready for review January 23, 2023 10:15
@martincostello
Copy link
Member

I'm on vacation until Monday, I'll take a look then.

@martintmk martintmk merged commit 8affebc into main Mar 15, 2023
@martincostello martincostello deleted the mtomka/mutation-testing branch March 15, 2023 09:36
@martincostello martincostello added this to the v8.0.0 milestone Mar 16, 2023
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.

3 participants