spacetelescope / stcal

https://stcal.readthedocs.io/en/latest/
Other
10 stars 32 forks source link

Update the jump detection unittest to check #231

Open WilliamJamieson opened 11 months ago

WilliamJamieson commented 11 months ago

In PR #227 the unit tests for jump detection were broken slightly in the sense that they did not actually attempt to account for if jumps were detected or not. This PR attempts to resolve these issues in the testing so that now the testing does account for all jumps being detected (there is one exception to this).

Checklist

codecov[bot] commented 11 months ago

Codecov Report

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

Comparison is base (fd2d6ce) 85.98% compared to head (9c6ebbf) 86.02%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #231 +/- ## ========================================== + Coverage 85.98% 86.02% +0.04% ========================================== Files 35 35 Lines 6542 6533 -9 ========================================== - Hits 5625 5620 -5 + Misses 917 913 -4 ```

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