You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by mc...@apache.org on 2019/02/07 01:14:44 UTC
[geode] branch develop updated: GEODE-6277: use await() instead of
hard-coded sleep(3000) in the test (#3086)
This is an automated email from the ASF dual-hosted git repository.
mcmellawatt 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 bfc0449 GEODE-6277: use await() instead of hard-coded sleep(3000) in the test (#3086)
bfc0449 is described below
commit bfc0449e81b7ac5429f7473b216762c22c10f95b
Author: Xiaojian Zhou <ge...@users.noreply.github.com>
AuthorDate: Wed Feb 6 17:14:34 2019 -0800
GEODE-6277: use await() instead of hard-coded sleep(3000) in the test (#3086)
---
.../java/org/apache/geode/cache30/MultiVMRegionTestCase.java | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/geode-dunit/src/main/java/org/apache/geode/cache30/MultiVMRegionTestCase.java b/geode-dunit/src/main/java/org/apache/geode/cache30/MultiVMRegionTestCase.java
index de400e0..ff4747c 100644
--- a/geode-dunit/src/main/java/org/apache/geode/cache30/MultiVMRegionTestCase.java
+++ b/geode-dunit/src/main/java/org/apache/geode/cache30/MultiVMRegionTestCase.java
@@ -4971,13 +4971,9 @@ public abstract class MultiVMRegionTestCase extends RegionTestCase {
// reset slow
org.apache.geode.internal.cache.InitialImageOperation.slowImageProcessing = 0;
// if global scope, the region doesn't get destroyed until after region creation
- try {
- Thread.sleep(3000);
- } catch (InterruptedException ie) {
- fail("interrupted");
- }
- assertThat(getRootRegion().getSubregion(name) == null
- || getRegionAttributes().getScope().isGlobal()).isTrue();
+ await().until(
+ () -> getRootRegion().getSubregion(name) == null || getRegionAttributes()
+ .getScope().isGlobal());
}
});
if (getRegionAttributes().getScope().isGlobal()) {