Update[JUnit]: add more JUnit tests for better test coverage

src/main/java/edu/group5/app/model/AppState.java

@@ -18,7 +18,7 @@ public class AppState {
   private User currentUser;
   private BigDecimal currentDonationAmount;
   private Organization currentOrganization;
-  private String currentDonation;
+  private String currentPaymentMethod;
 
   /**
    * Gets the current user of the application.
@@ -73,14 +73,14 @@ public void setCurrentDonationAmount(BigDecimal amount) {
    * @return the current payment method
    */
   public String getCurrentPaymentMethod() {
-    return this.currentDonation;
+    return this.currentPaymentMethod;
   }
 
   /**
    * Sets the current payment method.
    * @param paymentMethod the payment method to set as the current payment method
    */
   public void setCurrentPaymentMethod(String paymentMethod){
-    this.currentDonation = paymentMethod;
+    this.currentPaymentMethod = paymentMethod;
   }
 }

src/test/java/edu/group5/app/model/AppStateTest.java

@@ -0,0 +1,69 @@
+package edu.group5.app.model;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import java.math.BigDecimal;
+
+import org.junit.jupiter.api.Assertions.*;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+
+import edu.group5.app.model.organization.Organization;
+import edu.group5.app.model.user.Customer;
+import edu.group5.app.model.user.User;
+
+public class AppStateTest {
+  private User nullUser;
+  private BigDecimal nullDonationAmount;
+  private Organization nullOrganization;
+  private String nullPaymentMethod;
+
+  private User currentTestUser;
+  private BigDecimal currentTestDonationAmount;
+  private Organization currentTestOrganization;
+  private String currentTestPaymentMethod;
+
+  @BeforeEach
+  void setUp() {
+    nullUser = null;
+    nullDonationAmount = null;
+    nullOrganization = null;
+    nullPaymentMethod = null;
+
+    currentTestUser = new Customer(1, "Bob", 
+    "Builder", "testuser@example.com", 
+    "password123");
+
+    currentTestDonationAmount = BigDecimal.ZERO;
+
+    currentTestOrganization = new Organization(1738, "TestOrg", 
+    true, "https://testorg.example.com", true, 
+    "A test organization", "https://testorg.example.com/logo.png");
+
+    currentTestPaymentMethod = "Credit Card";
+  }
+
+  @Test
+  void gettersAndSetters_WorkCorrectly() {
+    AppState appState = new AppState();
+
+    // Test current user
+    assertEquals(nullUser, appState.getCurrentUser());
+    appState.setCurrentUser(currentTestUser);
+    assertEquals(currentTestUser, appState.getCurrentUser());
+
+    // Test current donation amount
+    assertEquals(nullDonationAmount, appState.getCurrentDonationAmount());
+    appState.setCurrentDonationAmount(currentTestDonationAmount);
+    assertEquals(currentTestDonationAmount, appState.getCurrentDonationAmount());
+
+    // Test current organization
+    assertEquals(nullOrganization, appState.getCurrentOrganization());
+    appState.setCurrentOrganization(currentTestOrganization);
+    assertEquals(currentTestOrganization, appState.getCurrentOrganization());
+
+    // Test current payment method
+    assertEquals(nullPaymentMethod, appState.getCurrentPaymentMethod());
+    appState.setCurrentPaymentMethod(currentTestPaymentMethod);
+    assertEquals(currentTestPaymentMethod, appState.getCurrentPaymentMethod());
+  }
+}

src/test/java/edu/group5/app/model/donation/DonationServiceTest.java

@@ -12,7 +12,7 @@
 import java.time.Instant;
 import java.util.ArrayList;
 import java.util.HashMap;
-
+import java.util.Map;
 
 import static org.junit.jupiter.api.Assertions.*;
 
@@ -61,6 +61,33 @@ void testConstructorThrowsIfOrganizationRepositoryIsNull() {
     assertEquals("OrganizationRepository can't be null", exception.getMessage());
   }
 
