Skip to content

Implement pytest-remotedata for test_psm3.py #882

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

Closed
CameronTStark opened this issue Feb 6, 2020 · 0 comments · Fixed by #896
Closed

Implement pytest-remotedata for test_psm3.py #882

CameronTStark opened this issue Feb 6, 2020 · 0 comments · Fixed by #896
Assignees
Labels
Milestone

Comments

@CameronTStark
Copy link
Contributor

An issue has been raised in #878 that the test suite is too slow. This issue suggests taking a quantitative step proposed by @kanderso-nrel to implement pytest-remotedata to help remove tests to speed up the process.

pytest-remotedata enables the ability to mark tests that need to access remote data and only execute them if pytest is provided a --remote-data flag. I plan to submit a PR implementing this for test_psm3.py.

I'm also proposing that the --remote-data flag is implemented in only one of the Azure Pipelines jobs so we still get the benefit of testing it via CI but still reduce the quantity of requests by 75-80%. I think the --remote-data flag would be best executed in 'Test_bare_linux' since it would be the easiest to troubleshoot from but I could be convinced otherwise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant