Skip to content
Failed

Changes

Summary

  1. Adapt tests to updated service API and behavior (commit: 94f2f16) (details)
Commit 94f2f16c61adb2ddc8416c3afe470e06b5621e30 by juslopezg
Adapt tests to updated service API and behavior

Update unit tests to match renamed service methods and changed behaviors across services. Key changes:

- AdoptionTrackingServiceTest: removed unused import; call createAdoptionTracking, getAdoptionTrackings, getAdoptionTracking; removed a now-redundant invalid-date test.
- DevolutionServiceTest: switch to createDevolution, searchDevolutions, searchDevolution, updateDevolution, deleteDevolution; remove outdated assertions and an invalid-date test; clean up comments.
- NotificationServiceTest: adjust imports to IllegalOperationException, refine test data setup (userId, timestamps), remove an empty-message test, rename testSearchNotification to testFindById, and expect IllegalOperationException for deleting recent notifications; make delete success test declare the exception.
- ReviewServiceTest: minor data tweaks for generated comments, use createReview(adoptionId, adopterId), rename tests to match getReview/getReviews, add a test for listing reviews, and simplify update/delete assertions to validate returned or persisted state.

Overall these edits align tests with service API renames and updated validation/exception behavior, plus small cleanup and assertion adjustments.
(commit: 94f2f16)
The file was modifiedsrc/test/java/co/edu/udistrital/mdp/pets/services/AdoptionTrackingServiceTest.java (diff)
The file was modifiedsrc/test/java/co/edu/udistrital/mdp/pets/services/DevolutionServiceTest.java (diff)
The file was modifiedsrc/test/java/co/edu/udistrital/mdp/pets/services/ReviewServiceTest.java (diff)
The file was modifiedsrc/test/java/co/edu/udistrital/mdp/pets/services/NotificationServiceTest.java (diff)