Merge pull request #89 from TheJoeMoore/master

Fix and test for JERSEY-2556 - issue with JerseyUriBuilder.path and generic methods

Author: Adam Lindenthal

core-common/src/main/java/org/glassfish/jersey/uri/internal/JerseyUriBuilder.java

@@ -376,9 +376,9 @@ public JerseyUriBuilder path(Class resource, String methodName) {
         Method found = null;
         for (Method m : methods) {
             if (methodName.equals(m.getName())) {
-                if (found == null) {
+                if (found == null || found.isSynthetic()) {
                     found = m;
-                } else {
+                } else if (!m.isSynthetic()) {
                     throw new IllegalArgumentException();
                 }
             }

core-common/src/test/java/org/glassfish/jersey/uri/internal/JerseyUriBuilderTest.java

@@ -51,6 +51,7 @@
 
 import javax.ws.rs.GET;
 import javax.ws.rs.Path;
+import javax.ws.rs.PathParam;
 import javax.ws.rs.core.MultivaluedMap;
 import javax.ws.rs.core.PathSegment;
 import javax.ws.rs.core.UriBuilder;
@@ -1108,6 +1109,28 @@ public void testResourceWithTemplateRegexAppendPath() throws NoSuchMethodExcepti
         Assert.assertEquals(URI.create("http://localhost:8080/base/method/foo/locator/bar"), ub);
     }
 
+    interface GenericInterface<T, U> {
+        T find (U u);
+    }
+
+    @Path("resource/")
+    class ResourceWithGenericInterface implements GenericInterface<Object, String> {
+
+        @GET
+        @Path("{id}")
+        @Override
+        public Object find(@PathParam("id") String s) {
+            return null;
+        }
+    }
+
+    @Test
+    public void testResourceWithGenericInterfaceAppendPath() {
+        URI ub = UriBuilder.fromUri("http://localhost:8080/base")
+                .path(ResourceWithGenericInterface.class, "find").build("foo");
+        Assert.assertEquals(URI.create("http://localhost:8080/base/foo"), ub);
+    }
+
     @Test
     public void testBuildTemplates() {
         URI uri = UriBuilder.fromUri("http://localhost:8080/a/b/c").