You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2019/04/28 19:27:29 UTC

[directory-server] branch master updated: Fix flaky test

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

seelmann pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/directory-server.git


The following commit(s) were added to refs/heads/master by this push:
     new 453b479  Fix flaky test
453b479 is described below

commit 453b4793393a0129f082c39808c961d16c6dff83
Author: Stefan Seelmann <ma...@stefan-seelmann.de>
AuthorDate: Sun Apr 28 21:27:12 2019 +0200

    Fix flaky test
---
 .../client/api/operations/bind/SimpleBindRequestTest.java    | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/ldap-client-test/src/test/java/org/apache/directory/shared/client/api/operations/bind/SimpleBindRequestTest.java b/ldap-client-test/src/test/java/org/apache/directory/shared/client/api/operations/bind/SimpleBindRequestTest.java
index 8f21459..6b53575 100644
--- a/ldap-client-test/src/test/java/org/apache/directory/shared/client/api/operations/bind/SimpleBindRequestTest.java
+++ b/ldap-client-test/src/test/java/org/apache/directory/shared/client/api/operations/bind/SimpleBindRequestTest.java
@@ -641,9 +641,13 @@ public class SimpleBindRequestTest extends AbstractLdapTestUnit
 
         connection.unBind();
 
-        // this call hangs forever
-        assertFalse( cursor1.next() );
-        assertFalse( cursor2.next() );
-        assertFalse( cursor3.next() );
+        // this calls hung forever
+        /*
+         * Don't make any assumption whether cursor has next or not because entries
+         * are received async and may already been received or not.
+         */
+        cursor1.next();
+        cursor2.next();
+        cursor3.next();
     }
 }