Skip to content

Commit d9964c6

Browse files
authored
add and extend unit tests (#7685)
1 parent 9dfd409 commit d9964c6

File tree

3 files changed

+37
-0
lines changed

3 files changed

+37
-0
lines changed

src/test/java/org/jabref/logic/formatter/bibtexfields/AddBracesFormatterTest.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,4 +72,9 @@ public void formatDoesNotRemoveBracesInBrokenString() {
7272
public void formatExample() {
7373
assertEquals("{In CDMA}", formatter.format(formatter.getExampleInput()));
7474
}
75+
76+
@Test
77+
public void formatStringWithMinimalRequiredLength() {
78+
assertEquals("{AB}", formatter.format("AB"));
79+
}
7580
}

src/test/java/org/jabref/logic/formatter/bibtexfields/NormalizePagesFormatterTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ private static Stream<Arguments> tests() {
6262
// special case, where -- is also put into
6363
Arguments.of("some--text", "some-text"),
6464
Arguments.of("pages 1--50", "pages 1-50"),
65+
Arguments.of("--43", "-43"),
6566

6667
// keep arbitrary text
6768
Arguments.of("some-text-with-dashes", "some-text-with-dashes"),
Lines changed: 31 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,31 @@
1+
package org.jabref.logic.integrity;
2+
3+
import java.util.Optional;
4+
import java.util.stream.Stream;
5+
6+
import org.jabref.logic.l10n.Localization;
7+
8+
import org.junit.jupiter.params.ParameterizedTest;
9+
import org.junit.jupiter.params.provider.Arguments;
10+
import org.junit.jupiter.params.provider.MethodSource;
11+
12+
import static org.junit.jupiter.api.Assertions.assertEquals;
13+
14+
public class ValidCitationKeyCheckerTest {
15+
16+
private final ValidCitationKeyChecker checker = new ValidCitationKeyChecker();
17+
18+
@ParameterizedTest
19+
@MethodSource("provideCitationKeys")
20+
void citationKeyValidity(Optional optionalArgument, String citationKey) {
21+
assertEquals(optionalArgument, checker.checkValue(citationKey));
22+
}
23+
24+
private static Stream<Arguments> provideCitationKeys() {
25+
return Stream.of(
26+
Arguments.of(Optional.of(Localization.lang("empty citation key")), ""),
27+
Arguments.of(Optional.empty(), "Seaver2019"),
28+
Arguments.of(Optional.of(Localization.lang("Invalid citation key")), "Seaver_2019}")
29+
);
30+
}
31+
}

0 commit comments

Comments
 (0)