diff --git a/src/org/labkey/test/tests/SampleTypeLimitsTest.java b/src/org/labkey/test/tests/SampleTypeLimitsTest.java index c9cdd5a0c4..4933febe16 100644 --- a/src/org/labkey/test/tests/SampleTypeLimitsTest.java +++ b/src/org/labkey/test/tests/SampleTypeLimitsTest.java @@ -36,6 +36,7 @@ import org.labkey.test.params.list.ListDefinition; import org.labkey.test.params.list.VarListDefinition; import org.labkey.test.util.DataRegionTable; +import org.labkey.test.util.OptionalFeatureHelper; import org.labkey.test.util.PortalHelper; import org.labkey.test.util.SampleTypeHelper; import org.labkey.test.util.TestDataGenerator; @@ -117,6 +118,13 @@ private void doSetup() } } + @Override + protected void doCleanup(boolean afterTest) + { + super.doCleanup(afterTest); + OptionalFeatureHelper.resetOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp"); + } + @Test public void testStringLookupFields() throws IOException, CommandException { @@ -196,6 +204,7 @@ private void insertSampleTypeRow(String regionName, String rowValue) public void testDeriveSamplesLookupFields() throws IOException, CommandException { goToProjectHome(); + OptionalFeatureHelper.setOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp", true); log("Create sample type with lookup field to " + SAMPLE_TYPE_NAME); String sampleTypeName = "SampleTypeWithLookup"; diff --git a/src/org/labkey/test/tests/SampleTypeLineageTest.java b/src/org/labkey/test/tests/SampleTypeLineageTest.java index d64c912d99..0858a0f9c1 100644 --- a/src/org/labkey/test/tests/SampleTypeLineageTest.java +++ b/src/org/labkey/test/tests/SampleTypeLineageTest.java @@ -38,6 +38,7 @@ import org.labkey.test.params.experiment.DataClassDefinition; import org.labkey.test.params.experiment.SampleTypeDefinition; import org.labkey.test.util.DataRegionTable; +import org.labkey.test.util.OptionalFeatureHelper; import org.labkey.test.util.PortalHelper; import org.labkey.test.util.SampleTypeHelper; import org.labkey.test.util.TestDataGenerator; @@ -86,7 +87,6 @@ public static void setupProject() { SampleTypeLineageTest init = getCurrentTest(); - // Comment out this line (after you run once) it will make iterating on tests much easier. init.doSetup(); } @@ -101,6 +101,7 @@ private void doSetup() portalHelper.addWebPart("Sample Types"); portalHelper.exitAdminMode(); + OptionalFeatureHelper.setOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp", true); } @Override @@ -108,9 +109,7 @@ protected void doCleanup(boolean afterTest) throws TestTimeoutException { super.doCleanup(afterTest); - // If you are debugging tests change this function to do nothing. - // It can make re-running faster but you need to valid the integrity of the test data on your own. -// log("Do nothing."); + OptionalFeatureHelper.resetOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp"); } /** diff --git a/src/org/labkey/test/tests/SampleTypeLinkToStudyTest.java b/src/org/labkey/test/tests/SampleTypeLinkToStudyTest.java index a652b4d807..f35c7089f3 100644 --- a/src/org/labkey/test/tests/SampleTypeLinkToStudyTest.java +++ b/src/org/labkey/test/tests/SampleTypeLinkToStudyTest.java @@ -40,6 +40,7 @@ import org.labkey.test.params.FieldDefinition; import org.labkey.test.params.experiment.SampleTypeDefinition; import org.labkey.test.util.DataRegionTable; +import org.labkey.test.util.OptionalFeatureHelper; import org.labkey.test.util.PortalHelper; import org.labkey.test.util.SampleTypeHelper; import org.labkey.test.util.StudyHelper; @@ -105,8 +106,12 @@ private void doSetup() new PortalHelper(getDriver()).addBodyWebPart("Datasets"); createSampleTypes(); + OptionalFeatureHelper.setOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp", true); + } + + private void createSampleTypes() { SampleTypeHelper sampleHelper = new SampleTypeHelper(this); @@ -881,5 +886,6 @@ protected void doCleanup(boolean afterTest) throws TestTimeoutException _containerHelper.deleteProject(DATE_BASED_STUDY, false); _containerHelper.deleteProject(SAMPLE_TYPE_PROJECT + " Study 1", false); _containerHelper.deleteProject(SAMPLE_TYPE_PROJECT + " Study 2", false); + OptionalFeatureHelper.resetOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp"); } } diff --git a/src/org/labkey/test/tests/SampleTypeNameExpressionTest.java b/src/org/labkey/test/tests/SampleTypeNameExpressionTest.java index 25af8ca18e..7b3ab760d0 100644 --- a/src/org/labkey/test/tests/SampleTypeNameExpressionTest.java +++ b/src/org/labkey/test/tests/SampleTypeNameExpressionTest.java @@ -39,6 +39,7 @@ import org.labkey.test.util.AuditLogHelper; import org.labkey.test.util.DataRegionTable; import org.labkey.test.util.EscapeUtil; +import org.labkey.test.util.OptionalFeatureHelper; import org.labkey.test.util.PortalHelper; import org.labkey.test.util.SampleTypeHelper; import org.labkey.test.util.TestDataGenerator; @@ -132,6 +133,13 @@ public static void setupProject() throws IOException, CommandException test.doSetup(); } + @Override + protected void doCleanup(boolean afterTest) + { + super.doCleanup(afterTest); + OptionalFeatureHelper.resetOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp"); + } + private void addDataRow(TestDataGenerator dataGenerator, String name, int intVal) { Map sampleData = new HashMap<>(); @@ -196,7 +204,7 @@ private void doSetup() throws IOException, CommandException // Just want to get an updated view of the sample types. refresh(); - + OptionalFeatureHelper.setOptionalFeature(createDefaultConnection(), "deriveSamplesNotInApp", true); } @Before