reduce system timing sensitivity in SocketConnectorBaseTest

Now using a larger delay tolerance and additional Thread.join calls to
give the connector thread time to exit before testing assertions that
it did exit properly.

logback-core/src/test/java/ch/qos/logback/core/net/SocketConnectorBaseTest.java

@@ -37,7 +37,7 @@
  */
 public class SocketConnectorBaseTest {
 
-  private static final int DELAY = 250;
+  private static final int DELAY = 1000;
 
   private MockExceptionHandler exceptionHandler = new MockExceptionHandler();
 
@@ -65,6 +65,7 @@ public void testConnect() throws Exception {
     thread.start();
     Socket socket = connector.awaitConnection(2 * DELAY);
     assertNotNull(socket);
+    thread.join(DELAY);
     assertFalse(thread.isAlive());
     socket.close();
   }
@@ -79,7 +80,7 @@ public void testConnectionFails() throws Exception {
     assertTrue(exceptionHandler.getLastException() instanceof ConnectException);
     assertTrue(thread.isAlive());
     thread.interrupt();
-    thread.join(1000);
+    thread.join(DELAY);
     assertFalse(thread.isAlive());
     assertTrue(exceptionHandler.getLastException() instanceof InterruptedException);
   }
@@ -104,6 +105,7 @@ public void testConnectEventually() throws Exception {
     // now we should be able to connect
     socket = connector.awaitConnection(2 * DELAY);
     assertNotNull(socket);
+    thread.join(DELAY);
     assertFalse(thread.isAlive());
     socket.close();
   }