Skip to content

Commit 8c98bb4

Browse files
committed
Fixed an error in the unit tests by chaning equivalency asserts.
1 parent ba55aac commit 8c98bb4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/testUtils/java/org/broadinstitute/hellbender/testutils/VariantContextTestUtils.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@
1919
import org.broadinstitute.hellbender.tools.walkers.annotator.Annotation;
2020
import org.broadinstitute.hellbender.tools.walkers.genotyper.AlleleSubsettingUtils;
2121
import org.broadinstitute.hellbender.tools.walkers.genotyper.GenotypeAssignmentMethod;
22-
import org.broadinstitute.hellbender.utils.MathUtils;
2322
import org.broadinstitute.hellbender.utils.Utils;
2423
import org.broadinstitute.hellbender.utils.variant.GATKVCFConstants;
2524
import org.broadinstitute.hellbender.utils.variant.GATKVCFHeaderLines;
@@ -130,7 +129,7 @@ static Object normalizeScientificNotation(final Object attribute){
130129
return Arrays.stream(split).map(
131130
s -> {return Arrays.stream(s.split(",",-1))
132131
.map(d -> {if (d.equals("")) return d;
133-
else return Double.toString(Double.parseDouble(d));})
132+
else return Double.toString(Double.parseDouble(d));})
134133
.collect(Collectors.joining(","));})
135134
.collect(Collectors.joining("|"));
136135
} else {
@@ -428,6 +427,7 @@ public static void assertVariantContextsAreEqual(final VariantContext actual, fi
428427
BaseTest.assertEqualsDoubleSmart(actual.getPhredScaledQual(), expected.getPhredScaledQual());
429428

430429
assertVariantContextsHaveSameGenotypes(actual, expected, attributesToIgnore);
430+
Assert.assertEquals(actual.getType(), expected.getType(), "type");
431431
}
432432

433433
private static Map<String, Object> filterIgnoredAttributes(final Map<String,Object> attributes, final List<String> attributesToIgnore) {

0 commit comments

Comments
 (0)