Skip to content

Commit fe6fb30

Browse files
Rename confusing test names
1 parent 22f8440 commit fe6fb30

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

application/src/ext-test/java/org/opentripplanner/ext/fares/impl/gtfs/TimeLimitEvaluatorTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ class TimeLimitEvaluatorTest {
3030
.withEndTime("10:20")
3131
.build();
3232

33-
private static List<Arguments> withInLimitCases() {
33+
private static List<Arguments> withinLimitCases() {
3434
return List.of(
3535
Arguments.of(DEPARTURE_TO_ARRIVAL, ofMinutes(20)),
3636
Arguments.of(DEPARTURE_TO_ARRIVAL, ofHours(1)),
@@ -43,13 +43,13 @@ private static List<Arguments> withInLimitCases() {
4343
}
4444

4545
@ParameterizedTest
46-
@MethodSource("withInLimitCases")
46+
@MethodSource("withinLimitCases")
4747
void withinLimit(TimeLimitType type, Duration duration) {
4848
var limit = new TimeLimit(type, duration);
4949
assertTrue(TimeLimitEvaluator.withinTimeLimit(limit, FIRST, SECOND));
5050
}
5151

52-
private static List<Arguments> belowLimitCases() {
52+
private static List<Arguments> outsideLimitCases() {
5353
return List.of(
5454
Arguments.of(DEPARTURE_TO_ARRIVAL, ofMinutes(20).minusSeconds(20)),
5555
Arguments.of(DEPARTURE_TO_DEPARTURE, ofMinutes(10).minusSeconds(1)),
@@ -59,8 +59,8 @@ private static List<Arguments> belowLimitCases() {
5959
}
6060

6161
@ParameterizedTest
62-
@MethodSource("belowLimitCases")
63-
void belowLimit(TimeLimitType type, Duration duration) {
62+
@MethodSource("outsideLimitCases")
63+
void outsideLimit(TimeLimitType type, Duration duration) {
6464
var limit = new TimeLimit(type, duration);
6565
assertFalse(TimeLimitEvaluator.withinTimeLimit(limit, FIRST, SECOND));
6666
}

0 commit comments

Comments
 (0)