-
Notifications
You must be signed in to change notification settings - Fork 92
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
Integration tests are flaky -- replace dataset sampling with top 50 datasets #215
Comments
Maybe we could keep the random-sample test as an optional test suite? It would be useful to keep track of corner cases, but we wouldn't really want our main regression test suite to fail because of a new corner case. |
@mfisher87 we're leaving this open for now knowing that it's likely resolved with #749. Are you still needing reviewers to move the PR forward? Lmk how I can help facilitate! |
We do need more work on that PR. It's been tough moving it forward just during hack days! I think the remaining tasks are achievable without needing decision input, just requires some time. |
@asteiker the PR is ready now, just needs review! |
Right now the integration tests use randomly sampled datasets from different DAACs, this results in intermittent failures due EULAs/unavailability etc. A probably better approach would be to test only the most popular datasets (+IS2 data). A list of the most popular datasets from NASA is attached.
collection_short_names.txt
The text was updated successfully, but these errors were encountered: