Skip to content
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

debug tests behind EDL - failing builds for PRs to main #448

Closed
rwegener2 opened this issue Sep 14, 2023 · 1 comment · Fixed by #480
Closed

debug tests behind EDL - failing builds for PRs to main #448

rwegener2 opened this issue Sep 14, 2023 · 1 comment · Fixed by #480

Comments

@rwegener2
Copy link
Contributor

rwegener2 commented Sep 14, 2023

PR #447 is an example of a build that ran well with merges to development or release, but the build fails due to an authentication error when merging to main. This was an issue for the release of both 0.7.0 and 0.8.0, after the integration of earthaccess.

The main branch is failing because:

because the behind_NSIDC_login test suite is only run on the main branch (so in the list of checks, the "branch" test run passes, but when the additional tests are triggered for the "pull request" (what the merged code will look like) it fails).
(From @JessicaS11 in #447)

@JessicaS11
Copy link
Member

Per the .travis.yml setup, including the string nsidc_tests in the commit message should trigger the test_behind_earthdata_login tests.

@rwegener2 rwegener2 changed the title Failing builds for PRs to main debug tests behind EDL - failing builds for PRs to main Sep 14, 2023
@JessicaS11 JessicaS11 linked a pull request Dec 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants