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

testing: Use cargo-mutants to surface gaps in pavex_session's test suite #459

Merged
merged 1 commit into from
Feb 26, 2025

Conversation

LukeMathWalker
Copy link
Owner

Mutation testing highlighted a few gaps in the test suite for pavex_session that had not been caught by looking at coverage on its own.
New tests have been added to catch all discovered "MISSED" mutants, as well as guidance on how to rerun these tools when touching the crate in the future.

Kudos to @xd009642 for the ping on cargo-mutants!

…t suite

Mutation testing highlighted a few gaps in the test suite for `pavex_session`
that had not been caught by looking at coverage on its own.
New tests have been added to catch all discovered "MISSED" mutants,
as well as guidance on how to rerun these tools when touching the crate
in the future.
@LukeMathWalker
Copy link
Owner Author

/ok-to-test sha=9a55186

@LukeMathWalker LukeMathWalker merged commit c6e79c5 into main Feb 26, 2025
23 checks passed
@LukeMathWalker LukeMathWalker deleted the session-tests-again branch February 26, 2025 08:13
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.

1 participant