From 404fd43d272d26a9bce43b8e3d540d242a0878e5 Mon Sep 17 00:00:00 2001 From: James Date: Mon, 25 Jul 2022 15:02:31 -0400 Subject: [PATCH] fixed the broken test switch and added a test --- ...irichletAlleleDepthAndFractionIntegrationTest.java | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/src/test/java/org/broadinstitute/hellbender/tools/walkers/annotator/DirichletAlleleDepthAndFractionIntegrationTest.java b/src/test/java/org/broadinstitute/hellbender/tools/walkers/annotator/DirichletAlleleDepthAndFractionIntegrationTest.java index 48e4666fefd..472b76c49c6 100644 --- a/src/test/java/org/broadinstitute/hellbender/tools/walkers/annotator/DirichletAlleleDepthAndFractionIntegrationTest.java +++ b/src/test/java/org/broadinstitute/hellbender/tools/walkers/annotator/DirichletAlleleDepthAndFractionIntegrationTest.java @@ -7,6 +7,7 @@ import org.broadinstitute.hellbender.tools.walkers.haplotypecaller.HaplotypeCaller; import org.broadinstitute.hellbender.tools.walkers.mutect.SomaticLikelihoodsEngine; import org.broadinstitute.hellbender.utils.Utils; +import org.testng.Assert; import org.testng.annotations.DataProvider; import org.testng.annotations.Test; @@ -20,13 +21,21 @@ public class DirichletAlleleDepthAndFractionIntegrationTest extends GatkToolInte public static final String TEST_FILES_DIR = toolsTestDir + "haplotypecaller/"; - public static final boolean UPDATE_EXACT_MATCH_EXPECTED_OUTPUTS = true; + public static final boolean UPDATE_EXACT_MATCH_EXPECTED_OUTPUTS = false; @Override public String getTestedToolName() { return HaplotypeCaller.class.getSimpleName(); } + /* + * Make sure that someone didn't leave the UPDATE_EXACT_MATCH_EXPECTED_OUTPUTS toggle turned on + */ + @Test + public void assertThatExpectedOutputUpdateToggleIsDisabled() { + Assert.assertFalse(UPDATE_EXACT_MATCH_EXPECTED_OUTPUTS, "The toggle to update expected outputs should not be left enabled"); + } + /* * Test that in VCF mode we're consistent with past GATK4 results */