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

Workaround DISCOVERY-531 #500

Closed mirekdlugosz closed 5 months ago

mirekdlugosz commented 5 months ago

This introduces a workaround for DISCOVERY-531: 424 errors in response to /reports/ endpoint, and JavaScript console errors on scans page are not going to be logged. This should prevent test teardown phase from erroring out.

This issue is related to timing, so it's hard to say if it will never happen again. I had another workaround prepared and I needed about 20 local runs of all e2e UI tests to encounter this problem once. I have run the same set of tests over 30 times with this patch on, and didn't have a single failure or error. That gives me confidence we should not see these pipelines failures again, but who knows.

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.97%. Comparing base (9b052ba) to head (fdcd494). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #500 +/- ## ========================================== - Coverage 76.69% 75.97% -0.72% ========================================== Files 7 7 Lines 695 720 +25 ========================================== + Hits 533 547 +14 - Misses 162 173 +11 ```

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