feat(loans): add mutation testing decorations #971
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
|
Related: a long time ago we also looked into getting code coverage info in the CI. I remember it worked, but we never integrated it |
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.
Decorates all functions in the loans pallet with
#[cfg_attr(test, mutate)], somutagencan mutate them and check if the unit tests break, "killing" the mutations. Only unit tests are used when checking, so some of the surviving mutants exist because they're tested in an integration test.Most surviving mutants just remove the
deposit_eventcall, however some logic in the interest rate model should actually be better tested. This is the output about surviving mutants: SURVIVED.txtThe
mutagendependency uses the latest master on their GitHub, because version2.xwasn't released to crates.io for some reason.