No longer require that Resources implement equals() and hashCode().

Two problems:
1. Most corresponding underlying middleware implementations don't require them to be implemented, so it's probably futile to require it at this level.
2. Resources are now intrinsically tied to Sessions, which means that equality doesn't quite make sense anymore.

Author: rickbw

src/main/java/crud/core/Resource.java

@@ -38,18 +38,5 @@
  * @author Rick Warren
  */
 public interface Resource extends AsyncCloseable {
-
-    /**
-     * A resource is considered to have an intrinsic identity. Two resource
-     * objects are considered to be equal if they have the same identity.
-     * For example, two resources available via HTTP would be equal if they
-     * point to the same URI. Equality should <em>not</em> require performing
-     * expensive value retrievals and comparing the results.
-     */
-    @Override
-    public abstract boolean equals(Object other);
-
-    @Override
-    public abstract int hashCode();
-
+    // empty
 }

src/main/java/crud/pattern/ResourceMerger.java

@@ -15,7 +15,6 @@
 package crud.pattern;
 
 import crud.core.ReadableResource;
-import crud.core.Resource;
 import crud.core.WritableResource;
 import crud.transform.TransformedReadableResource;
 import crud.transform.TransformedWritableResource;
@@ -84,16 +83,6 @@ public static <RRSRC, WRSRC, RESPONSE> ResourceMerger<RESPONSE> mapToWriter(
      */
     public abstract Func0<Observable<RESPONSE>> toFunction();
 
-    /**
-     * Two mergers are considered to be equal if and only if their
-     * constituent {@link Resource}s and {@link Function}s (if any) are equal.
-     */
-    @Override
-    public abstract boolean equals(Object other);
-
-    @Override // overridden here to avoid compiler warnings; see impl. below
-    public abstract int hashCode();
-
     private ResourceMerger() {
         // restrict visibility
     }

src/main/java/crud/transform/DelegateObjectMethods.java

@@ -15,7 +15,6 @@
 package crud.transform;
 
 import java.util.Objects;
-import java.util.concurrent.Callable;
 
 import crud.core.ReadableResource;
 import rx.Observable;
@@ -64,21 +63,7 @@ public <TO> TransformedReadableResource<TO> mapValue(
      * this resource.
      */
     public Func0<Observable<RSRC>> toFunction() {
-        return toResourceCallable();
-    }
-
-    /**
-     * Return a {@link Callable} that delegates to {@link #read()}.
-     * The {@code Callable} overrides {@link Object#equals(Object)},
-     * {@link Object#hashCode()}, and {@link Object#toString()} in terms of
-     * this resource.
-     */
-    public Callable<Observable<RSRC>> toCallable() {
-        return toResourceCallable();
-    }
-
-    private DelegateObjectMethods.Callable<Observable<RSRC>> toResourceCallable() {
-        return new DelegateObjectMethods.Callable<Observable<RSRC>>(this) {
+        return new Func0<Observable<RSRC>>() {
             @Override
             public Observable<RSRC> call() {
                 return TransformedReadableResource.this.read();

src/main/java/crud/transform/TransformedReadableResource.java

@@ -69,7 +69,7 @@ public TransformedReadableResource<RSRC> get(final K key, final Session session)
     }
 
     public Func2<KEY, Session, TransformedReadableResource<RSRC>> toFunction() {
-        return new DelegateObjectMethods.Function2<KEY, Session, TransformedReadableResource<RSRC>>(this) {
+        return new Func2<KEY, Session, TransformedReadableResource<RSRC>>() {
             @Override
             public TransformedReadableResource<RSRC> call(final KEY key, final Session session) {
                 return get(key, session);

src/main/java/crud/transform/TransformedReadableResourceSet.java

@@ -78,7 +78,7 @@ public <TO> TransformedWritableResource<TO, RESPONSE> adaptNewValue(
      * this resource.
      */
     public Func1<RSRC, Observable<RESPONSE>> toFunction() {
-        return new DelegateObjectMethods.Function1<RSRC, Observable<RESPONSE>>(this) {
+        return new Func1<RSRC, Observable<RESPONSE>>() {
             @Override
             public Observable<RESPONSE> call(final RSRC newValue) {
                 return TransformedWritableResource.this.write(newValue);

src/main/java/crud/transform/TransformedWritableResource.java

@@ -83,7 +83,7 @@ public TransformedWritableResource<RSRC, RESPONSE> get(final K key, final Sessio
     }
 
     public Func2<KEY, Session, TransformedWritableResource<RSRC, RESPONSE>> toFunction() {
-        return new DelegateObjectMethods.Function2<KEY, Session, TransformedWritableResource<RSRC, RESPONSE>>(this) {
+        return new Func2<KEY, Session, TransformedWritableResource<RSRC, RESPONSE>>() {
             @Override
             public TransformedWritableResource<RSRC, RESPONSE> call(final KEY key, final Session session) {
                 return get(key, session);

src/main/java/crud/transform/TransformedWritableResourceSet.java

@@ -14,7 +14,6 @@
  */
 package crud.core;
 
-import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertNotNull;
 import static org.mockito.Mockito.mock;
 
@@ -42,20 +41,6 @@ public void getDefaultKeyReturnsNonNullResource() {
         assertNotNull(resource);
     }
 
-    @Test
-    public void twoResourcesFromSameKeyAreEqual() {
-        // given:
-        final ResourceSet<KEY, ?> rsrcSet = createDefaultResourceSet();
-        final KEY key = createDefaultKey();
-
-        // when:
-        final Resource resource1 = rsrcSet.get(key, this.mockSession);
-        final Resource resource2 = rsrcSet.get(key, this.mockSession);
-
-        // then:
-        assertEquals(resource1, resource2);
-    }
-
     /**
      * Create and return a new instance of the {@link ResourceSet} class
      * under test.

src/test/java/crud/core/ResourceSetTest.java

@@ -1,4 +1,4 @@
-/* Copyright 2014 Rick Warren
+/* Copyright 2014–2015 Rick Warren
  *
  * Licensed under the Apache License, Version 2.0 (the "License"); you may not
  * use this file except in compliance with the License. You may obtain a copy of
@@ -14,10 +14,6 @@
  */
 package crud.core;
 
-import static org.junit.Assert.assertEquals;
-
-import org.junit.Test;
-
 
 /**
  * A base class for all unit test classes that operate on
@@ -26,26 +22,6 @@
  */
 public abstract class ResourceTest {
 
-    @Test
-    public void twoDefaultInstancesAreEqual() {
-        // given:
-        final Resource resource1 = createDefaultResource();
-        final Resource resource2 = createDefaultResource();
-
-        // then:
-        assertEquals(resource1, resource2);
-    }
-
-    @Test
-    public void twoDefaultInstancesHaveSameHashcode() {
-        // given:
-        final Resource resource1 = createDefaultResource();
-        final Resource resource2 = createDefaultResource();
-
-        // then:
-        assertEquals(resource1.hashCode(), resource2.hashCode());
-    }
-
     /**
      * Create a new instance of the resource.
      */

src/test/java/crud/core/ResourceTest.java

@@ -15,7 +15,6 @@
 package crud.pattern;
 
 import static crud.core.RxAssertions.assertObservablesEqual;
-import static org.junit.Assert.assertEquals;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
@@ -37,71 +36,7 @@ public abstract class ResourceMergerTest {
 
     @Before
     public void setup() {
-        when(this.mockReader.read()).thenReturn(mockReaderState);
-    }
-
-    @Test
-    public void twoDefaultMergersAreEqual() {
-        // given:
-        final ResourceMerger<Object> merger1 = createDefaultMerger(this.mockReader);
-        final ResourceMerger<Object> merger2 = createDefaultMerger(this.mockReader);
-
-        // then:
-        assertEquals(merger1, merger2);
-    }
-
-    @Test
-    public void hashCodesOfEqualMergersAreEqual() {
-        // given:
-        final ResourceMerger<Object> merger1 = createDefaultMerger(this.mockReader);
-        final ResourceMerger<Object> merger2 = createDefaultMerger(this.mockReader);
-
-        // then:
-        assertEquals(merger1.hashCode(), merger2.hashCode());
-    }
-
-    @Test
-    public void toStringResultsOfEqualMergersAreEqual() {
-        // given:
-        final ResourceMerger<Object> merger1 = createDefaultMerger(this.mockReader);
-        final ResourceMerger<Object> merger2 = createDefaultMerger(this.mockReader);
-
-        // then:
-        assertEquals(merger1.toString(), merger2.toString());
-    }
-
-    @Test
-    public void functionsOfEqualMergersAreEqual() {
-        // given:
-        final ResourceMerger<Object> merger1 = createDefaultMerger(this.mockReader);
-        final ResourceMerger<Object> merger2 = createDefaultMerger(this.mockReader);
-
-        // then:
-        assertEquals(merger1.toFunction(), merger2.toFunction());
-    }
-
-    @Test
-    public void functionsOfEqualMergersHaveEqualHashCodes() {
-        // given:
-        final ResourceMerger<Object> merger1 = createDefaultMerger(this.mockReader);
-        final ResourceMerger<Object> merger2 = createDefaultMerger(this.mockReader);
-        final Func0<Observable<Object>> function1 = merger1.toFunction();
-        final Func0<Observable<Object>> function2 = merger2.toFunction();
-
-        // then:
-        assertEquals(function1.hashCode(), function2.hashCode());
-    }
-
-    @Test
-    public void functionsOfEqualMergersHaveEqualStrings() {
-        // given:
-        final ResourceMerger<Object> merger1 = createDefaultMerger(this.mockReader);
-        final ResourceMerger<Object> merger2 = createDefaultMerger(this.mockReader);
-        final Func0<Observable<Object>> function1 = merger1.toFunction();
-        final Func0<Observable<Object>> function2 = merger2.toFunction();
-
-        // then:
-        assertEquals(function1.toString(), function2.toString());
+        when(this.mockReader.read()).thenReturn(this.mockReaderState);
     }
 
     @Test