Skip to content

Commit a358457

Browse files
fix: clean up tests
- Fix error message - Remove whitespaces from citation keys
1 parent e15d40d commit a358457

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

jabgui/src/test/java/org/jabref/gui/mergebibfilesintocurrentbib/MergeBibFilesIntoCurrentBibTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -203,7 +203,7 @@ public void sameCitationKeyMergeTest() {
203203
);
204204

205205
action.execute();
206-
assertEquals(1, mockedMergeEntriesAction.constructed().size(), "Expected 1 MergeEntriesAction instance but found");
206+
assertEquals(1, mockedMergeEntriesAction.constructed().size(), "Expected MergeEntriesAction instance not found");
207207
}
208208
}
209209

@@ -250,7 +250,7 @@ public void sameCitationKeyNoMergePreferenceTest() {
250250
@Test
251251
public void duplicateMergeTest() {
252252
BibEntry currentEntry = new BibEntry(StandardEntryType.Article)
253-
.withCitationKey("DIFFERENT CITATION KEY")
253+
.withCitationKey("DIFFERENTCITATIONKEY")
254254
.withField(StandardField.AUTHOR, "Foo Bar")
255255
.withField(StandardField.TITLE, "First Article")
256256
.withField(StandardField.JOURNAL, "International Journal of Something")
@@ -275,7 +275,7 @@ public void duplicateMergeTest() {
275275
);
276276

277277
action.execute();
278-
assertEquals(1, mockedMergeEntriesAction.constructed().size(), "Expected 1 MergeEntriesAction instance but found");
278+
assertEquals(1, mockedMergeEntriesAction.constructed().size(), "Expected MergeEntriesAction instance not found");
279279
}
280280
}
281281

@@ -284,7 +284,7 @@ public void duplicateNoMergePreferenceTest() {
284284
when(mergeBibFilesIntoCurrentBibPreferences.shouldMergeSameKeyEntries()).thenReturn(false);
285285
when(mergeBibFilesIntoCurrentBibPreferences.shouldMergeDuplicateEntries()).thenReturn(false);
286286
BibEntry currentEntry = new BibEntry(StandardEntryType.Article)
287-
.withCitationKey("DIFFERENT CITATION KEY")
287+
.withCitationKey("DIFFERENTCITATIONKEY")
288288
.withField(StandardField.AUTHOR, "Foo Bar")
289289
.withField(StandardField.TITLE, "First Article")
290290
.withField(StandardField.JOURNAL, "International Journal of Something")
@@ -312,7 +312,7 @@ public void duplicateNoMergePreferenceTest() {
312312
assertEquals(1, entries.size(), "Should still have one entry");
313313

314314
BibEntry entry1 = entries.stream()
315-
.filter(e -> "DIFFERENT CITATION KEY".equals(e.getCitationKey().orElse("")))
315+
.filter(e -> "DIFFERENTCITATIONKEY".equals(e.getCitationKey().orElse("")))
316316
.findFirst()
317317
.orElseThrow(() -> new AssertionError("Entry 'test1' not found"));
318318

0 commit comments

Comments
 (0)