feat: added tests to Organization classes

Group-5 · Mar 24, 2026 · 33484e3 · 33484e3
1 parent 27aff0f
commit 33484e3
Show file tree

Hide file tree

Showing 3 changed files with 44 additions and 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
@@ -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
@@ -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
@@ -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
     ));
   }