Fixed issue 1746 - Api and ApiImplicitParam annotations not read when they are in a parent of the resource class

Also credit to mohitmutha for finding a solution to a similar problem and captdestrukto to provide a pull request for that one

Author: janhoeve

modules/swagger-jaxrs/src/main/java/io/swagger/jaxrs/Reader.java

@@ -187,7 +187,7 @@ private Swagger read(Class<?> cls, String parentPath, String parentMethod, boole
         String[] produces = new String[0];
         final Set<Scheme> globalSchemes = EnumSet.noneOf(Scheme.class);
 
-        Api api = (Api) cls.getAnnotation(Api.class);
+        Api api = ReflectionUtils.getAnnotation(cls, Api.class);
 
         boolean hasPathAnnotation = (ReflectionUtils.getAnnotation(cls, javax.ws.rs.Path.class) != null);
         boolean hasApiAnnotation = (api != null);
@@ -410,7 +410,7 @@ private Swagger read(Class<?> cls, String parentPath, String parentMethod, boole
     }
 
     private void readImplicitParameters(Method method, Operation operation) {
-        ApiImplicitParams implicitParams = method.getAnnotation(ApiImplicitParams.class);
+        ApiImplicitParams implicitParams = ReflectionUtils.getAnnotation(method, ApiImplicitParams.class);
         if (implicitParams != null && implicitParams.value().length > 0) {
             for (ApiImplicitParam param : implicitParams.value()) {
                 Parameter p = readImplicitParam(param);

modules/swagger-jaxrs/src/test/java/io/swagger/ReaderTest.java

@@ -3,6 +3,7 @@
 import io.swagger.jaxrs.Reader;
 import io.swagger.models.Operation;
 import io.swagger.models.Swagger;
+import io.swagger.models.Tag;
 import io.swagger.models.parameters.*;
 import io.swagger.resources.*;
 import org.testng.annotations.Test;
@@ -316,6 +317,27 @@ public void scanDeclaredExceptionsAndCombineWithMethodResponsesClassLevel() {
 
     }
 
+    @Test(description = "scan resource (impl) which has the Api annotations only declared in its interface")
+    public void scanApiAnnotationWhichAreOnlyPresentInInterfaceAndNotInImplementation() {
+        Swagger swagger = getSwagger(ResourceWithAnnotationsOnlyInInterfaceImpl.class);
+        assertNotNull(swagger);
+
+        final List<Tag> tags = swagger.getTags();
+        assertEquals(tags.size(), 1);
+        assertEquals(tags.get(0).getName(), "someTag");
+    }
+
+    @Test(description = "scan resource (impl) which has the ApiParam annotations only declared in its interface")
+    public void scanApiImplicitParamAnnotationWhichAreOnlyPresentInInterfaceAndNotInImplementation() {
+        Swagger swagger = getSwagger(ResourceWithAnnotationsOnlyInInterfaceImpl.class);
+        assertNotNull(swagger);
+
+        List<Parameter> parameters = getGet(swagger, "/pet/randomPet").getParameters();
+        assertNotNull(parameters);
+        assertEquals(parameters.size(), 1);
+        assertEquals(parameters.get(0).getName(), "petImplicitIdParam");
+    }
+
     private Swagger getSwagger(Class<?> cls) {
         return new Reader(new Swagger()).read(cls);
     }

modules/swagger-jaxrs/src/test/java/io/swagger/resources/ResourceWithAnnotationsOnlyInInterface.java

@@ -0,0 +1,20 @@
+package io.swagger.resources;
+
+import io.swagger.annotations.Api;
+import io.swagger.annotations.ApiImplicitParam;
+import io.swagger.annotations.ApiImplicitParams;
+import io.swagger.annotations.ApiOperation;
+
+import javax.ws.rs.GET;
+import javax.ws.rs.Path;
+
+@Path("/pet")
+@Api(tags = "someTag")
+public interface ResourceWithAnnotationsOnlyInInterface {
+
+    @GET
+    @Path("/randomPet")
+    @ApiOperation(value = "getRandomPet")
+    @ApiImplicitParams({ @ApiImplicitParam(name = "petImplicitIdParam", paramType = "query", dataType = "string") })
+    String getRandomPet();
+}

modules/swagger-jaxrs/src/test/java/io/swagger/resources/ResourceWithAnnotationsOnlyInInterfaceImpl.java

@@ -0,0 +1,9 @@
+package io.swagger.resources;
+
+public class ResourceWithAnnotationsOnlyInInterfaceImpl implements ResourceWithAnnotationsOnlyInInterface {
+
+    @Override
+    public String getRandomPet() {
+        return "No pet today..";
+    }
+}