Removed TimeoutTest#testUnknownHost.

- does not test what it should (google.com is not unknown)
- introduces intermittent build failures (different exceptions are thrown from on various platforms)
- we don't want to have dependency on external service during mvn clean install

Change-Id: Ida425e587fb0049589f3cb64145e6e68b152faa4

Author: pavelbucek

connectors/apache-connector/src/test/java/org/glassfish/jersey/apache/connector/TimeoutTest.java

@@ -59,6 +59,7 @@
 import static org.hamcrest.CoreMatchers.instanceOf;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThat;
+import static org.junit.Assert.fail;
 
 /**
  * @author Martin Matula
@@ -110,6 +111,7 @@ public void testFast() {
     public void testSlow() {
         try {
             target("test/timeout").request().get();
+            fail("Timeout expected.");
         } catch (final ProcessingException e) {
             assertThat("Unexpected processing exception cause",
                     e.getCause(), instanceOf(SocketTimeoutException.class));

connectors/grizzly-connector/src/test/java/org/glassfish/jersey/grizzly/connector/TimeoutTest.java

@@ -57,6 +57,7 @@
 import static org.hamcrest.CoreMatchers.instanceOf;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThat;
+import static org.junit.Assert.fail;
 
 /**
  * @author Martin Matula
@@ -103,6 +104,7 @@ public void testFast() {
     public void testSlow() {
         try {
             target("test/timeout").request().get();
+            fail("Timeout expected.");
         } catch (ProcessingException e) {
             assertThat("Unexpected processing exception cause",
                     e.getCause(), instanceOf(TimeoutException.class));

connectors/jetty-connector/src/test/java/org/glassfish/jersey/jetty/connector/TimeoutTest.java

@@ -40,10 +40,7 @@
 
 package org.glassfish.jersey.jetty.connector;
 
-import java.net.NoRouteToHostException;
-import java.net.SocketTimeoutException;
 import java.net.URI;
-import java.net.URISyntaxException;
 import java.util.concurrent.TimeoutException;
 import java.util.logging.Logger;
 
@@ -63,10 +60,10 @@
 import org.glassfish.jersey.test.JerseyTest;
 
 import org.junit.Test;
-import static org.hamcrest.CoreMatchers.anyOf;
 import static org.hamcrest.CoreMatchers.instanceOf;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertThat;
+import static org.junit.Assert.fail;
 
 /**
  * @author Martin Matula
@@ -122,29 +119,12 @@ public void testSlow() {
         WebTarget t = c.target(u);
         try {
             t.path("test/timeout").request().get();
+            fail("Timeout expected.");
         } catch (ProcessingException e) {
             assertThat("Unexpected processing exception cause",
                     e.getCause(), instanceOf(TimeoutException.class));
         } finally {
             c.close();
         }
     }
-
-    @Test
-    public void testUnknownHost() throws URISyntaxException {
-        ClientConfig config = new ClientConfig().property(ClientProperties.CONNECT_TIMEOUT, 1000);
-        config.connectorProvider(new JettyConnectorProvider());
-        Client c = ClientBuilder.newClient(config);
-        final URI u = new URI("http://google.com:81");
-        WebTarget target = c.target(u);
-        try {
-            target.request().get();
-        } catch (ProcessingException e) {
-            assertThat("Unexpected processing exception cause",
-                    e.getCause().getCause(),
-                    anyOf(instanceOf(SocketTimeoutException.class), instanceOf(NoRouteToHostException.class)));
-        } finally {
-            c.close();
-        }
-    }
 }

tests/e2e/src/test/java/org/glassfish/jersey/tests/e2e/client/TimeoutTest.java

@@ -101,6 +101,7 @@ public void testFast() {
     public void testSlow() {
         try {
             target("test/timeout").request().get();
+            fail("Timeout expected.");
         } catch (ProcessingException e) {
             if (!(e.getCause() instanceof SocketTimeoutException)) {
                 e.printStackTrace();