Merge pull request #103 from ChenCMD/master

Implement hashCode() in ArrayRecord and Record classes

modules/core/src/main/scala/com/github/tarao/record4s/ArrayRecord.scala

@@ -68,6 +68,8 @@ abstract class ProductRecord extends Record with Product {
       case _ =>
         false
     }
+
+  override def hashCode(): Int = __fields.hashCode()
 }
 
 object ProductRecord {

modules/core/src/main/scala/com/github/tarao/record4s/Record.scala

@@ -387,6 +387,8 @@ abstract class % extends Record with Selectable {
       case _ =>
         false
     }
+
+  override def hashCode(): Int = __iterable.hashCode()
 }
 
 val % = new Record.Extensible(Record.empty)

modules/core/src/test/scala/com/github/tarao/record4s/ArrayRecordSpec.scala

@@ -913,6 +913,18 @@ class ArrayRecordSpec extends helper.UnitSpec {
       }
     }
 
+    describe(".hashCode()") {
+      it("should return the same hash code for equal records") {
+        val r1 = ArrayRecord(name = "tarao", age = 3)
+        val r2 = ArrayRecord(name = "tarao", age = 3)
+        val r3 = ArrayRecord(name = "ikura", age = 1)
+
+        (r1.hashCode() == r1.hashCode()) shouldBe true
+        (r1.hashCode() == r2.hashCode()) shouldBe true
+        (r2.hashCode() == r1.hashCode()) shouldBe true
+      }
+    }
+
     describe(".toString()") {
       it("can express empty Record") {
         ArrayRecord.empty.toString() shouldBe "ArrayRecord()"

modules/core/src/test/scala/com/github/tarao/record4s/RecordSpec.scala

@@ -807,6 +807,22 @@ class RecordSpec extends helper.UnitSpec {
       }
     }
 
+    describe(".hashCode()") {
+      it("should return the same hash code for equal records") {
+        val r1 = %(name = "tarao", age = 3)
+        val r2 = %(name = "tarao", age = 3)
+        val r3 = %(name = "ikura", age = 1)
+        val r4 = %(name = "tarao", age = 3, email = "[email protected]")
+        val r5: % { val name: String; val age: Int } = r4
+
+        (r1.hashCode() == r1.hashCode()) shouldBe true
+        (r1.hashCode() == r2.hashCode()) shouldBe true
+        (r2.hashCode() == r1.hashCode()) shouldBe true
+        (r1.hashCode() == r5.as.hashCode()) shouldBe true
+        (r5.as.hashCode() == r1.hashCode()) shouldBe true
+      }
+    }
+
     describe(".toString()") {
       it("can express empty Record") {
         Record.empty.toString() shouldBe "%()"