From 43a5eb1c8b41d889f86112da19e1656a69bf0eda Mon Sep 17 00:00:00 2001
From: Leonard Ehrenfried <mail@leonard.io>
Date: Mon, 9 Oct 2023 17:13:12 +0200
Subject: [PATCH] Improve logging of fare products

---
 .../opentripplanner/smoketest/SmokeTest.java   | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/src/test/java/org/opentripplanner/smoketest/SmokeTest.java b/src/test/java/org/opentripplanner/smoketest/SmokeTest.java
index 6ae12c5bc30..d7dbd924d28 100644
--- a/src/test/java/org/opentripplanner/smoketest/SmokeTest.java
+++ b/src/test/java/org/opentripplanner/smoketest/SmokeTest.java
@@ -16,6 +16,8 @@
 import org.opentripplanner.client.model.VehicleRentalStation;
 import org.opentripplanner.client.parameters.TripPlanParameters;
 import org.opentripplanner.smoketest.util.SmokeTestRequest;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * This is both a utility class and a category to select or deselect smoke tests during test
@@ -27,6 +29,8 @@
  */
 public class SmokeTest {
 
+  private static final Logger LOG = LoggerFactory.getLogger(SmokeTest.class);
+
   public static final OtpApiClient API_CLIENT = new OtpApiClient(
     ZoneId.of("America/New_York"),
     "http://localhost:8080"
@@ -112,8 +116,16 @@ static void assertThereArePatternsWithVehiclePositions() {
 
   static void assertThatAllTransitLegsHaveFareProducts(TripPlan plan) {
     var transitLegs = plan.transitItineraries().stream().flatMap(i -> i.transitLegs().stream());
-    transitLegs.forEach(leg ->
-      assertFalse(leg.fareProducts().isEmpty(), "Leg %s should have fare products".formatted(leg))
-    );
+    transitLegs.forEach(leg -> {
+      assertFalse(leg.fareProducts().isEmpty(), "Leg %s should have fare products".formatted(leg));
+
+      LOG.info(
+        "Leg with mode {} from {} to {} has {} fare products.",
+        leg.mode(),
+        leg.from(),
+        leg.to(),
+        leg.fareProducts().size()
+      );
+    });
   }
 }