-
Notifications
You must be signed in to change notification settings - Fork 0
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
Unit tests/mission classes #277
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again good job on getting so many tests done - this is great.
My main concern is the general lack of comments in the tests - you don't have to explain every little thing (particularly in the cases where they're essentially copy pasted across different missions), but a little context would be good (maybe think in terms of someone else who might want to add to the tests after you, what would help them get a handle on what is going on)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All good - away we go!
The majority of the unit tests are in test_erosita.py.
Other test_{telescope}.py files are all just making sure properties are returning the correct values.
I found some small bugs while testing that are also fixed in this branch.