[geode] branch develop updated: GEODE-6421 CI failure: org.apache.geode.distributed.LocatorDUnitTest.testStartTwoLocators failed with AssertionError

[bs...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

bschuchardt 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 d600aa1  GEODE-6421 CI failure: org.apache.geode.distributed.LocatorDUnitTest.testStartTwoLocators failed with AssertionError
d600aa1 is described below

commit d600aa137ad3f1620fdfa086ec37f47854f97512
Author: Bruce Schuchardt <bs...@pivotal.io>
AuthorDate: Tue Feb 26 15:00:18 2019 -0800

    GEODE-6421 CI failure: org.apache.geode.distributed.LocatorDUnitTest.testStartTwoLocators failed with AssertionError
    
    Added an await() to wait for the final membership view to be processed
    before giving up.
---
 .../java/org/apache/geode/distributed/LocatorDUnitTest.java            | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java b/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java
index 289bbf0..5b92518 100644
--- a/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java
+++ b/geode-core/src/distributedTest/java/org/apache/geode/distributed/LocatorDUnitTest.java
@@ -136,7 +136,8 @@ public class LocatorDUnitTest implements java.io.Serializable {
 
   private static void expectSystemToContainThisManyMembers(final int expectedMembers) {
     assertThat(system).isNotNull();
-    assertEquals(expectedMembers, system.getDM().getViewMembers().size());
+    await()
+        .untilAsserted(() -> assertEquals(expectedMembers, system.getDM().getViewMembers().size()));
   }
 
   private static boolean isSystemConnected() {