canonical / mir-ci

Mir CI helpers
1 stars 1 forks source link

pytest: fix handling of multiple `deps` marks #63

Closed Saviq closed 5 months ago

Saviq commented 5 months ago

We were skipping after the closest mark, this makes it so that we always go through the whole list of marks and only skip at the end.

Also move the robot dependency to a mark.

TODO: detect dependency conflicts (e.g. different channels for snaps, different package versions).

codecov[bot] commented 5 months ago

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (43da10d) 64.21% compared to head (2e5103e) 64.40%. Report is 1 commits behind head on main.

Files Patch % Lines
mir-ci/mir_ci/conftest.py 57.14% 3 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #63 +/- ## ========================================== + Coverage 64.21% 64.40% +0.18% ========================================== Files 11 11 Lines 721 722 +1 Branches 88 88 ========================================== + Hits 463 465 +2 + Misses 235 234 -1 Partials 23 23 ```

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