You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by ro...@apache.org on 2008/11/17 19:19:54 UTC

svn commit: r718312 - /activemq/camel/trunk/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/InterfacesTest.java

Author: romkal
Date: Mon Nov 17 10:19:54 2008
New Revision: 718312

URL: http://svn.apache.org/viewvc?rev=718312&view=rev
Log:
CAMEL-1091 - Fix test that can recognize localhost as remote interface

Modified:
    activemq/camel/trunk/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/InterfacesTest.java

Modified: activemq/camel/trunk/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/InterfacesTest.java
URL: http://svn.apache.org/viewvc/activemq/camel/trunk/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/InterfacesTest.java?rev=718312&r1=718311&r2=718312&view=diff
==============================================================================
--- activemq/camel/trunk/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/InterfacesTest.java (original)
+++ activemq/camel/trunk/components/camel-jetty/src/test/java/org/apache/camel/component/jetty/InterfacesTest.java Mon Nov 17 10:19:54 2008
@@ -3,7 +3,6 @@
 import java.io.IOException;
 import java.net.InetAddress;
 import java.net.NetworkInterface;
-import java.net.SocketException;
 import java.net.URL;
 import java.util.Enumeration;
 
@@ -19,13 +18,13 @@
         // Retrieve an address of some remote network interface
         Enumeration<NetworkInterface> interfaces =  NetworkInterface.getNetworkInterfaces();
         
-        while(interfaces.hasMoreElements()) {
+        while(remoteInterfaceAddress == null && interfaces.hasMoreElements()) {
             NetworkInterface interfaze = interfaces.nextElement();
             Enumeration<InetAddress> addresses =  interfaze.getInetAddresses();
             if(addresses.hasMoreElements()) {                
                 InetAddress nextAddress = addresses.nextElement();
-                if (nextAddress.isLoopbackAddress() || nextAddress.isReachable(2000)) {
-                    break;
+                if (nextAddress.isLoopbackAddress() || !nextAddress.isReachable(2000)) {
+                    continue;
                 }
                 remoteInterfaceAddress = nextAddress.getHostAddress();
             }