Update and clean up test environments

astropy / specreduce

Tools for the reduction of spectroscopic observations from Optical and NIR instruments

https://specreduce.readthedocs.io

58 stars 38 forks source link

Update and clean up test environments #190

Closed rosteen closed 11 months ago

rosteen commented 11 months ago

This does a few things:

Cleans up lines related to numpy and astropy test envs that aren't actually used in any tests.
Adds/updates tests to use python 3.12, installs dev astropy for those to get around deprecation warning.
Fixes (I hope) the weekly cron test, which was failing because there was no predeps test environment defined.
Forces dev numpy to install in devdeps after it would be downgraded by matplotlib.

Closes #185, #186, #187, #188.

codecov[bot] commented 11 months ago

Codecov Report

Merging #190 (793f658) into main (0d800fd) will decrease coverage by 2.53%. The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #190      +/-   ##
==========================================
- Coverage   81.35%   78.83%   -2.53%     
==========================================
  Files          10       10              
  Lines         992      992              
==========================================
- Hits          807      782      -25     
- Misses        185      210      +25

see 2 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

rosteen commented 11 months ago

Alright, I think I got all the 3.8 and 3.9 tests removed or upgraded to 3.10. Hopefully good to go now.

tepickering commented 11 months ago

yup, looks good to me! thanks!

tepickering commented 11 months ago

ah, one thing i forgot to mention. the coverage went down because it looks like a couple of tests are now getting missed. it's for orphaned sections of the code, though, so i'd be fine with accepting that and merging anyway.