feat: adds SavingsAccount Tests

Author: TatsuyaRyujin

lesson_17/bank/bank_app/src/main/java/com/codedifferently/lesson17/bank/BusinessCheckingAccount.java

@@ -0,0 +1,32 @@
+package com.codedifferently.lesson17.bank;
+
+import java.util.Set;
+
+/** Represents a business checking account */
+public class BusinessCheckingAccount extends CheckingAccount {
+
+  /**
+   * Creates a new business checking account
+   *
+   * @param accountNumber The accounnt number.
+   * @param owners The owners of the account.
+   * @param initialBalance The initial balance of the account.
+   */
+  public BusinessCheckingAccount(
+      String accountNumber, Set<Customer> owners, double initialBalance) {
+    super(accountNumber, owners, initialBalance);
+  }
+
+  @Override
+  public String toString() {
+    return "BusinessCheckingAccount{"
+        + "accountNumber='"
+        + getAccountNumber()
+        + '\''
+        + ", balance="
+        + getBalance()
+        + ", isActive="
+        + isActive()
+        + '}';
+  }
+}

lesson_17/bank/bank_app/src/main/java/com/codedifferently/lesson17/bank/Check.java

@@ -7,7 +7,7 @@ public class Check {
 
   private final String checkNumber;
   private final double amount;
-  private final Account account;
+  private final CheckingAccount account;
   private boolean isVoided = false;
 
   /**
@@ -17,7 +17,7 @@ public class Check {
    * @param amount The amount of the check.
    * @param account The account the check is drawn on.
    */
-  public Check(String checkNumber, double amount, Account account) {
+  public Check(String checkNumber, double amount, CheckingAccount account) {
     if (amount < 0) {
       throw new IllegalArgumentException("Check amount must be positive");
     }

lesson_17/bank/bank_app/src/main/java/com/codedifferently/lesson17/bank/Customer.java

@@ -19,7 +19,7 @@ public class Customer {
   /**
    * Creates a new customer.
    *
-   * @param id   The ID of the customer.
+   * @param id The ID of the customer.
    * @param name The name of the customer.
    * @param type The type of customer (individual or business)
    */
@@ -32,11 +32,11 @@ public Customer(UUID id, String name, CustomerType type) {
   /**
    * Creates an new individual customer.
    *
-   * @param id   The ID of the customer.
+   * @param id The ID of the customer.
    * @param name The name of the customer.
    * @param type The type of custimer (individual or business)
    */
-  public Customer(UUID id, String name, Customer type) {
+  public Customer(UUID id, String name) {
     this.id = id;
     this.name = name;
     this.type = CustomerType.INDIVIDUAL;

lesson_17/bank/bank_app/src/test/java/com/codedifferently/lesson17/bank/BankAtmTest.java

@@ -43,14 +43,14 @@ void testAddAccount() {
     classUnderTest.addAccount(account3);
 
     // Assert
-    Set<CheckingAccount> accounts = classUnderTest.findAccountsByCustomerId(customer3.getId());
+    Set<Account> accounts = classUnderTest.findAccountsByCustomerId(customer3.getId());
     assertThat(accounts).containsOnly(account3);
   }
 
   @Test
   void testFindAccountsByCustomerId() {
     // Act
-    Set<CheckingAccount> accounts = classUnderTest.findAccountsByCustomerId(customer1.getId());
+    Set<Account> accounts = classUnderTest.findAccountsByCustomerId(customer1.getId());
 
     // Assert
     assertThat(accounts).containsOnly(account1, account2);

lesson_17/bank/bank_app/src/test/java/com/codedifferently/lesson17/bank/SavingAccountTest.java

@@ -1,3 +1,67 @@
 package com.codedifferently.lesson17.bank;
 
-class CheckingAccountTest {}
+import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+import com.codedifferently.lesson17.bank.exceptions.InsufficientFundsException;
+import java.util.HashSet;
+import java.util.Set;
+import java.util.UUID;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+class SavingsAccountTest {
+
+  private SavingsAccount account;
+  private Set<Customer> owners;
+
+  @BeforeEach
+  void setUp() {
+    owners = new HashSet<>();
+    owners.add(new Customer(UUID.randomUUID(), "Yuji Nishida"));
+    owners.add(new Customer(UUID.randomUUID(), "Masahiro Sekita"));
+
+    account = new SavingsAccount("123456789", owners, 100.0);
+  }
+
+  @Test
+  void deposit() {
+    account.deposit(100.0);
+    assertEquals(200.0, account.getBalance());
+  }
+
+  @Test
+  void withdraw() {
+    account.withdraw(25.0);
+    assertEquals(75.0, account.getBalance());
+  }
+
+  @Test
+  void withdraw_withNegativeAmount() {
+    assertThatExceptionOfType(IllegalStateException.class)
+        .isThrownBy(() -> account.withdraw(-25.0))
+        .withMessage("Withdrawal amount must be positive");
+  }
+
+  @Test
+  void withdraw_withInsufficientBalance() {
+    assertThatExceptionOfType(InsufficientFundsException.class)
+        .isThrownBy(() -> account.withdraw(200.0))
+        .withMessage("Account does not have enough funds for withdrawal");
+  }
+
+  @Test
+  void closeAccount_withPositiveBalance() {
+    assertThatExceptionOfType(IllegalStateException.class).isThrownBy(() -> account.closeAccount());
+  }
+
+  @Test
+  void isClosed() {
+    assertFalse(account.isClosed());
+    account.withdraw(100.0);
+    account.closeAccount();
+    assertTrue(account.isClosed());
+  }
+}