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

Create integration tests for htsget module #367

Open
aaperis opened this issue Apr 22, 2024 · 2 comments · May be fixed by #435
Open

Create integration tests for htsget module #367

aaperis opened this issue Apr 22, 2024 · 2 comments · May be fixed by #435
Assignees
@pahatz
Labels
GDI Project: GDI summer task

Comments

@aaperis
Copy link
Contributor

aaperis commented Apr 22, 2024

As an sda-admin
I want to be sure that the htsget functionality works as intended
so that I can avoid any bugs that will downgrade the user experience.

A/C:

  • integration test(s) that will check that using sda-cli htsget to retrieve data from a specific chromosome to a local .c4gh file works. The file should be decryptable.
@aaperis aaperis added the GDI Project: GDI label Apr 22, 2024
@aaperis aaperis mentioned this issue Apr 22, 2024
Closed
@pahatz pahatz self-assigned this Jul 5, 2024
@pahatz pahatz linked a pull request Aug 26, 2024 that will close this issue
Draft
@pahatz
Copy link
Contributor

pahatz commented Sep 23, 2024

Current status:
htsget downloads the encrypted files but the decryption fails with the error:
header packet length 1887007331 exceeds maximum allowed 16777216, likely stream is corrupted
The encr download and decryption works fine from download endpoint.

Comparing with the starter kit setup that works.

@pahatz
Copy link
Contributor

pahatz commented Sep 24, 2024

A lot of back and forth but haven't concluded what's exactly at fault for this error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pahatz pahatz

Labels
GDI Project: GDI summer task
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add system tests for htsget NBISweden/sda-cli
3 participants
@MalinAhlberg @pahatz @aaperis