search

feder-observatory / stellarphot

Stellar aperture photometry
https://stellarphot.readthedocs.io
BSD 3-Clause "New" or "Revised" License
4 stars 12 forks source link

Move dictionaries needed for testing to new file #466

Closed mwcraig closed 1 month ago

mwcraig commented 1 month ago

This ensures that we can get at those constants without having to have test dependencies (e.g. pytest) installed.

Fixes #465

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 80.40%. Comparing base (7f6ab4c) to head (2ae556c). Report is 7 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #466 +/- ## ========================================== + Coverage 80.32% 80.40% +0.07% ========================================== Files 30 31 +1 Lines 3949 3960 +11 ========================================== + Hits 3172 3184 +12 + Misses 777 776 -1 ```

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

mwcraig commented 1 month ago

I think, though I haven't checked, that putting it in test might have required the test imports? Not actually positive about that. I was inspired by a similar thing in ipyautoui...