Enable org.openrewrite.java.testing.junit5.JUnit5BestPractices

JabRef · Nov 21, 2023 · 4f4ac59 · 4f4ac59
1 parent f6b772c
commit 4f4ac59
Show file tree

Hide file tree

Showing 351 changed files with 2,171 additions and 2,169 deletions.
diff --git a/rewrite.yml b/rewrite.yml
@@ -197,5 +197,5 @@ recipeList:
 #  - org.openrewrite.staticanalysis.WriteOctalValuesAsDecimal
 
   - org.openrewrite.java.testing.junit5.CleanupAssertions
-  # - org.openrewrite.java.testing.junit5.JUnit5BestPractices
+  - org.openrewrite.java.testing.junit5.JUnit5BestPractices
   # - org.openrewrite.java.testing.junit5.UpgradeOkHttpMockWebServer
diff --git a/src/test/java/org/jabref/IconsPropertiesTest.java b/src/test/java/org/jabref/IconsPropertiesTest.java
@@ -17,10 +17,10 @@
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-public class IconsPropertiesTest {
+class IconsPropertiesTest {
 
     @Test
-    public void testExistenceOfIconImagesReferencedFromIconsProperties() throws IOException {
+    void testExistenceOfIconImagesReferencedFromIconsProperties() throws IOException {
         String folder = "src/main/resources/images/external";
         String iconsProperties = "Icons.properties";
         String iconsPropertiesPath = "src/main/resources/images/" + iconsProperties;

diff --git a/src/test/java/org/jabref/cli/AuxCommandLineTest.java b/src/test/java/org/jabref/cli/AuxCommandLineTest.java
@@ -21,17 +21,17 @@
 import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.mockito.Mockito.mock;
 
-public class AuxCommandLineTest {
+class AuxCommandLineTest {
 
     private ImportFormatPreferences importFormatPreferences;
 
     @BeforeEach
-    public void setUp() throws Exception {
+    void setUp() throws Exception {
         importFormatPreferences = mock(ImportFormatPreferences.class, Answers.RETURNS_DEEP_STUBS);
     }
 
     @Test
-    public void test() throws URISyntaxException, IOException {
+    void test() throws URISyntaxException, IOException {
         InputStream originalStream = AuxCommandLineTest.class.getResourceAsStream("origin.bib");
 
         File auxFile = Path.of(AuxCommandLineTest.class.getResource("paper.aux").toURI()).toFile();

diff --git a/src/test/java/org/jabref/gui/autocompleter/AppendPersonNamesStrategyTest.java b/src/test/java/org/jabref/gui/autocompleter/AppendPersonNamesStrategyTest.java
@@ -6,7 +6,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-public class AppendPersonNamesStrategyTest {
+class AppendPersonNamesStrategyTest {
 
     @Test
     void testWithoutParam() {

diff --git a/src/test/java/org/jabref/gui/autocompleter/ContentSelectorSuggestionProviderTest.java b/src/test/java/org/jabref/gui/autocompleter/ContentSelectorSuggestionProviderTest.java
@@ -14,7 +14,7 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 
-public class ContentSelectorSuggestionProviderTest {
+class ContentSelectorSuggestionProviderTest {
 
     private ContentSelectorSuggestionProvider autoCompleter;
 

diff --git a/src/test/java/org/jabref/gui/autocompleter/PersonNameStringConverterTest.java b/src/test/java/org/jabref/gui/autocompleter/PersonNameStringConverterTest.java
@@ -10,7 +10,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-public class PersonNameStringConverterTest {
+class PersonNameStringConverterTest {
 
     /** The author. **/
     private Author author;

diff --git a/src/test/java/org/jabref/gui/autocompleter/SuggestionProvidersTest.java b/src/test/java/org/jabref/gui/autocompleter/SuggestionProvidersTest.java
@@ -24,7 +24,7 @@ class SuggestionProvidersTest {
     private SuggestionProviders suggestionProviders;
 
     @BeforeEach
-    public void initializeSuggestionProviders() {
+    void initializeSuggestionProviders() {
         BibDatabase database = new BibDatabase();
         JournalAbbreviationRepository abbreviationRepository = mock(JournalAbbreviationRepository.class);
         Set<Field> completeFields = Set.of(StandardField.AUTHOR, StandardField.XREF, StandardField.XDATA, StandardField.JOURNAL, StandardField.PUBLISHER, SpecialField.PRINTED);
@@ -57,7 +57,7 @@ private static Stream<Arguments> getTestPairs() {
 
     @ParameterizedTest
     @MethodSource("getTestPairs")
-    public void testAppropriateCompleterReturned(Class<SuggestionProvider<BibEntry>> expected, Field field) {
+    void testAppropriateCompleterReturned(Class<SuggestionProvider<BibEntry>> expected, Field field) {
         assertEquals(expected, suggestionProviders.getForField(field).getClass());
     }
 

diff --git a/src/test/java/org/jabref/gui/autosaveandbackup/BackupManagerDiscardedTest.java b/src/test/java/org/jabref/gui/autosaveandbackup/BackupManagerDiscardedTest.java
@@ -43,7 +43,7 @@ class BackupManagerDiscardedTest {
     private Path backupDir;
 
     @BeforeEach
-    public void setup(@TempDir Path tempDir) throws Exception {
+    void setup(@TempDir Path tempDir) throws Exception {
         this.backupDir = tempDir.resolve("backups");
         Files.createDirectories(backupDir);
 
@@ -85,22 +85,22 @@ private void makeBackup() {
     }
 
     @Test
-    public void noDiscardingAChangeLeadsToNewerBackupBeReported() throws Exception {
+    void noDiscardingAChangeLeadsToNewerBackupBeReported() throws Exception {
         databaseModification();
         makeBackup();
         assertTrue(BackupManager.backupFileDiffers(testBib, backupDir));
     }
 
     @Test
-    public void noDiscardingASavedChange() throws Exception {
+    void noDiscardingASavedChange() throws Exception {
         databaseModification();
         makeBackup();
         saveDatabase();
         assertFalse(BackupManager.backupFileDiffers(testBib, backupDir));
     }
 
     @Test
-    public void discardingAChangeLeadsToNewerBackupToBeIgnored() throws Exception {
+    void discardingAChangeLeadsToNewerBackupToBeIgnored() throws Exception {
         databaseModification();
         makeBackup();
         backupManager.discardBackup(backupDir);

diff --git a/src/test/java/org/jabref/gui/autosaveandbackup/BackupManagerTest.java b/src/test/java/org/jabref/gui/autosaveandbackup/BackupManagerTest.java
@@ -33,7 +33,7 @@
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
-public class BackupManagerTest {
+class BackupManagerTest {
 
     Path backupDir;
 
@@ -43,7 +43,7 @@ void setup(@TempDir Path tempDir) {
     }
 
     @Test
-    public void backupFileNameIsCorrectlyGeneratedInAppDataDirectory() {
+    void backupFileNameIsCorrectlyGeneratedInAppDataDirectory() {
         Path bibPath = Path.of("tmp", "test.bib");
         backupDir = OS.getNativeDesktop().getBackupDirectory();
         Path bakPath = BackupManager.getBackupPathForNewBackup(bibPath, backupDir);
@@ -53,13 +53,13 @@ public void backupFileNameIsCorrectlyGeneratedInAppDataDirectory() {
     }
 
     @Test
-    public void backupFileIsEqualForNonExistingBackup() throws Exception {
+    void backupFileIsEqualForNonExistingBackup() throws Exception {
         Path originalFile = Path.of(BackupManagerTest.class.getResource("no-autosave.bib").toURI());
         assertFalse(BackupManager.backupFileDiffers(originalFile, backupDir));
     }
 
     @Test
-    public void backupFileIsEqual() throws Exception {
+    void backupFileIsEqual() throws Exception {
         // Prepare test: Create backup file on "right" path
         Path source = Path.of(BackupManagerTest.class.getResource("no-changes.bib.bak").toURI());
         Path target = BackupFileUtil.getPathForNewBackupFileAndCreateDirectory(Path.of(BackupManagerTest.class.getResource("no-changes.bib").toURI()), BackupFileType.BACKUP, backupDir);
@@ -70,7 +70,7 @@ public void backupFileIsEqual() throws Exception {
     }
 
     @Test
-    public void backupFileDiffers() throws Exception {
+    void backupFileDiffers() throws Exception {
         // Prepare test: Create backup file on "right" path
         Path source = Path.of(BackupManagerTest.class.getResource("changes.bib.bak").toURI());
         Path target = BackupFileUtil.getPathForNewBackupFileAndCreateDirectory(Path.of(BackupManagerTest.class.getResource("changes.bib").toURI()), BackupFileType.BACKUP, backupDir);
@@ -81,7 +81,7 @@ public void backupFileDiffers() throws Exception {
     }
 
     @Test
-    public void correctBackupFileDeterminedForMultipleBakFiles() throws Exception {
+    void correctBackupFileDeterminedForMultipleBakFiles() throws Exception {
         Path noChangesBib = Path.of(BackupManagerTest.class.getResource("no-changes.bib").toURI());
         Path noChangesBibBak = Path.of(BackupManagerTest.class.getResource("no-changes.bib.bak").toURI());
 
@@ -105,7 +105,7 @@ public void correctBackupFileDeterminedForMultipleBakFiles() throws Exception {
     }
 
     @Test
-    public void bakFileWithNewerTimeStampLeadsToDiff() throws Exception {
+    void bakFileWithNewerTimeStampLeadsToDiff() throws Exception {
         Path changesBib = Path.of(BackupManagerTest.class.getResource("changes.bib").toURI());
         Path changesBibBak = Path.of(BackupManagerTest.class.getResource("changes.bib.bak").toURI());
 
@@ -116,7 +116,7 @@ public void bakFileWithNewerTimeStampLeadsToDiff() throws Exception {
     }
 
     @Test
-    public void bakFileWithOlderTimeStampDoesNotLeadToDiff() throws Exception {
+    void bakFileWithOlderTimeStampDoesNotLeadToDiff() throws Exception {
         Path changesBib = Path.of(BackupManagerTest.class.getResource("changes.bib").toURI());
         Path changesBibBak = Path.of(BackupManagerTest.class.getResource("changes.bib.bak").toURI());
 
@@ -130,7 +130,7 @@ public void bakFileWithOlderTimeStampDoesNotLeadToDiff() throws Exception {
     }
 
     @Test
-    public void shouldNotCreateABackup(@TempDir Path customDir) throws Exception {
+    void shouldNotCreateABackup(@TempDir Path customDir) throws Exception {
         Path backupDir = customDir.resolve("subBackupDir");
         Files.createDirectories(backupDir);
 
@@ -156,7 +156,7 @@ public void shouldNotCreateABackup(@TempDir Path customDir) throws Exception {
     }
 
     @Test
-    public void shouldCreateABackup(@TempDir Path customDir) throws Exception {
+    void shouldCreateABackup(@TempDir Path customDir) throws Exception {
         Path backupDir = customDir.resolve("subBackupDir");
         Files.createDirectories(backupDir);
 

diff --git a/src/test/java/org/jabref/gui/edit/CopyMoreActionTest.java b/src/test/java/org/jabref/gui/edit/CopyMoreActionTest.java
@@ -31,7 +31,7 @@
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
-public class CopyMoreActionTest {
+class CopyMoreActionTest {
 
     private final DialogService dialogService = spy(DialogService.class);
     private final ClipBoardManager clipBoardManager = mock(ClipBoardManager.class);
@@ -46,7 +46,7 @@ public class CopyMoreActionTest {
     private BibEntry entry;
 
     @BeforeEach
-    public void setUp() {
+    void setUp() {
         String title = "A tale from the trenches";
         entry = new BibEntry(StandardEntryType.Misc)
                 .withField(StandardField.AUTHOR, "Souti Chattopadhyay and Nicholas Nelson and Audrey Au and Natalia Morales and Christopher Sanchez and Rahul Pandita and Anita Sarma")
@@ -61,7 +61,7 @@ public void setUp() {
     }
 
     @Test
-    public void testExecuteOnFail() {
+    void testExecuteOnFail() {
         when(stateManager.getActiveDatabase()).thenReturn(Optional.empty());
         when(stateManager.getSelectedEntries()).thenReturn(FXCollections.emptyObservableList());
         copyMoreAction = new CopyMoreAction(StandardActions.COPY_TITLE, dialogService, stateManager, clipBoardManager, preferencesService, abbreviationRepository);
@@ -72,7 +72,7 @@ public void testExecuteOnFail() {
     }
 
     @Test
-    public void testExecuteCopyTitleWithNoTitle() {
+    void testExecuteCopyTitleWithNoTitle() {
         BibEntry entryWithNoTitle = (BibEntry) entry.clone();
         entryWithNoTitle.clearField(StandardField.TITLE);
         ObservableList<BibEntry> entriesWithNoTitles = FXCollections.observableArrayList(entryWithNoTitle);
@@ -88,7 +88,7 @@ public void testExecuteCopyTitleWithNoTitle() {
     }
 
     @Test
-    public void testExecuteCopyTitleOnPartialSuccess() {
+    void testExecuteCopyTitleOnPartialSuccess() {
         BibEntry entryWithNoTitle = (BibEntry) entry.clone();
         entryWithNoTitle.clearField(StandardField.TITLE);
         ObservableList<BibEntry> mixedEntries = FXCollections.observableArrayList(entryWithNoTitle, entry);
@@ -106,7 +106,7 @@ public void testExecuteCopyTitleOnPartialSuccess() {
     }
 
     @Test
-    public void testExecuteCopyTitleOnSuccess() {
+    void testExecuteCopyTitleOnSuccess() {
         ObservableList<BibEntry> entriesWithTitles = FXCollections.observableArrayList(entry);
         BibDatabaseContext databaseContext = new BibDatabaseContext(new BibDatabase(entriesWithTitles));
 
@@ -122,7 +122,7 @@ public void testExecuteCopyTitleOnSuccess() {
     }
 
     @Test
-    public void testExecuteCopyKeyWithNoKey() {
+    void testExecuteCopyKeyWithNoKey() {
         BibEntry entryWithNoKey = (BibEntry) entry.clone();
         entryWithNoKey.clearCiteKey();
         ObservableList<BibEntry> entriesWithNoKeys = FXCollections.observableArrayList(entryWithNoKey);
@@ -138,7 +138,7 @@ public void testExecuteCopyKeyWithNoKey() {
     }
 
     @Test
-    public void testExecuteCopyKeyOnPartialSuccess() {
+    void testExecuteCopyKeyOnPartialSuccess() {
         BibEntry entryWithNoKey = (BibEntry) entry.clone();
         entryWithNoKey.clearCiteKey();
         ObservableList<BibEntry> mixedEntries = FXCollections.observableArrayList(entryWithNoKey, entry);
@@ -156,7 +156,7 @@ public void testExecuteCopyKeyOnPartialSuccess() {
     }
 
     @Test
-    public void testExecuteCopyKeyOnSuccess() {
+    void testExecuteCopyKeyOnSuccess() {
         ObservableList<BibEntry> entriesWithKeys = FXCollections.observableArrayList(entry);
         BibDatabaseContext databaseContext = new BibDatabaseContext(new BibDatabase(entriesWithKeys));
 
@@ -172,7 +172,7 @@ public void testExecuteCopyKeyOnSuccess() {
     }
 
     @Test
-    public void testExecuteCopyDoiWithNoDoi() {
+    void testExecuteCopyDoiWithNoDoi() {
         BibEntry entryWithNoDoi = (BibEntry) entry.clone();
         entryWithNoDoi.clearField(StandardField.DOI);
         ObservableList<BibEntry> entriesWithNoDois = FXCollections.observableArrayList(entryWithNoDoi);
@@ -188,7 +188,7 @@ public void testExecuteCopyDoiWithNoDoi() {
     }
 
     @Test
-    public void testExecuteCopyDoiOnPartialSuccess() {
+    void testExecuteCopyDoiOnPartialSuccess() {
         BibEntry entryWithNoDoi = (BibEntry) entry.clone();
         entryWithNoDoi.clearField(StandardField.DOI);
         ObservableList<BibEntry> mixedEntries = FXCollections.observableArrayList(entryWithNoDoi, entry);
@@ -206,7 +206,7 @@ public void testExecuteCopyDoiOnPartialSuccess() {
     }
 
     @Test
-    public void testExecuteCopyDoiOnSuccess() {
+    void testExecuteCopyDoiOnSuccess() {
         ObservableList<BibEntry> entriesWithDois = FXCollections.observableArrayList(entry);
         BibDatabaseContext databaseContext = new BibDatabaseContext(new BibDatabase(entriesWithDois));
 

diff --git a/src/test/java/org/jabref/gui/edit/CopyOrMoveFieldContentTabViewModelTest.java b/src/test/java/org/jabref/gui/edit/CopyOrMoveFieldContentTabViewModelTest.java
@@ -15,7 +15,7 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.Mockito.mock;
 
-public class CopyOrMoveFieldContentTabViewModelTest {
+class CopyOrMoveFieldContentTabViewModelTest {
     CopyOrMoveFieldContentTabViewModel copyOrMoveFieldContentTabViewModel;
     BibEntry entryA;
     BibEntry entryB;

diff --git a/src/test/java/org/jabref/gui/edit/EditFieldContentTabViewModelTest.java b/src/test/java/org/jabref/gui/edit/EditFieldContentTabViewModelTest.java
@@ -16,7 +16,7 @@
 import static org.junit.jupiter.api.Assertions.assertNotEquals;
 import static org.mockito.Mockito.mock;
 
-public class EditFieldContentTabViewModelTest {
+class EditFieldContentTabViewModelTest {
     EditFieldContentViewModel editFieldContentViewModel;
     BibEntry entryA;
     BibEntry entryB;

diff --git a/src/test/java/org/jabref/gui/edit/ManageKeywordsViewModelTest.java b/src/test/java/org/jabref/gui/edit/ManageKeywordsViewModelTest.java
@@ -19,7 +19,7 @@
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
-public class ManageKeywordsViewModelTest {
+class ManageKeywordsViewModelTest {
 
     private final BibEntryPreferences bibEntryPreferences = mock(BibEntryPreferences.class);
     private ManageKeywordsViewModel keywordsViewModel;

diff --git a/src/test/java/org/jabref/gui/edit/RenameFieldViewModelTest.java b/src/test/java/org/jabref/gui/edit/RenameFieldViewModelTest.java
@@ -18,7 +18,7 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.Mockito.mock;
 
-public class RenameFieldViewModelTest {
+class RenameFieldViewModelTest {
     RenameFieldViewModel renameFieldViewModel;
     BibEntry entryA;
     BibEntry entryB;

diff --git a/src/test/java/org/jabref/gui/edit/ReplaceStringViewModelTest.java b/src/test/java/org/jabref/gui/edit/ReplaceStringViewModelTest.java
@@ -20,7 +20,7 @@
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
-public class ReplaceStringViewModelTest {
+class ReplaceStringViewModelTest {
 
     private final LibraryTab libraryTab = mock(LibraryTab.class);
     private ReplaceStringViewModel viewModel;

diff --git a/src/test/java/org/jabref/gui/entryeditor/CommentsTabTest.java b/src/test/java/org/jabref/gui/entryeditor/CommentsTabTest.java
@@ -124,7 +124,7 @@ void testDetermineFieldsToShowWorksForMultipleUsers() {
     }
 
     @Test
-    public void testDifferentiateCaseInUserName() {
+    void testDifferentiateCaseInUserName() {
         UserSpecificCommentField field1 = new UserSpecificCommentField("USER");
         UserSpecificCommentField field2 = new UserSpecificCommentField("user");
         assertNotEquals(field1, field2, "Two UserSpecificCommentField instances with usernames that differ only by case should be considered different");