Closed mirekdlugosz closed 1 year ago
Merging #412 (438dd90) into main (bc61228) will decrease coverage by
2.11%
. The diff coverage is40.00%
.
@@ Coverage Diff @@
## main #412 +/- ##
==========================================
- Coverage 81.22% 79.11% -2.11%
==========================================
Files 5 5
Lines 245 249 +4
==========================================
- Hits 199 197 -2
- Misses 46 52 +6
Impacted Files | Coverage Δ | |
---|---|---|
camayoc/utils.py | 71.87% <14.28%> (-8.78%) |
:arrow_down: |
camayoc/api.py | 69.47% <100.00%> (+0.65%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
This PR enables reports and scan / scanjobs tests. I tried to make commits relatively complete, so it's best to review things commit-by-commit.
It depends on config changes introduced in discovery-ci PR #7.
Most changes revolve around using DataProvider and untangling dependencies between functions and fixtures. I decided to not port tests that are marked as skipped - I ran them before starting this work, and it seems that referenced issues are still not solved.
Performance of current version is not great, as most tests run scans they need - this leads to servers being scanned multiple times. I think this is something we can try to tackle later on.
Test results are below. There's one failure, but that tests seems flaky - it usually fails, but I saw it passing few times. Another thing to tackle later.