Acceptance test passing, still to test DateCreator and ProConsole

marabesi · Apr 4, 2023 · 232b639 · 232b639
1 parent 7c168b8
commit 232b639
Show file tree

Hide file tree

Showing 7 changed files with 113 additions and 17 deletions.
diff --git a/Bank/Account.cs b/Bank/Account.cs
@@ -5,6 +5,7 @@ public class Account
     private ProConsole proConsole;
     private DateCreator dateCreator;
     private WalletRepository wallet;
+    private List<Movement> movements = new();
 
     public Account(ProConsole proConsole, DateCreator dateCreator, WalletRepository wallet)
     {
@@ -15,16 +16,36 @@ public Account(ProConsole proConsole, DateCreator dateCreator, WalletRepository
 
     public void Deposit(int amount)
     {
-        wallet.add(1000);
+        wallet.add(amount);
+        AddMovement(amount);
     }
 
     public void Withdraw(int amount)
     {
-        throw new NotImplementedException();
+        wallet.remove(amount);
+        AddMovement(amount*-1);
     }
 
     public void PrintStatement()
     {
-        throw new NotImplementedException();
+        proConsole.printLine("Date       || Amount || Balance");
+
+        for(int i=movements.Count-1; i>=0; i--)
+        {
+            proConsole.printLine(FormatLine(i));
+        }
+
+    }
+
+    private void AddMovement(int amount)
+    {
+        movements.Add(new Movement(dateCreator.CreateCurrentDate(), amount, wallet.getBalance()));
+    }
+
+    private string FormatLine(int index)
+    {
+        return movements[index].date.ToString("dd/MM/yyyy") + " || " +
+               movements[index].operationAmount + " || " +
+               movements[index].remainingBalance;
     }
 }
diff --git a/Bank/Movement.cs b/Bank/Movement.cs
@@ -0,0 +1,15 @@
+namespace Bank;
+
+public class Movement
+{
+    public Movement(DateTime date, int operationAmount, int remainingBalance)
+    {
+        this.date = date;
+        this.operationAmount = operationAmount;
+        this.remainingBalance = remainingBalance;
+    }
+
+    public DateTime date {get; }
+    public int operationAmount {get; }
+    public int remainingBalance {get; }
+}
diff --git a/Bank/Wallet.cs b/Bank/Wallet.cs
@@ -2,8 +2,20 @@
 
 public class Wallet : WalletRepository
 {
+    private int balance = 0;
+
     public void add(int amount)
     {
-        throw new NotImplementedException("NOT IMPLEMENTED: Wallet repository not implemented");
+        balance += amount;
+    }
+
+    public void remove(int amount)
+    {
+        balance -= amount;
+    }
+
+    public int getBalance()
+    {
+        return balance;
     }
 }
diff --git a/Bank/WalletRepository.cs b/Bank/WalletRepository.cs
@@ -3,4 +3,6 @@
 public interface WalletRepository
 {
     public void add(int amount);
+    void remove(int amount);
+    public int getBalance();
 }
diff --git a/BankTests/AccountShould.cs b/BankTests/AccountShould.cs
@@ -1,23 +1,29 @@
 using Bank;
 using Moq;
 
-namespace AccountShould;
+namespace BankUnitTests;
 
 public class AccountShould
 {
     private Account account;
 
     private Mock<WalletRepository> walletRepository = new();
-
+    private Mock<DateCreator> dateMocker = new();
+
 
     public AccountShould()
     {
-        account = new Account(new ProConsole(), new DateCreator(), walletRepository.Object);
+        account = new Account(new ProConsole(), dateMocker.Object, walletRepository.Object);
     }
 
-    [Fact]
-    public void DepositTheGivenAmountAndSaveTheValue() {
-        account.Deposit(1000);
-        walletRepository.Verify(wallet => wallet.add(1000));
+    [Theory]
+    [InlineData(1000)]
+    [InlineData(2000)]
+    public void DepositTheGivenAmountAndSaveTheValue(int depositAmount) {
+        dateMocker.Setup(x => x.CreateCurrentDate())
+            .Returns(DateTime.Parse("01/01/2000"));
+
+        account.Deposit(depositAmount);
+        walletRepository.Verify(wallet => wallet.add(depositAmount));
     }
 }
diff --git a/BankTests/BankAcceptanceShould.cs b/BankTests/BankAcceptanceShould.cs
@@ -1,7 +1,7 @@
 using Bank;
 using Moq;
 
-namespace BankTests;
+namespace BankAcceptanceTests;
 
 public class BankAcceptanceShould
 {
@@ -26,9 +26,9 @@ public void PerformPrintAListOfPreviousDepositsAndWithdrawals()
         // given
         var account = new Account(proConsoleMock, dateCreatorMock, walletInMemoryRepo);
         dateMocker.SetupSequence(x => x.CreateCurrentDate())
-            .Returns(DateTime.Parse("14/01/2012"))
+            .Returns(DateTime.Parse("10/01/2012"))
             .Returns(DateTime.Parse("13/01/2012"))
-            .Returns(DateTime.Parse("10/01/2012"));
+            .Returns(DateTime.Parse("14/01/2012"));
 
         // when
         account.Deposit(1000);
@@ -40,9 +40,9 @@ public void PerformPrintAListOfPreviousDepositsAndWithdrawals()
 
         // It could also check a bunch of lines if the console has a method for stack of transactions
         consoleMocker.Verify(console => console.printLine("Date       || Amount || Balance"));
-        consoleMocker.Verify(console => console.printLine("14/01/2012 || -500   || 2500"));
-        consoleMocker.Verify(console => console.printLine("13/01/2012 || 2000   || 3000"));
-        consoleMocker.Verify(console => console.printLine("10/01/2012 || 1000   || 1000"));
+        consoleMocker.Verify(console => console.printLine("14/01/2012 || -500 || 2500"));
+        consoleMocker.Verify(console => console.printLine("13/01/2012 || 2000 || 3000"));
+        consoleMocker.Verify(console => console.printLine("10/01/2012 || 1000 || 1000"));
 
     }
 }

diff --git a/BankTests/WalletShould.cs b/BankTests/WalletShould.cs
@@ -0,0 +1,40 @@
+using Bank;
+
+namespace BankUnitTests;
+
+public class WalletShould
+{
+    private Wallet wallet;
+
+    public WalletShould()
+    {
+        wallet = new();
+    }
+
+    [Fact]
+    public void beInitializedWithABalanceOf0()
+    {
+        Assert.Equal(0, wallet.getBalance());
+    }
+
+    [Theory]
+    [InlineData(1000)]
+    [InlineData(2000)]
+    public void AddGivenAmountToTheBalanceWhenADepositIsMade(int depositAmount)
+    {
+        wallet.add(depositAmount);
+
+        Assert.Equal(depositAmount, wallet.getBalance());
+    }
+
+    [Theory]
+    [InlineData(10000, 1000, 9000)]
+    [InlineData(10000, 2000, 8000)]
+    public void RemoveGivenAmountToTheBalanceWhenAWithdrawIsMade(int startingAmount, int withdrawAmount, int expectedAmount)
+    {
+        wallet.add(startingAmount);
+        wallet.remove(withdrawAmount);
+
+        Assert.Equal(expectedAmount, wallet.getBalance());
+    }
+}