From 33484e3609cf2ed2215fea9b13d4f94d58380acd Mon Sep 17 00:00:00 2001
From: MatheaGjerde <mathea_10_@hotmail.com>
Date: Tue, 24 Mar 2026 19:28:49 +0100
Subject: [PATCH] feat: added tests to Organization classes

---
 .../OrganizationRepositoryTest.java           |  6 ++--
 .../organization/OrganizationServiceTest.java | 15 +++++++++
 .../model/organization/OrganizationTest.java  | 32 +++++++++++++++----
 3 files changed, 44 insertions(+), 9 deletions(-)

diff --git a/src/test/java/edu/group5/app/model/organization/OrganizationRepositoryTest.java b/src/test/java/edu/group5/app/model/organization/OrganizationRepositoryTest.java
index 7a5ece5..f821e35 100644
--- a/src/test/java/edu/group5/app/model/organization/OrganizationRepositoryTest.java
+++ b/src/test/java/edu/group5/app/model/organization/OrganizationRepositoryTest.java
@@ -73,7 +73,7 @@ void getTrustedOrganizations_OnlyReturnsTrustedOrganizations() {
   @Test
   void testFindByOrgNumberReturnsOrganization() {
     assertEquals(new Organization(1, "Trusted Org1", true,
-    "org.com", true, "Information about Trusted Org1"), 
+    "org.com", true, "Information about Trusted Org1", null),
     repository.findByOrgNumber(1));
   }
 
@@ -93,7 +93,7 @@ void testFindByOrgNumberIfOrgNumberNotFound() {
   @Test
   void testFindByOrgNameReturnsOrganization() {
     assertEquals(new Organization(1, "Trusted Org1", true,
-    "org.com", true, "Information about Trusted Org1"), 
+    "org.com", true, "Information about Trusted Org1", null),
     repository.findByOrgName("Trusted Org1"));
   }
 
@@ -116,7 +116,7 @@ void testFindByOrgNameIfNameNotFound() {
   @Test
   void testFindByOrgNameIsCaseInsensitive() {
     assertEquals(new Organization(1, "Trusted Org1", true,
-    "org.com", true, "Information about Trusted Org1"), 
+    "org.com", true, "Information about Trusted Org1", null),
     repository.findByOrgName("trusted org1"));
   }
 
diff --git a/src/test/java/edu/group5/app/model/organization/OrganizationServiceTest.java b/src/test/java/edu/group5/app/model/organization/OrganizationServiceTest.java
index e34aba7..0920e67 100644
--- a/src/test/java/edu/group5/app/model/organization/OrganizationServiceTest.java
+++ b/src/test/java/edu/group5/app/model/organization/OrganizationServiceTest.java
@@ -67,4 +67,19 @@ void testFindByOrgName() {
     assertEquals(1, org.orgNumber());
     assertEquals("Misjonsalliansen", org.name());
   }
+
+  @Test
+  void fetchLogoUrlReturnsNullWhenUrlIsNull() {
+    assertNull(service.fetchLogoUrl(null));
+  }
+  @Test
+  void fetchLogoUrlReturnsNullWhenUrlIsBlank() {
+    assertNull(service.fetchLogoUrl(""));
+  }
+  @Test
+  void fetchLogoUrlCachesResultOnSecondCall() {
+    String result1 = service.fetchLogoUrl("https://");
+    String result2 = service.fetchLogoUrl("https://");
+    assertEquals(result1, result2);
+  }
 }
diff --git a/src/test/java/edu/group5/app/model/organization/OrganizationTest.java b/src/test/java/edu/group5/app/model/organization/OrganizationTest.java
index f921b60..0b97840 100644
--- a/src/test/java/edu/group5/app/model/organization/OrganizationTest.java
+++ b/src/test/java/edu/group5/app/model/organization/OrganizationTest.java
@@ -14,7 +14,8 @@ void constructor_CreatesAnOrganizationWhenInputIsValid() {
         true,
         "org.com",
         true,
-        "Org description"
+        "Org description",
+        null
     );
 
     assertAll(
@@ -35,7 +36,8 @@ void constructor_ThrowsWhenOrgNumberIsNegative() {
         true,
         "org.com",
         true,
-        "Org description"
+        "Org description",
+        null
     ));
   }
 
@@ -47,7 +49,8 @@ void constructor_ThrowsWhenNameIsNull() {
         true,
         "org.com",
         true,
-        "Org description"
+        "Org description",
+        null
     ));
   }
 
@@ -59,7 +62,8 @@ void constructor_ThrowsWhenNameIsBlank() {
         true,
         "org.com",
         true,
-        "Org description"
+        "Org description",
+        null
     ));
   }
 
@@ -71,7 +75,8 @@ void constructor_ThrowsWhenWebsiteURLIsNull() {
         true,
         null,
         true,
-        "Org description"
+        "Org description",
+        null
     ));
   }
 
@@ -83,7 +88,8 @@ void constructor_ThrowsWhenWebsiteURLIsBlank() {
         true,
         "",
         true,
-        "Org description"
+        "Org description",
+        null
     ));
   }
 
@@ -95,6 +101,20 @@ void constructor_ThrowsWhenDescriptionIsNull() {
         true,
         "org.com",
         true,
+        null,
+        null
+    ));
+  }
+
+  @Test
+  void constructor_AcceptsNullLogoUrl() {
+    assertDoesNotThrow(() -> new Organization(
+        1,
+        "Org",
+        true,
+        "org.com",
+        true,
+        "description",
         null
     ));
   }