🐛 sort checklists by last update

Author: FabianWilms

personalization-service/src/main/java/de/muenchen/dbs/personalization/checklist/ChecklistRepository.java

@@ -9,6 +9,6 @@
 @Repository
 public interface ChecklistRepository extends JpaRepository<Checklist, UUID> {
 
-    List<Checklist> findChecklistByLhmExtId(String lhmExtId);
+    List<Checklist> findChecklistByLhmExtIdOrderByLastUpdateDesc(String lhmExtId);
 
 }

personalization-service/src/main/java/de/muenchen/dbs/personalization/checklist/ChecklistService.java

@@ -29,13 +29,14 @@ public Checklist createChecklist(final Checklist checklist) {
         final String lhmExtId = getLhmExtIdFromAuthenticationOrThrow();
         log.debug("Create Checklist {} for {}", checklist, lhmExtId);
         checklist.setLhmExtId(lhmExtId);
+        checklist.setLastUpdate(ZonedDateTime.now());
         return checklistRepository.save(checklist);
     }
 
     public List<Checklist> getChecklists() {
         final String lhmExtId = getLhmExtIdFromAuthenticationOrThrow();
         log.debug("Get all checklists of {}", lhmExtId);
-        return checklistRepository.findChecklistByLhmExtId(lhmExtId);
+        return checklistRepository.findChecklistByLhmExtIdOrderByLastUpdateDesc(lhmExtId);
     }
 
     public Checklist getChecklist(final UUID checklistId) {

personalization-service/src/test/java/de/muenchen/dbs/personalization/checklist/ChecklistServiceTest.java

@@ -88,14 +88,14 @@ void givenLhmExtId_thenReturnChecklists() {
             final Checklist checklist1 = createTestChecklist(id1, USER_LHM_EXT_ID, null);
             final Checklist checklist2 = createTestChecklist(id2, USER_LHM_EXT_ID, null);
 
-            when(checklistRepository.findChecklistByLhmExtId(USER_LHM_EXT_ID)).thenReturn(List.of(checklist1, checklist2));
+            when(checklistRepository.findChecklistByLhmExtIdOrderByLastUpdateDesc(USER_LHM_EXT_ID)).thenReturn(List.of(checklist1, checklist2));
 
             // When
             final List<Checklist> result = checklistService.getChecklists();
 
             // Then
             Assertions.assertEquals(List.of(checklist1, checklist2), result);
-            verify(checklistRepository, times(1)).findChecklistByLhmExtId(USER_LHM_EXT_ID);
+            verify(checklistRepository, times(1)).findChecklistByLhmExtIdOrderByLastUpdateDesc(USER_LHM_EXT_ID);
         }
 
     }