geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From dschnei...@apache.org
Subject incubator-geode git commit: GEODE-712: Move junit asserts out of finally block This closes #68
Date Wed, 06 Jan 2016 23:33:07 GMT
Repository: incubator-geode
Updated Branches:
  refs/heads/develop dfb3306c5 -> 7c281327a


GEODE-712: Move junit asserts out of finally block
This closes #68


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

Branch: refs/heads/develop
Commit: 7c281327a5d7e249755a4d87a7dc12d39615eb03
Parents: dfb3306
Author: Scott Jewell <sjewell@pivotal.io>
Authored: Wed Jan 6 13:52:34 2016 -0800
Committer: Darrel Schneider <dschneider@pivotal.io>
Committed: Wed Jan 6 15:32:07 2016 -0800

----------------------------------------------------------------------
 .../gemfire/internal/offheap/SimpleMemoryAllocatorJUnitTest.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/7c281327/gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/SimpleMemoryAllocatorJUnitTest.java
----------------------------------------------------------------------
diff --git a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/SimpleMemoryAllocatorJUnitTest.java
b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/SimpleMemoryAllocatorJUnitTest.java
index 4dda498..6bc488d 100644
--- a/gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/SimpleMemoryAllocatorJUnitTest.java
+++ b/gemfire-core/src/test/java/com/gemstone/gemfire/internal/offheap/SimpleMemoryAllocatorJUnitTest.java
@@ -217,13 +217,13 @@ public class SimpleMemoryAllocatorJUnitTest {
         assertFalse(listener.isClosed());
         assertFalse(stats2.isClosed());
         stats = stats2;
-      } finally {
         ma.close();
         assertTrue(listener.isClosed());
         assertFalse(stats.isClosed());
+      } finally {
         SimpleMemoryAllocatorImpl.freeOffHeapMemory();
-        assertTrue(stats.isClosed());
       }
+      assertTrue(stats.isClosed());
     }
   }
   @Test


Mime
View raw message