You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by dk...@apache.org on 2009/04/08 20:25:31 UTC

svn commit: r763339 - in /cxf/trunk: rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java

Author: dkulp
Date: Wed Apr  8 18:25:30 2009
New Revision: 763339

URL: http://svn.apache.org/viewvc?rev=763339&view=rev
Log:
Ignore a couple tests.  I think Sergey forgot to svn add a testcase

Modified:
    cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
    cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java

Modified: cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
URL: http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java?rev=763339&r1=763338&r2=763339&view=diff
==============================================================================
--- cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java (original)
+++ cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java Wed Apr  8 18:25:30 2009
@@ -32,6 +32,7 @@
 import javax.ws.rs.core.Application;
 import javax.ws.rs.ext.Provider;
 
+import org.apache.commons.lang.ClassUtils;
 import org.apache.cxf.jaxrs.JAXRSServerFactoryBean;
 import org.apache.cxf.jaxrs.lifecycle.PerRequestResourceProvider;
 import org.apache.cxf.jaxrs.lifecycle.ResourceProvider;
@@ -217,10 +218,9 @@
     
     private Class<?> loadClass(String cName) throws ServletException {
         try {
-            return CXFNonSpringJaxrsServlet.class.getClassLoader().loadClass(cName.trim());
+            return ClassUtils.getClass(CXFNonSpringJaxrsServlet.class.getClassLoader(), cName.trim());
         } catch (ClassNotFoundException ex) {
-            ex.printStackTrace();
-            throw new ServletException("Resource class " + cName.trim() + " can not be loaded"); 
+            throw new ServletException("Resource class " + cName.trim() + " can not be loaded", ex); 
         }
     }
 }

Modified: cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java
URL: http://svn.apache.org/viewvc/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java?rev=763339&r1=763338&r2=763339&view=diff
==============================================================================
--- cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java (original)
+++ cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java Wed Apr  8 18:25:30 2009
@@ -49,6 +49,7 @@
     }
     
     @Test
+    @org.junit.Ignore
     public void testGetBook123ApplicationSingleton() throws Exception {
         getAndCompareAsStrings("http://localhost:9080/application/bookstore/books/123",
                                "resources/expected_get_book123.txt",
@@ -57,6 +58,7 @@
     }
     
     @Test
+    @org.junit.Ignore
     public void testGetBook123ApplicationPerRequest() throws Exception {
         getAndCompareAsStrings("http://localhost:9080/application/bookstore2/bookheaders",
                                "resources/expected_get_book123.txt",



Re: svn commit: r763339 - in /cxf/trunk: rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java

Posted by Sergey Beryozkin <sb...@progress.com>.
Thanks Dan, indeed I forgot to add one file - will do shortly....

cheers, Sergey

----- Original Message ----- 
From: <dk...@apache.org>
To: <co...@cxf.apache.org>
Sent: Wednesday, April 08, 2009 7:25 PM
Subject: svn commit: r763339 - in /cxf/trunk: 
rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java 
systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java


> Author: dkulp
> Date: Wed Apr  8 18:25:30 2009
> New Revision: 763339
>
> URL: http://svn.apache.org/viewvc?rev=763339&view=rev
> Log:
> Ignore a couple tests.  I think Sergey forgot to svn add a testcase
>
> Modified:
>    cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
>    cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java
>
> Modified: cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java
> URL: 
> http://svn.apache.org/viewvc/cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java?rev=763339&r1=763338&r2=763339&view=diff
> ==============================================================================
> --- cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java (original)
> +++ cxf/trunk/rt/frontend/jaxrs/src/main/java/org/apache/cxf/jaxrs/servlet/CXFNonSpringJaxrsServlet.java Wed Apr  8 18:25:30 2009
> @@ -32,6 +32,7 @@
> import javax.ws.rs.core.Application;
> import javax.ws.rs.ext.Provider;
>
> +import org.apache.commons.lang.ClassUtils;
> import org.apache.cxf.jaxrs.JAXRSServerFactoryBean;
> import org.apache.cxf.jaxrs.lifecycle.PerRequestResourceProvider;
> import org.apache.cxf.jaxrs.lifecycle.ResourceProvider;
> @@ -217,10 +218,9 @@
>
>     private Class<?> loadClass(String cName) throws ServletException {
>         try {
> -            return CXFNonSpringJaxrsServlet.class.getClassLoader().loadClass(cName.trim());
> +            return ClassUtils.getClass(CXFNonSpringJaxrsServlet.class.getClassLoader(), cName.trim());
>         } catch (ClassNotFoundException ex) {
> -            ex.printStackTrace();
> -            throw new ServletException("Resource class " + cName.trim() + " can not be loaded");
> +            throw new ServletException("Resource class " + cName.trim() + " can not be loaded", ex);
>         }
>     }
> }
>
> Modified: cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java
> URL: 
> http://svn.apache.org/viewvc/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java?rev=763339&r1=763338&r2=763339&view=diff
> ==============================================================================
> --- cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java (original)
> +++ cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSClientServerNonSpringBookTest.java Wed Apr  8 18:25:30 2009
> @@ -49,6 +49,7 @@
>     }
>
>     @Test
> +    @org.junit.Ignore
>     public void testGetBook123ApplicationSingleton() throws Exception {
>         getAndCompareAsStrings("http://localhost:9080/application/bookstore/books/123",
>                                "resources/expected_get_book123.txt",
> @@ -57,6 +58,7 @@
>     }
>
>     @Test
> +    @org.junit.Ignore
>     public void testGetBook123ApplicationPerRequest() throws Exception {
>         getAndCompareAsStrings("http://localhost:9080/application/bookstore2/bookheaders",
>                                "resources/expected_get_book123.txt",
>
>