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

Change test to a different large file #169

Merged
merged 1 commit into from
Jul 16, 2024
Merged

Conversation

coverbeck
Copy link
Contributor

@coverbeck coverbeck commented Jul 12, 2024

Description
A unit test started failing because the ICGC data portal was shut down.

Found a reasonably large file to replace the test with, although it's "only" 78M, compared to the previous file's 5.24GB.

The point of the test is to verify that a HEAD operation works, and while it's less than ideal, I think the file is big enough.

Also changed CircleCI build to use npm ci instead of npm i.

Issue
dockstore/dockstore#5935
DOCK-2548

Security
If there are any concerns that require extra attention from the security team, highlight them here.

Please make sure that you've checked the following before submitting your pull request. Thanks!

  • Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.

@coverbeck coverbeck self-assigned this Jul 12, 2024
@coverbeck coverbeck merged commit efff2c6 into develop Jul 16, 2024
11 checks passed
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 this pull request may close these issues.

4 participants