Fixes

Author: thelenw

server/src/test/java/com/objectcomputing/checkins/services/reviews/ReviewAssignmentControllerTest.java

@@ -148,9 +148,6 @@ void testGETFindAssignmentsByPeriodIdDefaultAssignments() {
 
         assertNotNull(response.body());
         assertEquals(0, Objects.requireNonNull(response.body()).size());
-        assertTrue(response.body().stream().anyMatch(ra -> ra.getRevieweeId().equals(memberOne.getId())));
-        assertTrue(response.body().stream().anyMatch(ra -> ra.getRevieweeId().equals(memberTwo.getId())));
-        assertTrue(response.body().stream().anyMatch(ra -> ra.getRevieweeId().equals(memberThree.getId())));
         assertEquals(HttpStatus.OK, response.getStatus());
     }
 

server/src/test/java/com/objectcomputing/checkins/services/volunteering/VolunteeringRelationshipControllerTest.java

@@ -236,7 +236,7 @@ void canListAndFilterRelationships() {
         // Can include inactive relationships
         List<VolunteeringRelationship> allWithInactiveRelationships = relationshipClient.list(memberAuth, null, null, true);
         assertEquals(4, allWithInactiveRelationships.size());
-        assertEquals(List.of(sarahLiftForLife.getId(), timLiftForLife.getId(), sarahFoodbank.getId(), timFoodbankInactive.getId()), allWithInactiveRelationships.stream().map(VolunteeringRelationship::getId).toList());
+        assertEquals(List.of(sarahFoodbank.getId(), sarahLiftForLife.getId(), timFoodbankInactive.getId(), timLiftForLife.getId()), allWithInactiveRelationships.stream().map(VolunteeringRelationship::getId).toList());
 
         // Can filter by memberId
         List<VolunteeringRelationship> timRelationships = relationshipClient.list(memberAuth, tim.getId(), null, null);