You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by se...@apache.org on 2013/12/20 17:05:33 UTC

svn commit: r1552712 - in /cxf/trunk: rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/LocalClientState.java systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java

Author: sergeyb
Date: Fri Dec 20 16:05:33 2013
New Revision: 1552712

URL: http://svn.apache.org/r1552712
Log:
More Response cleanup on the JAX-RS client side

Modified:
    cxf/trunk/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/LocalClientState.java
    cxf/trunk/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java

Modified: cxf/trunk/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/LocalClientState.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/LocalClientState.java?rev=1552712&r1=1552711&r2=1552712&view=diff
==============================================================================
--- cxf/trunk/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/LocalClientState.java (original)
+++ cxf/trunk/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/LocalClientState.java Fri Dec 20 16:05:33 2013
@@ -118,6 +118,9 @@ public class LocalClientState implements
     
     public void reset() {
         requestHeaders.clear();
+        if (response != null) {
+            response.close();
+        }
         response = null;
         currentBuilder = new UriBuilderImpl().uri(baseURI);
         templates = null;

Modified: cxf/trunk/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java
URL: http://svn.apache.org/viewvc/cxf/trunk/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java?rev=1552712&r1=1552711&r2=1552712&view=diff
==============================================================================
--- cxf/trunk/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java (original)
+++ cxf/trunk/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSAsyncClientTest.java Fri Dec 20 16:05:33 2013
@@ -119,6 +119,7 @@ public class JAXRSAsyncClientTest extend
         } catch (ExecutionException ex) {
             assertTrue(ex.getCause() instanceof NotFoundException);
         }
+        wc.close();
     }
     
     @Test
@@ -135,6 +136,7 @@ public class JAXRSAsyncClientTest extend
         } catch (ExecutionException ex) {
             assertTrue(ex.getCause() instanceof ProcessingException);
         }
+        wc.close();
     }
     
     @Test
@@ -151,6 +153,7 @@ public class JAXRSAsyncClientTest extend
         } catch (ExecutionException ex) {
             assertTrue(ex.getCause() instanceof ResponseProcessingException);
         }
+        wc.close();
     }
     
     @Test
@@ -166,6 +169,7 @@ public class JAXRSAsyncClientTest extend
             assertTrue(ex.getCause() instanceof NotFoundException);
             assertTrue(ex.getCause() == holder.value);
         }
+        wc.close();
     }