feat: added Audit log and Savings Account class with test By Yemi

Author: jimoye244

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

@@ -12,8 +12,12 @@
  */
 public class AuditLog {
 
-    // Method to add a value to an existing ArrayList or create a new one if accountNumber doesn't exist
-    public void addToMap(HashMap<String, ArrayList<Double>> accountNumberByValueLog, String accountNumber, Double value) {
-        accountNumberByValueLog.computeIfAbsent(accountNumber, k -> new ArrayList<>()).add(value);
-    }
-}    
+  // Method to add a value to an existing ArrayList or create a new one if accountNumber doesn't
+  // exist
+  public void addToMap(
+      HashMap<String, ArrayList<Double>> accountNumberByValueLog,
+      String accountNumber,
+      Double value) {
+    accountNumberByValueLog.computeIfAbsent(accountNumber, k -> new ArrayList<>()).add(value);
+  }
+}

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

@@ -1,12 +1,10 @@
 package com.codedifferently.lesson17.bank;
 
+import com.codedifferently.lesson17.bank.exceptions.InsufficientFundsException;
 import java.util.Set;
-
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import com.codedifferently.lesson17.bank.exceptions.InsufficientFundsException;
-
 /** Represents a checking account. */
 public class BankAccount {
 
@@ -16,7 +14,6 @@ public class BankAccount {
   private boolean isActive;
   private static final Logger logger = LoggerFactory.getLogger(SavingAccount.class);
 
-
   /**
    * Creates a new checking account.
    *
@@ -82,7 +79,7 @@ public void withdraw(double amount) throws InsufficientFundsException {
       throw new IllegalStateException("Withdrawal amount must be positive");
     }
     if (balance < amount) {
-      logger.info("Account does not have enough funds for withdrawal"); 
+      logger.info("Account does not have enough funds for withdrawal");
       throw new InsufficientFundsException("Account does not have enough funds for withdrawal");
     }
     balance -= amount;
@@ -100,7 +97,7 @@ public double getBalance() {
   /** Closes the account. */
   public void closeAccount() throws IllegalStateException {
     if (balance > 0) {
-      logger.info("Cannot close account with a positive balance");  
+      logger.info("Cannot close account with a positive balance");
       throw new IllegalStateException("Cannot close account with a positive balance");
     }
     isActive = false;

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

@@ -1,7 +1,6 @@
 package com.codedifferently.lesson17.bank;
 
 import com.codedifferently.lesson17.bank.exceptions.AccountNotFoundException;
-
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Map;

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

@@ -3,9 +3,9 @@
 import java.util.Set;
 
 /** Represents a checking account. */
-public class CheckingAccount extends BankAccount{
+public class CheckingAccount extends BankAccount {
 
-   //private boolean isActive;
+  // private boolean isActive;
 
   /**
    * Creates a new checking account.
@@ -15,7 +15,7 @@ public class CheckingAccount extends BankAccount{
    * @param initialBalance The initial balance of the account.
    */
   public CheckingAccount(String accountNumber, Set<Customer> owners, double balance) {
-    super(accountNumber, owners,  balance);
-    //isActive = true;
+    super(accountNumber, owners, balance);
+    // isActive = true;
   }
 }

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

@@ -1,16 +1,14 @@
 package com.codedifferently.lesson17.bank;
 
-
 import java.util.Set;
-
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 /** Represents a saving account. */
-public class SavingAccount extends BankAccount{
-   //private boolean isActive;
-   private boolean isCheckCreationAllowed;
-   private static final Logger logger = LoggerFactory.getLogger(SavingAccount.class);
+public class SavingAccount extends BankAccount {
+  // private boolean isActive;
+  private boolean isCheckCreationAllowed;
+  private static final Logger logger = LoggerFactory.getLogger(SavingAccount.class);
 
   /**
    * Creates a new saving account.
@@ -20,13 +18,12 @@ public class SavingAccount extends BankAccount{
    * @param initialBalance The initial balance of the account.
    */
   public SavingAccount(String accountNumber, Set<Customer> owners, double balance) {
-    super(accountNumber, owners,  balance);
+    super(accountNumber, owners, balance);
     logger.info("Saving Account constructor accessed...");
     isCheckCreationAllowed = false;
   }
 
   public boolean isCheckCreationAllowed() {
-      return isCheckCreationAllowed;
+    return isCheckCreationAllowed;
   }
-
 }

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

@@ -1,17 +1,16 @@
 package com.codedifferently.lesson17.bank;
 
-import java.util.HashSet;
-import java.util.Set;
-import java.util.UUID;
-
 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 org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Test;
 
 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 SavingAccountTest {
 
@@ -106,13 +105,14 @@ void toStringTest() {
   }
 
   @Test
-    public void testIsCheckCreationAllowed_WhenFalse() {
-        // Arrange: 
+  public void testIsCheckCreationAllowed_WhenFalse() {
+    // Arrange:
 
-        // Act: 
-        boolean result = classUnderTest.isCheckCreationAllowed();
+    // Act:
+    boolean result = classUnderTest.isCheckCreationAllowed();
 
-        // Assert: 
-        assertFalse(result, "Check creation should not be allowed when isCheckCreationAllowed is false");
-    }
+    // Assert:
+    assertFalse(
+        result, "Check creation should not be allowed when isCheckCreationAllowed is false");
+  }
 }