From ecc7a9042506e6a5842a3bf6ddc3056fa797cab2 Mon Sep 17 00:00:00 2001 From: karlsb <36365664+karlsb@users.noreply.github.com> Date: Mon, 4 Mar 2024 10:23:30 +0100 Subject: [PATCH] Resolve request changes (#4) * test: enhanced RemoveLinksToNotExistentFilesTest * test: add real content * test: change structure * test: added expectedChanges * test: added a third test to RemoveLinksToNotExistentFilesTest * refactor: removed unnecessary toString * style: removed a whitespace * refactor: checkstyle formatting. formatted files according to checkstyle plugin. * refactor: replaced .get(0) with .getFirst() for first accessing element of List. Changed was required from ./gradlew rewriteDryRun --------- Co-authored-by: burcukilic <94201593+burcukilic@users.noreply.github.com> --- .../logic/cleanup/RemoveLinksToNotExistentFilesTest.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/jabref/logic/cleanup/RemoveLinksToNotExistentFilesTest.java b/src/test/java/org/jabref/logic/cleanup/RemoveLinksToNotExistentFilesTest.java index 671bad68bcc..aeedcc7172c 100644 --- a/src/test/java/org/jabref/logic/cleanup/RemoveLinksToNotExistentFilesTest.java +++ b/src/test/java/org/jabref/logic/cleanup/RemoveLinksToNotExistentFilesTest.java @@ -105,7 +105,7 @@ void deleteFileInMultipleLinkedEntry() { fileBefore.toFile().delete(); List changes = removeLinks.cleanup(entry); - assertEquals(expectedChange, changes.get(0)); + assertEquals(expectedChange, changes.getFirst()); assertEquals(expectedEntry, entry); } @@ -159,7 +159,7 @@ void deleteLinkedFile() { fileBefore.toFile().delete(); List changes = removeLinks.cleanup(entry); - assertEquals(expectedChange, changes.get(0)); + assertEquals(expectedChange, changes.getFirst()); assertEquals(expectedEntry, entry); } }