Closed mirekdlugosz closed 1 month ago
Attention: Patch coverage is 16.66667%
with 10 lines
in your changes missing coverage. Please review.
Project coverage is 76.85%. Comparing base (
a7c813e
) to head (5100fc6
). Report is 1 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
camayoc/utils.py | 16.66% | 10 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Subject says it all - with that patch, UI tests that use network credentials with SSH key pass when running against UI v2. In fact, with that patch all the UI tests pass when running against UI v2.
The only new commit here is 735f6686772f40c3b2cd93d0af03b8bce388dd7d . Otherwise, it includes entirety of https://github.com/quipucords/camayoc/pull/541 . I open it as draft for now, with the plan that we merge #541 and then I will rebase that and open it for merge.