search

spacetelescope / calcos

Calibration for HST/COS.
https://hst-docs.stsci.edu/cosdhb/chapter-3-cos-calibration
3 stars 27 forks source link

use `tmp_path` in unit tests to fix concurrent file access / deletion #243

Closed zacharyburnett closed 2 months ago

zacharyburnett commented 4 months ago

so that tests can be run concurrently without stepping on each others' toes

zacharyburnett commented 4 months ago

~whoops, it looks like my editor auto-formatted the test files with ruff 😅~ all fixed now

it should be functionally identical however (tests pass now)

codecov[bot] commented 4 months ago

Codecov Report

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

Project coverage is 83.90%. Comparing base (8eb63d1) to head (4908465). Report is 6 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #243 +/- ## ========================================== - Coverage 87.64% 83.90% -3.75% ========================================== Files 17 17 Lines 1392 1323 -69 ========================================== - Hits 1220 1110 -110 - Misses 172 213 +41 ```

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