Improve libazureinit unit test coverage to 100%#295
Open
peytonr18 wants to merge 19 commits intoAzure:mainfrom
Open
Improve libazureinit unit test coverage to 100%#295peytonr18 wants to merge 19 commits intoAzure:mainfrom
peytonr18 wants to merge 19 commits intoAzure:mainfrom
Conversation
…y testable." This reverts commit b92381c.
…ision/user.rs - Remove unreachable \r stripping in encode_report - Fix llvm-cov false negatives on multi-line asserts - Add test for real backend fallback in provision - Split UserProvisioner::create() to make sudoers path testable
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #295 +/- ##
==========================================
+ Coverage 84.77% 91.52% +6.74%
==========================================
Files 17 17
Lines 3435 3905 +470
==========================================
+ Hits 2912 3574 +662
+ Misses 523 331 -192 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
peytonr18
force-pushed
the
probertson-code-coverage
branch
from
d871e97 to
d38d62a
Compare
…to fix coverage gaps
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Summary
This PR brings unit test code coverage (as measured by
cargo-llvm-cov) to 100% across 11 source files in thelibazureinitcrate. The moduleskvp.rsandlogging.rsare excluded as they are scheduled for refactoring.