Merge pull request #57 from Group-5/fix/failingtests

Merge fix/failingtests into release/v1.0.0
Group-5 · Mar 20, 2026 · 1a99990 · 1a99990
2 parents ae861cb + d60c6d5
commit 1a99990
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 9 deletions.
diff --git a/src/test/java/edu/group5/app/model/user/UserRepositoryTest.java b/src/test/java/edu/group5/app/model/user/UserRepositoryTest.java
@@ -139,15 +139,6 @@ void getNextUserIdReturns1IfEmpty() {
     assertEquals(1, emptyRepo.getNextUserId());
   }
 
-  @Test
-  void exportContainsAllUsers() {
-    List<Object[]> exported = repo.export();
-    assertEquals(2, exported.size());
-    assertEquals(1, exported.get(0)[0]);
-    assertEquals(2, exported.get(1)[0]);
-    assertEquals("Customer", exported.get(0)[1]);
-  }
-
   @Test
   void exportExportsOnlyNewInformation() {
     assertEquals(0, repo.export().size());

diff --git a/src/test/java/edu/group5/app/model/user/UserServiceTest.java b/src/test/java/edu/group5/app/model/user/UserServiceTest.java
@@ -149,4 +149,20 @@ void loginInvalidEmail() {
     assertNull(result2);
     assertNull(result3);
   }
+
+  @Test
+  void getUserByEmailValid() {
+    User result = service.getUserByEmail("jane.doe@example.com");
+    assertEquals("Jane", result.getFirstName());
+  }
+
+  @Test
+  void getUserByEmailNotFound() {
+    User result = service.getUserByEmail("");
+    User result2 = service.getUserByEmail(null);
+    User result3 = service.getUserByEmail("   ");
+    assertNull(result);
+    assertNull(result2);
+    assertNull(result3);
+  }
 }