You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ab...@apache.org on 2016/02/08 18:03:42 UTC
[27/50] [abbrv] incubator-geode git commit: GEODE-712: Move junit
asserts out of finally block This closes #68
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/master
Commit: 7c281327a5d7e249755a4d87a7dc12d39615eb03
Parents: dfb3306
Author: Scott Jewell <sj...@pivotal.io>
Authored: Wed Jan 6 13:52:34 2016 -0800
Committer: Darrel Schneider <ds...@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