Repository: incubator-geode Updated Branches: refs/heads/feature/GEODE-409 8cb3cf7ad -> 69b5f962e
GEODE-426: Making PartitionedRegionRedundancyZoneDUnitTest clean up This test had a method called "caseTearDown" which I don't believe was actually getting run. It's really important for the tearDown code to happen, because otherwise the test leaves members configured with redundancy zones. After changing caseTearDown to tearDown, PartitionedRegionTestUtilsDUnitTest now passes when it is run after this test. Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/69b5f962 Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/69b5f962 Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/69b5f962 Branch: refs/heads/feature/GEODE-409 Commit: 69b5f962e1b229cd392c7d742d743cfabea92402 Parents: 8cb3cf7 Author: Dan Smith <[email protected]> Authored: Mon Oct 19 14:38:31 2015 -0700 Committer: Dan Smith <[email protected]> Committed: Mon Oct 19 14:42:39 2015 -0700 ---------------------------------------------------------------------- .../internal/cache/PartitionedRegionRedundancyZoneDUnitTest.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/69b5f962/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionRedundancyZoneDUnitTest.java ---------------------------------------------------------------------- diff --git a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionRedundancyZoneDUnitTest.java b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionRedundancyZoneDUnitTest.java index f7427fd..8e187ea 100644 --- a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionRedundancyZoneDUnitTest.java +++ b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/cache/PartitionedRegionRedundancyZoneDUnitTest.java @@ -27,7 +27,9 @@ import dunit.VM; public class PartitionedRegionRedundancyZoneDUnitTest extends CacheTestCase { - public static void caseTearDown() { + @Override + public void tearDown2() throws Exception { + super.tearDown2(); // this makes sure we don't leave anything for the next tests // Tests that set redundancy zones causes other jvms connected // to the ds to have "enforce-unique-hosts" set to true.
