Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add tests for rrdesi and rrboss #71

Open
sbailey opened this issue Jan 19, 2018 · 0 comments
Open

add tests for rrdesi and rrboss #71

sbailey opened this issue Jan 19, 2018 · 0 comments
Milestone

Comments

@sbailey
Copy link
Collaborator

sbailey commented Jan 19, 2018

redrock unit tests currently test the algorithms but not the command-line scripts rrdesi and rrboss. We keep breaking these scripts but not immediately noticing because the tests don't cover them (e.g. PR #65 completely broke rrdesi due to an incorrect import, and #67 broke some options).

Add unit tests that create spectra and spPlate files on the fly and test that these scripts and their options still work.

@sbailey sbailey added this to the 18.3 milestone Jan 19, 2018
This was referenced Jan 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant