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.

Sign up for GitHub

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

Jump to bottom

Integration tests clobber existing .netrc file #806

Closed
1 task done
chuckwondo opened this issue Sep 15, 2024 · 0 comments · Fixed by #808
Closed
1 task done

Integration tests clobber existing .netrc file #806

chuckwondo opened this issue Sep 15, 2024 · 0 comments · Fixed by #808
Labels
bug Something isn't working

Comments

@chuckwondo
Copy link
Collaborator

Is this issue already tracked somewhere, or is this a new report?

  • I've reviewed existing issues and couldn't find a duplicate for this problem.

Current Behavior

Running integration tests locally clobbers existing .netrc file, causing all existing entries to be lost.

Expected Behavior

Integration tests should not clobber local .netrc file. Instead, they should use a unique temporary location for tests requiring a .netrc file.

Steps To Reproduce

Run integration tests locally.

Environment

- OS: Any
- Python: Any

Additional Context

No response
@mfisher87 mfisher87 added the bug Something isn't working label Sep 16, 2024
chuckwondo added a commit to chuckwondo/earthaccess that referenced this issue Sep 16, 2024
@chuckwondo
Use temp .netrc file for integration tests
2e1af6a 
Fixes nsidc#806
Fixes nsidc#743
Fixes nsidc#480
@chuckwondo chuckwondo mentioned this issue Sep 16, 2024
Merged
10 tasks
chuckwondo added a commit that referenced this issue Sep 16, 2024
@chuckwondo
Use temp .netrc file for integration tests
f5adfaa 
Also, support `NETRC` environment variable for specifying
alternative location of `.netrc` file.

Fixes #806
Fixes #743
Fixes #480
@chuckwondo chuckwondo mentioned this issue Sep 17, 2024
Closed
10 tasks
chuckwondo added a commit to chuckwondo/earthaccess that referenced this issue Sep 17, 2024
@chuckwondo
Use temp .netrc file for integration tests
e16a2be 
Fixes nsidc#806
Fixes nsidc#743
Fixes nsidc#480
chuckwondo added a commit to chuckwondo/earthaccess that referenced this issue Sep 22, 2024
@chuckwondo
Use temp .netrc file for integration tests
be0da17 
Fixes nsidc#806
Fixes nsidc#743
Fixes nsidc#480
chuckwondo added a commit to chuckwondo/earthaccess that referenced this issue Sep 24, 2024
@chuckwondo
Use temp .netrc file for integration tests
cbe3af1 
Fixes nsidc#806
Fixes nsidc#743
Fixes nsidc#480
chuckwondo added a commit to chuckwondo/earthaccess that referenced this issue Sep 26, 2024
@chuckwondo
Use temp .netrc file for integration tests
956fff3 
Fixes nsidc#806
Fixes nsidc#743
Fixes nsidc#480
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in earthaccess project Oct 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
earthaccess project
Status: ✅ Done
Milestone
No milestone
2 participants
@mfisher87 @chuckwondo