-
Notifications
You must be signed in to change notification settings - Fork 39
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
Update unit tests to remove some downloading #565
Conversation
f4990a0
to
9b06df3
Compare
9b06df3
to
b1bb0d6
Compare
@bbuzz31 can you take a quick look at the error message on this? This PR is passing locally and I ran the test independently and completes just fine. |
@bbuzz31 I think this can be merged! |
test/scenario_5/temp.yaml
Outdated
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.
can / should this yaml file be created on the fly like the other ones?
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.
Also, can/should it point to the global testing WM_DIR (RAiDER/test/weather_files)?
test/test_GUNW.py
Outdated
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.
This test will get skipped due to @pytest.mark.long?
yes?
I think we'd prefer that not to happen, as the GUNW workflow will be cranking lots of products
This PR focuses on streamlining unit tests to avoid downloading when possible
Description
Motivation and Context
How Has This Been Tested?
Type of change
Checklist: