You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by ag...@apache.org on 2016/05/03 23:45:44 UTC

[30/60] [abbrv] incubator-geode git commit: GEODE-1240: Changed the test to use Awaitility with a maximum timeout period. This might work better than the time sensitive conditionals that this test uses.

GEODE-1240: Changed the test to use Awaitility with a maximum timeout period. This might work better than the time sensitive conditionals that this test uses.


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/7e2ca6ca
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/7e2ca6ca
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/7e2ca6ca

Branch: refs/heads/feature/GEODE-1209
Commit: 7e2ca6ca8fb5d9768503b30bf71362513c6a5212
Parents: 9fbf219
Author: Udo Kohlmeyer <uk...@pivotal.io>
Authored: Thu Apr 28 11:37:15 2016 +1000
Committer: Udo Kohlmeyer <uk...@pivotal.io>
Committed: Thu Apr 28 11:37:15 2016 +1000

----------------------------------------------------------------------
 .../cache30/ClientMembershipDUnitTest.java      | 34 ++++++++++----------
 1 file changed, 17 insertions(+), 17 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/7e2ca6ca/geode-core/src/test/java/com/gemstone/gemfire/cache30/ClientMembershipDUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/com/gemstone/gemfire/cache30/ClientMembershipDUnitTest.java b/geode-core/src/test/java/com/gemstone/gemfire/cache30/ClientMembershipDUnitTest.java
index 9036e5e..f8e036b 100755
--- a/geode-core/src/test/java/com/gemstone/gemfire/cache30/ClientMembershipDUnitTest.java
+++ b/geode-core/src/test/java/com/gemstone/gemfire/cache30/ClientMembershipDUnitTest.java
@@ -81,23 +81,23 @@ public class ClientMembershipDUnitTest extends ClientServerTestCase {
 
   private void waitForAcceptsInProgressToBe(final int target)
       throws Exception {
-    WaitCriterion ev = new WaitCriterion() {
-      String excuse;
-
-      public boolean done() {
-        int actual = getAcceptsInProgress();
-        if (actual == getAcceptsInProgress()) {
-          return true;
-        }
-        excuse = "accepts in progress (" + actual + ") never became " + target;
-        return false;
-      }
-
-      public String description() {
-        return excuse;
-      }
-    };
-    Awaitility.await().pollInterval(200, TimeUnit.MILLISECONDS).atMost(60, TimeUnit.SECONDS)
+//    WaitCriterion ev = new WaitCriterion() {
+//      String excuse;
+//
+//      public boolean done() {
+//        int actual = getAcceptsInProgress();
+//        if (actual == getAcceptsInProgress()) {
+//          return true;
+//        }
+//        excuse = "accepts in progress (" + actual + ") never became " + target;
+//        return false;
+//      }
+//
+//      public String description() {
+//        return excuse;
+//      }
+//    };
+    Awaitility.await().pollInterval(100, TimeUnit.MILLISECONDS).pollDelay(100,TimeUnit.MILLISECONDS).timeout(300, TimeUnit.SECONDS)
         .until(() -> {
           int actual = getAcceptsInProgress();
           if (actual == getAcceptsInProgress()) {