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

The last-but-one argument of test testQueryAlignmentStart in HtsCRAMCodec30And21QueryTest.java is long #1631

Closed
pgrt opened this issue Nov 3, 2022 · 1 comment · Fixed by #1632

Comments

@pgrt
Copy link

pgrt commented Nov 3, 2022

Hello,

In the test file HtsCRAMCodec30And21QueryTest.java, the last-but-one argument of testQueryAlignmentStart is long but the alignmentStartQueries data provider returns an int instead, leading to test failures on my side.

The attached patch provides a straightforward fix.

data-types-in-tests.txt

Cheers,

Pierre

@cmnbroad
Copy link
Collaborator

cmnbroad commented Nov 7, 2022

@pgrt Yes, thanks for reporting this!

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