Merge pull request #8174 from kristip17/UpdateContextManagerTest

Issue 8173: Update timeout on ContextManagerTest

Author: kristip17

dev/com.ibm.ws.security.wim.adapter.ldap/test/com/ibm/ws/security/wim/adapter/ldap/context/ContextManagerTest.java

@@ -1,5 +1,5 @@
 /*******************************************************************************
- * Copyright (c) 2018 IBM Corporation and others.
+ * Copyright (c) 2018, 2019 IBM Corporation and others.
  * All rights reserved. This program and the accompanying materials
  * are made available under the terms of the Eclipse Public License v1.0
  * which accompanies this distribution, and is available at
@@ -730,9 +730,11 @@ public void connectTimeout() throws Exception {
 
         try {
             /*
-             * Configure the context manager with a 100 ms connect timeout.
+             * Configure the context manager with a 500 ms connect timeout.
+             * Bumped up from a 100ms timeout as that's probably too much precision to expect from our build machines
+             * Looking for a balance of testing the timeout without making the unit test take a long time
              */
-            long expectedTimeout = 100L;
+            long expectedTimeout = 500L;
             ContextManager cm = new ContextManager();
             cm.setPrimaryServer("localhost", serverSocket.getLocalPort());
             cm.setContextPool(false, null, null, null, null, null);
@@ -744,16 +746,16 @@ public void connectTimeout() throws Exception {
                 cm.createDirContext(USER_DN, "password".getBytes());
                 fail("Expected NamingException.");
             } catch (NamingException e) {
-                // javax.naming.NamingException: LDAP response read timed out, timeout used:100ms.
+                // javax.naming.NamingException: LDAP response read timed out, timeout used:500ms.
                 time = TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - time);
                 assertTrue("Expected connect timeout to be " + expectedTimeout + " ms but was " + time,
                            time >= expectedTimeout && time <= (expectedTimeout + 200));
             }
 
             /*
-             * Configure the context manager with a 500 ms connect timeout.
+             * Configure the context manager with a 1000 ms connect timeout.
              */
-            expectedTimeout = 500L;
+            expectedTimeout = 1000L;
             cm.setConnectTimeout(expectedTimeout);
             cm.initialize();
 
@@ -762,7 +764,7 @@ public void connectTimeout() throws Exception {
                 cm.createDirContext(USER_DN, "password".getBytes());
                 fail("Expected NamingException.");
             } catch (NamingException e) {
-                // javax.naming.NamingException: LDAP response read timed out, timeout used:500ms.
+                // javax.naming.NamingException: LDAP response read timed out, timeout used:1000ms.
                 time = TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - time);
                 assertTrue("Expected connect timeout to be " + expectedTimeout + " millisecond but was " + time,
                            time >= expectedTimeout && time <= (expectedTimeout + 200));