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

TEST: Remove external file downloads in tests #936

Open
asoplata opened this issue Nov 14, 2024 · 2 comments
Open

TEST: Remove external file downloads in tests #936

asoplata opened this issue Nov 14, 2024 · 2 comments
Assignees
Labels
Milestone

Comments

@asoplata
Copy link
Collaborator

Currently, there are several tests that retrieve various files from different external locations on GitHub:

These files should all be downloaded and moved into the test assets directory https://github.com/jonescompneurolab/hnn-core/tree/master/hnn_core/tests/assets since they are small, and it is unnecessary to download them every time tests are run (which is frequent!)

@asoplata asoplata self-assigned this Nov 14, 2024
@asoplata asoplata added this to the 0.5 milestone Nov 14, 2024
@jasmainak
Copy link
Collaborator

agreed! If it's a small file, definitely add it to a local folder. Remember that git tracks everything, so if you expect the file to change often, it's not a good idea to add to the local repo

@asoplata
Copy link
Collaborator Author

It seems that Github server errors related to serving these files is currently leading to test runner failures: https://github.com/jonescompneurolab/hnn-core/actions/runs/13037964941/job/36372944225#step:9:74 . To be clear, these new failures appear to be due to issues with Github servers themselves, not our use of them. These failures will probably spontaneously go away soon, but this is another reason to tackle this issue.

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

No branches or pull requests

2 participants