This is an automated email from the ASF dual-hosted git repository.
lgallinat pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git
The following commit(s) were added to refs/heads/develop by this push:
new 5627f05 GEODE-3190 Changed test hook to be volatile, added additional
log to … (#789)
5627f05 is described below
commit 5627f0529fed802c13df4181813160b4e22c57f3
Author: agingade <[email protected]>
AuthorDate: Fri Sep 22 09:18:41 2017 -0700
GEODE-3190 Changed test hook to be volatile, added additional log to …
(#789)
* GEODE-3190 Added additional log to diagnose the issue in test hook.
---
.../org/apache/geode/internal/cache/Bug48182JUnitTest.java | 13 ++++++++++++-
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git
a/geode-core/src/test/java/org/apache/geode/internal/cache/Bug48182JUnitTest.java
b/geode-core/src/test/java/org/apache/geode/internal/cache/Bug48182JUnitTest.java
index 61bc5a4..c6a3354 100644
---
a/geode-core/src/test/java/org/apache/geode/internal/cache/Bug48182JUnitTest.java
+++
b/geode-core/src/test/java/org/apache/geode/internal/cache/Bug48182JUnitTest.java
@@ -161,6 +161,8 @@ public class Bug48182JUnitTest {
// e.printStackTrace();
fail("Did not receive a CacheClosedException. Received a " +
e.getClass().getName()
+ " instead.");
+ } finally {
+ AbstractRegionMap.testHookRunnableFor48182 = null;
}
assertTrue("A CacheClosedException was not triggered", correctException);
@@ -177,7 +179,14 @@ public class Bug48182JUnitTest {
AbstractRegionMap.testHookRunnableFor48182 = new Runnable() {
@Override
public void run() {
- getCache().getRegion(getRegionName()).destroyRegion();
+ Cache cache = getCache();
+ Region region = cache.getRegion(getRegionName());
+ region.destroyRegion();
+ if (cache.getLogger() != null) {
+ cache.getLogger()
+ .info("Region " + getRegionName() + " is destroyed : " +
region.isDestroyed());
+ }
+ assertTrue("Region " + getRegionName() + " is not destroyed.",
region.isDestroyed());
}
};
@@ -196,6 +205,8 @@ public class Bug48182JUnitTest {
// e.printStackTrace();
fail("Did not receive a RegionDestroyedException. Received a " +
e.getClass().getName()
+ " instead.");
+ } finally {
+ AbstractRegionMap.testHookRunnableFor48182 = null;
}
assertTrue("A RegionDestroyedException was not triggered",
correctException);
--
To stop receiving notification emails like this one, please contact
['"[email protected]" <[email protected]>'].
