Supporting hashCode and equals calls on jersey client proxy objects.

Change-Id: I7d3814ab15af847c8465b10d56ca4492bd467ece

Author: Chandler

ext/proxy-client/src/main/java/org/glassfish/jersey/client/proxy/WebResourceFactory.java

@@ -1,7 +1,7 @@
 /*
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
  *
- * Copyright (c) 2012-2015 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2012-2017 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
@@ -163,6 +163,16 @@ public Object invoke(final Object proxy, final Method method, final Object[] arg
             return toString();
         }
 
+        if (args == null && method.getName().equals("hashCode")) {
+            //unique instance in the JVM, and no need to override
+            return hashCode();
+        }
+
+        if (args != null && args.length == 1 && method.getName().equals("equals")) {
+            //unique instance in the JVM, and no need to override
+            return equals(args[0]);
+        }
+
         // get the interface describing the resource
         final Class<?> proxyIfc = proxy.getClass().getInterfaces()[0];
 

ext/proxy-client/src/test/java/org/glassfish/jersey/client/proxy/WebResourceFactoryTest.java

@@ -1,7 +1,7 @@
 /*
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
  *
- * Copyright (c) 2012-2015 Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2012-2017 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
@@ -61,6 +61,8 @@
 import org.junit.Ignore;
 import org.junit.Test;
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotEquals;
 import static org.junit.Assert.assertTrue;
 
 /**
@@ -69,6 +71,7 @@
 public class WebResourceFactoryTest extends JerseyTest {
 
     private MyResourceIfc resource;
+    private MyResourceIfc resource2;
     private MyResourceIfc resourceWithXML;
 
     @Override
@@ -86,6 +89,7 @@ protected ResourceConfig configure() {
     public void setUp() throws Exception {
         super.setUp();
         resource = WebResourceFactory.newResource(MyResourceIfc.class, target());
+        resource2 = WebResourceFactory.newResource(MyResourceIfc.class, target());
 
         final MultivaluedMap<String, Object> headers = new MultivaluedHashMap<>(1);
         headers.add(HttpHeaders.CONTENT_TYPE, MediaType.APPLICATION_XML);
@@ -342,4 +346,16 @@ public void testToString() throws Exception {
 
         assertEquals(expected, actual);
     }
+
+    @Test
+    public void testHashCode() throws Exception {
+        int h1 = resource.hashCode();
+        int h2 = resource2.hashCode();
+        assertNotEquals("The hash codes should not match", h1, h2);
+    }
+
+    @Test
+    public void testEquals() {
+        assertFalse("The two resource instances should not be considered equals as they are unique", resource.equals(resource2));
+    }
 }