You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by an...@apache.org on 2006/10/20 11:58:54 UTC

svn commit: r466048 - in /incubator/cxf/trunk: rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java systests/pom.xml

Author: andreasmyth
Date: Fri Oct 20 02:58:53 2006
New Revision: 466048

URL: http://svn.apache.org/viewvc?view=rev&rev=466048
Log:
Fixed check for partial response and re-enabled addressing system test.

Modified:
    incubator/cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java
    incubator/cxf/trunk/systests/pom.xml

Modified: incubator/cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java
URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java?view=diff&rev=466048&r1=466047&r2=466048
==============================================================================
--- incubator/cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java (original)
+++ incubator/cxf/trunk/rt/core/src/main/java/org/apache/cxf/endpoint/ClientImpl.java Fri Oct 20 02:58:53 2006
@@ -301,19 +301,7 @@
     }
 
     private boolean isPartialResponse(Message in) {
-        // andreasmyth:
-        // return always false (and disable the addressing system test) 
-        // as for some reason checking the content of the message 
-        // slows performance: with some exceptions (possibly random) 
-        // individual system tests take more or less the same time either
-        // way but there seem to be problems with freeing up resources
-        // after each test run so that the total time for all system tests is
-        // excessively long.
-
-        return false;
-        /*
         return in.getContent(List.class) == null
-            && in.getContent(Exception.class) == null;
-        */
+            && getException(in.getExchange()) == null;
     }
 }

Modified: incubator/cxf/trunk/systests/pom.xml
URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/pom.xml?view=diff&rev=466048&r1=466047&r2=466048
==============================================================================
--- incubator/cxf/trunk/systests/pom.xml (original)
+++ incubator/cxf/trunk/systests/pom.xml Fri Oct 20 02:58:53 2006
@@ -158,28 +158,4 @@
         
     </dependencies>
 
-    <build>
-        <pluginManagement>
-            <plugins>
-                <plugin>
-                    <groupId>org.apache.maven.plugins</groupId>
-                    <artifactId>maven-surefire-plugin</artifactId>
-                    <version>${surefire.version}</version>
-                    <configuration>
-                        <systemProperties>
-                            <property>
-                                <name>java.util.logging.config.file</name>
-                                <value>${basedir}/target/test-classes/logging.properties</value>
-                            </property>
-                        </systemProperties>
-                        <excludes>
-                            <exclude>**/*$*</exclude>
-                            <exclude>**/MAPTest.java</exclude>
-                        </excludes>
-                    </configuration>
-                </plugin>
-            </plugins>
-        </pluginManagement>
-    </build>
-
 </project>