diff --git a/core-server/src/main/java/org/glassfish/jersey/server/model/internal/ModelHelper.java b/core-server/src/main/java/org/glassfish/jersey/server/model/internal/ModelHelper.java index 61bcb946ec..0cc95a3a4a 100644 --- a/core-server/src/main/java/org/glassfish/jersey/server/model/internal/ModelHelper.java +++ b/core-server/src/main/java/org/glassfish/jersey/server/model/internal/ModelHelper.java @@ -1,7 +1,7 @@ /* * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. * - * Copyright (c) 2013 Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2013-2014 Oracle and/or its affiliates. All rights reserved. * * The contents of this file are subject to the terms of either the GNU * General Public License Version 2 only ("GPL") or the Common Development @@ -46,6 +46,7 @@ * Common model helper methods. * * @author Michal Gajdos (michal.gajdos at oracle.com) + * @author Constantino Cronemberger (ccronemberger at yahoo.com.br) */ public final class ModelHelper { @@ -58,15 +59,20 @@ public final class ModelHelper { * annotation. */ public static Class getAnnotatedResourceClass(Class resourceClass) { - if (resourceClass.isAnnotationPresent(Path.class)) { - return resourceClass; - } - for (Class i : resourceClass.getInterfaces()) { - if (i.isAnnotationPresent(Path.class)) { - return i; + // traverse the class hierarchy to find the annotation + Class cls = resourceClass; + do { + if (cls.isAnnotationPresent(Path.class)) { + return cls; } - } + + for (Class i : cls.getInterfaces()) { + if (i.isAnnotationPresent(Path.class)) { + return i; + } + } + } while ((cls = cls.getSuperclass()) != null); return resourceClass; } diff --git a/core-server/src/test/java/org/glassfish/jersey/server/model/internal/ModelHelperTest.java b/core-server/src/test/java/org/glassfish/jersey/server/model/internal/ModelHelperTest.java new file mode 100644 index 0000000000..ac368f36ca --- /dev/null +++ b/core-server/src/test/java/org/glassfish/jersey/server/model/internal/ModelHelperTest.java @@ -0,0 +1,88 @@ +/* + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. + * + * Copyright (c) 2013-2014 Oracle and/or its affiliates. All rights reserved. + * + * The contents of this file are subject to the terms of either the GNU + * General Public License Version 2 only ("GPL") or the Common Development + * and Distribution License("CDDL") (collectively, the "License"). You + * may not use this file except in compliance with the License. You can + * obtain a copy of the License at + * http://glassfish.java.net/public/CDDL+GPL_1_1.html + * or packager/legal/LICENSE.txt. See the License for the specific + * language governing permissions and limitations under the License. + * + * When distributing the software, include this License Header Notice in each + * file and include the License file at packager/legal/LICENSE.txt. + * + * GPL Classpath Exception: + * Oracle designates this particular file as subject to the "Classpath" + * exception as provided by Oracle in the GPL Version 2 section of the License + * file that accompanied this code. + * + * Modifications: + * If applicable, add the following below the License Header, with the fields + * enclosed by brackets [] replaced by your own identifying information: + * "Portions Copyright [year] [name of copyright owner]" + * + * Contributor(s): + * If you wish your version of this file to be governed by only the CDDL or + * only the GPL Version 2, indicate your decision by adding "[Contributor] + * elects to include this software in this distribution under the [CDDL or GPL + * Version 2] license." If you don't indicate a single choice of license, a + * recipient has the option to distribute your version of this file under + * either the CDDL, the GPL Version 2 or to extend the choice of license to + * its licensees as provided above. However, if you add GPL Version 2 code + * and therefore, elected the GPL Version 2 license, then the option applies + * only if the new code is made subject to such option by the copyright + * holder. + */ +package org.glassfish.jersey.server.model.internal; + +import org.junit.Assert; +import org.junit.Test; + +import javax.ws.rs.Path; +import java.lang.reflect.InvocationHandler; +import java.lang.reflect.Method; +import java.lang.reflect.Proxy; + +/** + * @author Constantino Cronemberger (ccronemberger at yahoo.com.br) + */ +public class ModelHelperTest { + + @Test + public void testClass() { + Class cls = ModelHelper.getAnnotatedResourceClass(MyAnnotatedClass.class); + Assert.assertSame(MyAnnotatedClass.class, cls); + } + + @Test + public void testSubClass() { + // Spring with CGLIB proxies creates sub-classes + Object obj = new MyAnnotatedClass() {}; + Assert.assertNotSame(MyAnnotatedClass.class,obj.getClass()); + Class cls = ModelHelper.getAnnotatedResourceClass(obj.getClass()); + Assert.assertSame(MyAnnotatedClass.class, cls); + } + + @Test + public void testProxyClass() throws Exception { + // Spring can also create proxies for beans + Object obj = Proxy.newProxyInstance(getClass().getClassLoader(), new Class[] {MyServiceInterface.class}, new InvocationHandler() { + @Override + public Object invoke(Object proxy, Method method, Object[] args) throws Throwable { + return null; + } + }); + Class cls = ModelHelper.getAnnotatedResourceClass(obj.getClass()); + Assert.assertSame(MyServiceInterface.class, cls); + } + + @Path("test") + public static interface MyServiceInterface {} + + @Path("test") + public static class MyAnnotatedClass implements MyServiceInterface {} +}