Closed mirekdlugosz closed 1 year ago
CI failure here is https://about.codecov.io/blog/message-regarding-the-pypi-package/ . Will tackle that in separate PR #393
Merging #392 (0b3dd61) into master (64f1d04) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #392 +/- ##
=======================================
Coverage 80.57% 80.57%
=======================================
Files 5 5
Lines 242 242
=======================================
Hits 195 195
Misses 47 47
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Based on discussion in DISCOVERY-286 - add Satellite to this test coverage.
Also, update expected error message - it was changed in https://github.com/quipucords/quipucords/commit/13ae8a1d3fe449844b82224813cb465155c6c41e .
The point of DISCOVERY-286 is that there's legitimate change in behavior, so these tests are currently failing anyway. There might be more changes needed as DISCOVERY-286 is fixed.