+  @Test
+  void getUserDonationsReturnsEmptyMapIfNoDonations() {
+    assertTrue(donationService.getUserDonations(customer.getUserId()).isEmpty());
+  }
+
+  @Test
+  void getOrganizationDonationsReturnsMapOfDonations() {
+    Donation donation1 = new Donation(1, customer.getUserId(), 
+    101, new BigDecimal("20.00"), Timestamp.from(Instant.now()), "Card");
+    Donation donation2 = new Donation(2, customer.getUserId(), 
+    101, new BigDecimal("30.00"), Timestamp.from(Instant.now()), "PayPal");
+    donationRepository.addContent(donation1);
+    donationRepository.addContent(donation2);
+
+    Map<Integer, Donation> donations = donationService.getOrganizationDonations(101);
+    assertEquals(2, donations.size());
+    assertTrue(donations.containsKey(1));
+    assertTrue(donations.containsKey(2));
+    assertEquals(donation1, donations.get(1));
+    assertEquals(donation2, donations.get(2));
+  }
+
+  @Test
+  void getOrganizationDonationsReturnsEmptyMapIfNoDonations() {
+    assertTrue(donationService.getOrganizationDonations(1).isEmpty());
+  }
+
   @Test
   void donateReturnsFalseIfCustomerNull() {
     boolean result = donationService.donate(null,

src/test/java/edu/group5/app/model/donation/DonationTest.java

@@ -74,11 +74,19 @@ void testIfThrowsExceptionWhenOrganizationIdIsNotPositive() {
         amount1, date1, paymentMethod1);
   }
   @Test
-  void testIfThrowsExceptionWhenAmountIsNotPositive() {
+  void testIfThrowsExceptionWhenAmountIsNegative() {
     expectedMessage = "Amount must be positive and not null";
     exceptionTest(donationId1, userId1, organizationId1,
-        new BigDecimal("0.00"), date1, paymentMethod1);
+        new BigDecimal("-1.00"), date1, paymentMethod1);
   }
+
+  @Test
+  void testIfThrowsExceptionWhenAmountIsNull() {
+    expectedMessage = "Amount must be positive and not null";
+    exceptionTest(donationId1, userId1, organizationId1,
+        null, date1, paymentMethod1);
+  }
+
   @Test
   void testIfThrowsExceptionWhenDateIsNull() {
     expectedMessage = "Date must not be null";

src/test/java/edu/group5/app/model/organization/OrganizationRepositoryTest.java

@@ -60,6 +60,30 @@ void constructor_ThrowsWhenContentIsNull() {
     constructorTest(null, "Input data can't be null");
   }
 
+  @Test
+  void constructor_SkipsOrganizationWithMissingOrgNumber() {
+    Object[] content = new Object[] {
+        Map.of(
+            "name", "Good Org",
+            "status", "approved",
+            "url", "org.com",
+            "is_pre_approved", true
+        ),
+        Map.of(
+            "org_number", "999",
+            "name", "Bad Org",
+            "status", "approved",
+            "url", "org.com",
+            "is_pre_approved", true
+        )
+    };
+
+    OrganizationRepository repo = new OrganizationRepository(content, scraper);
+
+    assertEquals(1, repo.findByOrgNumber(999) != null ? 1 : 0);
+    assertNull(repo.findByOrgNumber(1));
+  }
+
   @Test
   void constructor_ThrowsWhenScraperIsNull() {
     Object[] content = new Object[] {

src/test/java/edu/group5/app/model/organization/OrganizationScraperTest.java

@@ -63,4 +63,17 @@ void fetchLogoUrl_CachesResultOnSecondCall() {
     // If no exception thrown, cache works
     assertTrue(true);
   }
+
+  @Test
+  void fetchDescription_ReturnsCachedValue() {
+    OrganizationScraper scraper = new OrganizationScraper();
+
+    // First call - caches (but makes real request)
+    String result1 = scraper.fetchDescription("https://example.com");
+
+    // Second call - should return cached without new request
+    String result2 = scraper.fetchDescription("https://example.com");
+
+    assertEquals(result1, result2); // Same object = cached
+  }
 }