fix flashcard deletion (#58)

* fix flashcard deletion Signed-off-by: PaulBredl <[email protected]> * fix test Signed-off-by: PaulBredl <[email protected]> --------- Signed-off-by: PaulBredl <[email protected]>
MEITREX · Oct 13, 2023 · bdd1a26 · bdd1a26
1 parent 58744a3
commit bdd1a26
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/de/unistuttgart/iste/gits/flashcard_service/service/FlashcardService.java b/src/main/java/de/unistuttgart/iste/gits/flashcard_service/service/FlashcardService.java
@@ -146,7 +146,7 @@ public void deleteFlashcardSetIfContentIsDeleted(final ContentChangeEvent dto) t
             return;
         }
 
-        flashcardSetRepository.deleteAllByIdInBatch(dto.getContentIds());
+        flashcardSetRepository.deleteAllById(dto.getContentIds());
     }
     /**
      * helper function to make sure received event message is complete

diff --git a/src/test/java/de/unistuttgart/iste/gits/flashcard_service/service/FlashcardServiceTest.java b/src/test/java/de/unistuttgart/iste/gits/flashcard_service/service/FlashcardServiceTest.java
@@ -47,7 +47,7 @@ void removeContentIds() {
 
         // invoke method under test
         assertDoesNotThrow(() -> flashcardService.deleteFlashcardSetIfContentIsDeleted(contentChangeEvent));
-        verify(flashcardSetRepository, times(1)).deleteAllByIdInBatch(any());
+        verify(flashcardSetRepository, times(1)).deleteAllById(any());
     }
     @Test
     void removeContentIdsWithNoIdsToBeRemovedTest() {
@@ -65,7 +65,7 @@ void removeContentIdsWithNoIdsToBeRemovedTest() {
         // invoke method under test
         assertDoesNotThrow(() -> flashcardService.deleteFlashcardSetIfContentIsDeleted(contentChangeEvent));
 
-        verify(flashcardSetRepository, times(1)).deleteAllByIdInBatch(any());
+        verify(flashcardSetRepository, times(1)).deleteAllById(any());
     }
 
     @Test
@@ -105,14 +105,14 @@ void removeContentIdsInvalidInputTest() {
             //invoke method under test
             assertDoesNotThrow(() -> flashcardService.deleteFlashcardSetIfContentIsDeleted(event));
             verify(flashcardSetRepository, never()).findAllById(any());
-            verify(flashcardSetRepository, never()).deleteAllInBatch(any());
+            verify(flashcardSetRepository, never()).deleteAllById(any());
         }
 
         for (ContentChangeEvent errorEvent : errorEvents) {
             //invoke method under test
             assertThrows(IncompleteEventMessageException.class, () -> flashcardService.deleteFlashcardSetIfContentIsDeleted(errorEvent));
             verify(flashcardSetRepository, never()).findAllById(any());
-            verify(flashcardSetRepository, never()).deleteAllInBatch(any());
+            verify(flashcardSetRepository, never()).deleteAllById(any());
         }
 
     }