test now closes the cache it creates

Project: http://git-wip-us.apache.org/repos/asf/geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/397b1b39
Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/397b1b39
Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/397b1b39

Branch: refs/heads/feature/GEODE-2485
Commit: 397b1b39797f88e71f0334ca14f39b4335d9a2a0
Parents: 0aebf39
Author: Darrel Schneider <dschnei...@pivotal.io>
Authored: Tue Apr 11 10:42:55 2017 -0700
Committer: Darrel Schneider <dschnei...@pivotal.io>
Committed: Tue Apr 11 10:42:55 2017 -0700

----------------------------------------------------------------------
 .../internal/cache/GemFireCacheImplTest.java    | 24 ++++++++++++--------
 1 file changed, 14 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/geode/blob/397b1b39/geode-core/src/test/java/org/apache/geode/internal/cache/GemFireCacheImplTest.java
----------------------------------------------------------------------
diff --git 
a/geode-core/src/test/java/org/apache/geode/internal/cache/GemFireCacheImplTest.java
 
b/geode-core/src/test/java/org/apache/geode/internal/cache/GemFireCacheImplTest.java
index 85bc6a9..6838e74 100644
--- 
a/geode-core/src/test/java/org/apache/geode/internal/cache/GemFireCacheImplTest.java
+++ 
b/geode-core/src/test/java/org/apache/geode/internal/cache/GemFireCacheImplTest.java
@@ -42,19 +42,23 @@ public class GemFireCacheImplTest {
     TypeRegistry typeRegistry = mock(TypeRegistry.class);
     SystemTimer ccpTimer = mock(SystemTimer.class);
     GemFireCacheImpl gfc = GemFireCacheImpl.createWithAsyncEventListeners(ds, 
cc, typeRegistry);
-    gfc.setCCPTimer(ccpTimer);
-    for (int i = 1; i < GemFireCacheImpl.PURGE_INTERVAL; i++) {
-      gfc.purgeCCPTimer();
-      verify(ccpTimer, times(0)).timerPurge();
-    }
-    gfc.purgeCCPTimer();
-    verify(ccpTimer, times(1)).timerPurge();
-    for (int i = 1; i < GemFireCacheImpl.PURGE_INTERVAL; i++) {
+    try {
+      gfc.setCCPTimer(ccpTimer);
+      for (int i = 1; i < GemFireCacheImpl.PURGE_INTERVAL; i++) {
+        gfc.purgeCCPTimer();
+        verify(ccpTimer, times(0)).timerPurge();
+      }
       gfc.purgeCCPTimer();
       verify(ccpTimer, times(1)).timerPurge();
+      for (int i = 1; i < GemFireCacheImpl.PURGE_INTERVAL; i++) {
+        gfc.purgeCCPTimer();
+        verify(ccpTimer, times(1)).timerPurge();
+      }
+      gfc.purgeCCPTimer();
+      verify(ccpTimer, times(2)).timerPurge();
+    } finally {
+      gfc.close();
     }
-    gfc.purgeCCPTimer();
-    verify(ccpTimer, times(2)).timerPurge();
   }
 
   @Test

Reply via email to