Fix: Modified tests to closer fit the needs

Author: Mohamed

lesson_16/api/api_app/src/test/java/com/codedifferently/lesson16/web/PatronControllerTest.java

@@ -54,7 +54,7 @@ void testController_getsAllPatrons() throws Exception {
         .perform(get("/patrons").contentType(MediaType.APPLICATION_JSON))
         .andExpect(status().isOk())
         .andExpect(jsonPath("$.patrons").isArray())
-        .andExpect(jsonPath("$.patrons.length()").value(5));
+        .andExpect(jsonPath("$.patrons.length()").value(4));
   }
 
   @Test
@@ -139,14 +139,10 @@ void testController_deletesPatron() throws Exception {
   }
 
   UUID getGuestId(List<LibraryGuest> list) {
-    if (list.get(0).getCheckedOutMediaItems().size() == 0) {
-      return list.get(0).getId();
-    } else if (list.get(1).getCheckedOutMediaItems().size() == 0) {
-      return list.get(1).getId();
-    } else if (list.get(2).getCheckedOutMediaItems().size() == 0) {
-      return list.get(2).getId();
-    } else if (list.get(3).getCheckedOutMediaItems().size() == 0) {
-      return list.get(3).getId();
+    for (LibraryGuest guest : list) {
+      if (guest.getCheckedOutMediaItems().size() == 0) {
+        return guest.getId();
+      }
     }
     return list.get(4).getId();
   }