Skip to content

Commit

Permalink
Fix minor inconsistency in test data.
Browse files Browse the repository at this point in the history
test/sam's test_bam_set1_write_and_read_back.tmp.bam file had mate
information (rnext, pnext, tlen) for a record that was not paired.
This triggers a CRAM "fix" where some, but strangley not all, of this
was changed.

Arguably CRAM should correct all 3 or none, but regardless the test
data we are generating should be correct.
  • Loading branch information
jkbonfield authored and daviesrob committed Oct 9, 2024
1 parent 24e4e31 commit ca92061
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/sam.c
Original file line number Diff line number Diff line change
Expand Up @@ -2254,7 +2254,7 @@ static void test_bam_set1_write_and_read_back(void)
w_bam = bam_init1();
VERIFY(w_bam != NULL, "failed to initialize BAM struct.");
r = bam_set1(w_bam, strlen(qname), qname,
BAM_FREVERSE, 0, 1000, 42,
BAM_FPAIRED | BAM_FREVERSE, 0, 1000, 42,
sizeof(cigar) / 4, cigar, 0, 2000, 3000,
strlen(seq), seq, qual, 64);
VERIFY(r >= 0, "call to bam_set1() failed.");
Expand Down

0 comments on commit ca92061

Please sign in to comment.