-
Notifications
You must be signed in to change notification settings - Fork 35
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 unit test #643
Add unit test #643
Conversation
289e13a
to
5d7c2cf
Compare
ping |
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.
See comment, otherwise this is probably fine : )
python3 NuRadioReco/detector/test/test_response.py |
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.
I think this won't work reliably - if the first python script raises an error, the second one will probably still run, and the github test will only consider the output of the second test. Feel free to put this into a new test, I think.
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.
I see thanks! I fixed it in the workflow yaml. Would that work?
ping |
@cg-laser maybe you can approve this? |
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.
Oops! Should have approved this a long time ago. Thank Felix!
This PR adds unit tests for the Response class. So far only the multiply operator functionality is tests ....
Based on PR #635. Solves #642