svn commit: r588518 - in /incubator/cxf/trunk: parent/pom.xml rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java systests/src/test/java/org/apache/cxf/systest/ws/rm/SequenceTest.java

[ningjiang]

Author: ningjiang
Date: Thu Oct 25 21:27:41 2007
New Revision: 588518

URL: http://svn.apache.org/viewvc?rev=588518&view=rev
Log:
CXF-1134 fixed the rm systest error, also added the checking of addr length

Modified:
    incubator/cxf/trunk/parent/pom.xml
    
incubator/cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java
    
incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/rm/SequenceTest.java

Modified: incubator/cxf/trunk/parent/pom.xml
URL: 
http://svn.apache.org/viewvc/incubator/cxf/trunk/parent/pom.xml?rev=588518&r1=588517&r2=588518&view=diff
==============================================================================
--- incubator/cxf/trunk/parent/pom.xml (original)
+++ incubator/cxf/trunk/parent/pom.xml Thu Oct 25 21:27:41 2007
@@ -954,14 +954,7 @@
                                 <property>
                                     <name>spring.validation.mode</name>
                                     <value>${spring.validation.mode}</value>
-                                </property>
-                                <property>
-                                    <!-- If we close the port after a client 
in the same JVM has oppened a keep-alive
-                                    connection, on some Linux's, the socket 
remains consumed until
-                                    some timeout occurs.   That may cause some 
tests to timeout/hang. -->
-                                    
<name>org.apache.cxf.transports.http_jetty.DontClosePort</name>
-                                    <value>true</value>
-                                </property>
+                                </property>                                
                             </systemProperties>
                         </configuration>
                     </plugin>

Modified: 
incubator/cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java
URL: 
http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java?rev=588518&r1=588517&r2=588518&view=diff
==============================================================================
--- 
incubator/cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java
 (original)
+++ 
incubator/cxf/trunk/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java
 Thu Oct 25 21:27:41 2007
@@ -174,7 +174,8 @@
     }
 
     private String removeTrailingSeparator(String addr) {
-        if (addr != null && addr.lastIndexOf('/') == addr.length() - 1) {
+        if (addr != null && addr.length() > 0 
+            && addr.lastIndexOf('/') == addr.length() - 1) {
             return addr.substring(0, addr.length() - 1);
         } else {
             return addr;

Modified: 
incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/rm/SequenceTest.java
URL: 
http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/rm/SequenceTest.java?rev=588518&r1=588517&r2=588518&view=diff
==============================================================================
--- 
incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/rm/SequenceTest.java
 (original)
+++ 
incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/ws/rm/SequenceTest.java
 Thu Oct 25 21:27:41 2007
@@ -808,6 +808,8 @@
         if (!doTestOnewayMessageLoss) {
             return;
         }
+        // waite a while for the last bus shutdown
+        Thread.sleep(5000);
         testOnewayMessageLoss(null);
     }