Multi Currency Money [springframeworkguru#20. Privacy. Currenthly the…

… 'amount' property is public, only expose what you need]
gchulim · May 5, 2023 · 928b2d3 · 928b2d3
1 parent 4ba1ffa
commit 928b2d3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/test/java/guru/springframework/Dollar.java b/src/test/java/guru/springframework/Dollar.java
@@ -1,7 +1,7 @@
 package guru.springframework;
 
 public class Dollar {
-    int amount;
+    private int amount;
     public Dollar(int amount) {
         this.amount = amount;
     }

diff --git a/src/test/java/guru/springframework/MoneyTest.java b/src/test/java/guru/springframework/MoneyTest.java
@@ -10,9 +10,9 @@ public class MoneyTest {
     void testMultiplication() {
         Dollar five = new Dollar(5);
         Dollar product = five.times(2);
-        assertEquals(10, product.amount);
-        product =five.times(3);
-        assertEquals(15, product.amount);
+        assertEquals(new Dollar(10), product);
+        product = five.times(3);
+        assertEquals(new Dollar(15), product);
     }
 
     @Test