geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From upthewatersp...@apache.org
Subject incubator-geode git commit: GEODE-426: Making PartitionedRegionRedundancyZoneDUnitTest clean up
Date Mon, 19 Oct 2015 21:42:57 GMT
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 <upthewaterspout@apache.org>
Authored: Mon Oct 19 14:38:31 2015 -0700
Committer: Dan Smith <upthewaterspout@apache.org>
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.


Mime
View raw message