[#3] Modify test to use $50 and pass in real augend&addends

EthanLin-TWer · Jan 14, 2017 · a42819f · a42819f
1 parent 5e8131d
commit a42819f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/src/main/java/org/thoughtworks/linesh/multicurrencymoney/Money.java b/src/main/java/org/thoughtworks/linesh/multicurrencymoney/Money.java
@@ -42,7 +42,7 @@ public Money times(int multiplier) {
 
 
     @Override
-    public Expression plus(Money money) {
-        return new Sum(Money.dollar(5), Money.dollar(5));
+    public Expression plus(Money addend) {
+        return new Sum(this, addend);
     }
 }
diff --git a/src/test/java/org/thoughtworks/linesh/multicurrencymoney/MoneyTest.java b/src/test/java/org/thoughtworks/linesh/multicurrencymoney/MoneyTest.java
@@ -4,8 +4,6 @@
 import org.junit.jupiter.api.Test;
 import support.Tasking;
 
-import java.lang.reflect.Field;
-
 import static org.junit.jupiter.api.Assertions.*;
 
 class MoneyTest {
@@ -62,12 +60,12 @@ void should_get_10_dollars_when_adding_5_dollars_with_5_dollars() {
 
         @Test
         void should_test_our_assumptions_here() {
-            Money fiveDollars = Money.dollar(5);
-            Expression result = fiveDollars.plus(fiveDollars);
+            Money fiftyDollars = Money.dollar(50);
+            Expression result = fiftyDollars.plus(fiftyDollars);
             Sum sum = (Sum) result;
 
-            assertEquals(fiveDollars, sum.augend);
-            assertEquals(fiveDollars, sum.addend);
+            assertEquals(fiftyDollars, sum.augend);
+            assertEquals(fiftyDollars, sum.addend);
         }
     }
 }