Allow setting of from/to stop in test leg builder

leonardehrenfried · leonardehrenfried · commit cafc4aa9cc2d · 2025-12-19T13:14:00.000+01:00
diff --git a/application/src/ext-test/java/org/opentripplanner/ext/fares/service/gtfs/v2/SameGroupIdPriorityTest.java b/application/src/ext-test/java/org/opentripplanner/ext/fares/service/gtfs/v2/SameGroupIdPriorityTest.java
@@ -9,10 +9,8 @@
 import java.util.List;
 import java.util.Set;
 import java.util.function.Function;
-import java.util.stream.Stream;
 import org.junit.jupiter.api.Named;
 import org.junit.jupiter.params.ParameterizedTest;
-import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
 import org.opentripplanner.core.model.id.FeedScopedId;
 import org.opentripplanner.ext.fares.model.FareLegRule;
@@ -135,10 +133,6 @@ private static FareLegRuleBuilder rule2() {
   }
 
   private TransitLeg leg() {
-    return TestTransitLeg.of()
-      .withFrom(STOP1_ID)
-      .withTo(STOP2_ID)
-      .withRoute(ROUTE)
-      .build();
+    return TestTransitLeg.of().withFrom(STOP1_ID).withTo(STOP2_ID).withRoute(ROUTE).build();
   }
 }
diff --git a/application/src/test-fixtures/java/org/opentripplanner/model/plan/TestStopLocation.java b/application/src/test-fixtures/java/org/opentripplanner/model/plan/TestStopLocation.java
@@ -28,13 +28,13 @@ public int getIndex() {
 
   @Override
   @Nullable
-  public  I18NString getName() {
+  public I18NString getName() {
     return I18NString.of(id.toString());
   }
 
   @Override
   @Nullable
-  public  I18NString getDescription() {
+  public I18NString getDescription() {
     return null;
   }
 
diff --git a/application/src/test-fixtures/java/org/opentripplanner/model/plan/TestTransitLeg.java b/application/src/test-fixtures/java/org/opentripplanner/model/plan/TestTransitLeg.java
@@ -140,5 +140,4 @@ public List<FareOffer> fareOffers() {
   public static TestTransitLegBuilder of() {
     return new TestTransitLegBuilder();
   }
-
 }
diff --git a/application/src/test-fixtures/java/org/opentripplanner/model/plan/TestTransitLegBuilder.java b/application/src/test-fixtures/java/org/opentripplanner/model/plan/TestTransitLegBuilder.java
@@ -7,10 +7,8 @@
 import java.time.ZonedDateTime;
 import org.opentripplanner._support.time.ZoneIds;
 import org.opentripplanner.core.model.id.FeedScopedId;
-import org.opentripplanner.transit.model._data.TimetableRepositoryForTest;
 import org.opentripplanner.transit.model.basic.TransitMode;
 import org.opentripplanner.transit.model.network.Route;
-import org.opentripplanner.transit.model.network.TripPatternBuilder;
 import org.opentripplanner.transit.model.organization.Agency;
 import org.opentripplanner.transit.model.site.StopLocation;
 import org.opentripplanner.transit.model.timetable.Trip;
@@ -62,6 +60,7 @@ public TestTransitLegBuilder withTo(FeedScopedId id) {
     this.to = new TestStopLocation(id);
     return this;
   }
+
   public TestTransitLegBuilder withRoute(Route route) {
     this.trip = this.trip.copy().withRoute(route).build();
     return this;
@@ -70,7 +69,4 @@ public TestTransitLegBuilder withRoute(Route route) {
   public TestTransitLeg build() {
     return new TestTransitLeg(this);
   }
-
-
-
 }

Original file line number	Diff line number	Diff line change
`@@ -28,13 +28,13 @@ public int getIndex() {`
`28`	`28`
`29`	`29`	`@Override`
`30`	`30`	`@Nullable`
`31`		`- public I18NString getName() {`
	`31`	`+ public I18NString getName() {`
`32`	`32`	`return I18NString.of(id.toString());`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`@Override`
`36`	`36`	`@Nullable`
`37`		`- public I18NString getDescription() {`
	`37`	`+ public I18NString getDescription() {`
`38`	`38`	`return null;`
`39`	`39`	`}`
`40`	`40`
Original file line number	Diff line number	Diff line change
`@@ -140,5 +140,4 @@ public List<FareOffer> fareOffers() {`
`140`	`140`	`public static TestTransitLegBuilder of() {`
`141`	`141`	`return new TestTransitLegBuilder();`
`142`	`142`	`}`
`143`		`-`
`144`	`143`	`}`