chore(bench): exclude rides with low moving time ratio

1 files changed, 12 insertions, 2 deletions

diff --git a/cli/cli_test.go b/cli/cli_test.go
index 6a3e148..59dec79 100644
--- a/cli/cli_test.go
+++ b/cli/cli_test.go
@@ -8,13 +8,23 @@ import (
 )
 
 func TestBacktestEligibilityExcludesShortDistances(t *testing.T) {
-	short := rides.Ride{GPXPath: "ride.gpx", DistanceM: 10_000 - 1, MovingTimeS: 1}
-	minimum := rides.Ride{GPXPath: "ride.gpx", DistanceM: 10_000, MovingTimeS: 1}
+	short := rides.Ride{GPXPath: "ride.gpx", DistanceM: 10_000 - 1, MovingTimeS: 8_000, ElapsedTimeS: 10_000}
+	minimum := rides.Ride{GPXPath: "ride.gpx", DistanceM: 10_000, MovingTimeS: 8_000, ElapsedTimeS: 10_000}
 
 	assert.False(t, isBacktestEligible(short))
 	assert.True(t, isBacktestEligible(minimum))
 }
 
+func TestBacktestEligibilityExcludesLowMovingRatio(t *testing.T) {
+	low := rides.Ride{GPXPath: "ride.gpx", DistanceM: 10_000, MovingTimeS: 7_999, ElapsedTimeS: 10_000}
+	boundary := rides.Ride{GPXPath: "ride.gpx", DistanceM: 10_000, MovingTimeS: 8_000, ElapsedTimeS: 10_000}
+	noElapsed := rides.Ride{GPXPath: "ride.gpx", DistanceM: 10_000, MovingTimeS: 1}
+
+	assert.False(t, isBacktestEligible(low))
+	assert.True(t, isBacktestEligible(boundary))
+	assert.False(t, isBacktestEligible(noElapsed))
+}
+
 func TestBacktestStatsUseUnroundedSamples(t *testing.T) {
 	absErrors := []float64{0, 0.1, 0.1}