search

quipucords / camayoc

Test automation framework that facilitates functional testing of quipucords.
https://camayoc.readthedocs.io/
GNU General Public License v3.0
5 stars 4 forks source link

Use bulk delete API in cleanup #490

Closed mirekdlugosz closed 6 months ago

mirekdlugosz commented 6 months ago

Add support for bulk_delete and use it in data_provider cleanup code. Hopefully this way we will avoid any problems caused by rate throttling.

This may be merged only after https://github.com/quipucords/quipucords/pull/2609 is merged.

Camayoc tests for this PR are going to fail for this reason. I ran standalone job with this branch against container image from 2609: this is job build #69.

codecov[bot] commented 6 months ago

Codecov Report

Attention: Patch coverage is 71.42857% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 77.55%. Comparing base (96c08c7) to head (731e5a8).

Files Patch % Lines
camayoc/qpc_models.py 71.42% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #490 +/- ## ========================================== - Coverage 77.72% 77.55% -0.18% ========================================== Files 7 7 Lines 633 637 +4 ========================================== + Hits 492 494 +2 - Misses 141 143 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mirekdlugosz commented 6 months ago

Looks like standard Camayoc tests did not fail, which I find a bit surprising, but I guess that's even better.