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 2012/05/09 17:48:33 UTC

svn commit: r1336247 - in /cxf/trunk: api/src/main/java/org/apache/cxf/common/util/CglibProxyHelper.java rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/client/JAXRSClientFactoryBeanTest.java

Author: sergeyb
Date: Wed May  9 15:48:33 2012
New Revision: 1336247

URL: http://svn.apache.org/viewvc?rev=1336247&view=rev
Log:
[CXF-4290] Fixing the cglib proxy and improving the test a bit

Modified:
    cxf/trunk/api/src/main/java/org/apache/cxf/common/util/CglibProxyHelper.java
    cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/client/JAXRSClientFactoryBeanTest.java

Modified: cxf/trunk/api/src/main/java/org/apache/cxf/common/util/CglibProxyHelper.java
URL: http://svn.apache.org/viewvc/cxf/trunk/api/src/main/java/org/apache/cxf/common/util/CglibProxyHelper.java?rev=1336247&r1=1336246&r2=1336247&view=diff
==============================================================================
--- cxf/trunk/api/src/main/java/org/apache/cxf/common/util/CglibProxyHelper.java (original)
+++ cxf/trunk/api/src/main/java/org/apache/cxf/common/util/CglibProxyHelper.java Wed May  9 15:48:33 2012
@@ -57,6 +57,7 @@ class CglibProxyHelper extends ProxyHelp
         }
         if (superClass != null) {
             Enhancer enhancer = new Enhancer();
+            enhancer.setClassLoader(loader);
             enhancer.setSuperclass(superClass);
             enhancer.setInterfaces(theInterfaces.toArray(new Class[]{}));
             enhancer.setCallback(new MethodInterceptor() {

Modified: cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/client/JAXRSClientFactoryBeanTest.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/client/JAXRSClientFactoryBeanTest.java?rev=1336247&r1=1336246&r2=1336247&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/client/JAXRSClientFactoryBeanTest.java (original)
+++ cxf/trunk/rt/frontend/jaxrs/src/test/java/org/apache/cxf/jaxrs/client/JAXRSClientFactoryBeanTest.java Wed May  9 15:48:33 2012
@@ -62,6 +62,7 @@ public class JAXRSClientFactoryBeanTest 
         bean.setClassLoader(loader);
         BookStore client = (BookStore)bean.createWithValues(BookStore.class);
         assertNotNull(client);
+        assertSame(client.getClass().getClassLoader(), loader);
         // tricky to test the loader has been used correctly with Maven 
         // given that the system loader loads all the test classes