[geode] branch develop updated: GEODE-6277: use await() instead of hard-coded sleep(3000) in the test (#3086)

[mcmellawatt]

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 <gesterz...@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()) {