Multi Currency Money [springframeworkguru#23. Compare Francs with Dol…

…lars]
gchulim · May 5, 2023 · bf39b59 · bf39b59
1 parent e2a1056
commit bf39b59
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/test/java/guru/springframework/Money.java b/src/test/java/guru/springframework/Money.java
@@ -5,7 +5,8 @@ public class Money {
 
     public boolean equals(Object o) {
         Money money = (Money) o;
-        return amount == money.amount;
+        return amount == money.amount
+                && this.getClass() == money.getClass();
     }
 
 

diff --git a/src/test/java/guru/springframework/MoneyTest.java b/src/test/java/guru/springframework/MoneyTest.java
@@ -17,8 +17,9 @@ void testMultiplicationDollar() {
 
     @Test
     void testEqualityDollar() {
-        assertEquals(new Dollar(6), new Dollar(6));
-        assertNotEquals(new Dollar(6), new Dollar(8));
+        assertEquals(new Dollar(5), new Dollar(5));
+        assertNotEquals(new Dollar(5), new Dollar(8));
+        assertNotEquals(new Dollar(5), new Franc(5));
     }
 
     @Test
@@ -32,7 +33,8 @@ void testMultiplicationFranc() {
 
     @Test
     void testEqualityFranc() {
-        assertEquals(new Franc(6), new Franc(6));
-        assertNotEquals(new Franc(6), new Franc(8));
+        assertEquals(new Franc(5), new Franc(5));
+        assertNotEquals(new Franc(5), new Franc(8));
+        assertNotEquals(new Franc(5), new Dollar(5));
     }
 }