fix: resolve test errors

Author: rsny430

src/CycleComputer.h

@@ -1,6 +1,5 @@
 #pragma once
 
-#include "Config.h"
 #include "domain/Clock.h"
 #include "domain/Trip.h"
 #include "ui/Input.h"

tests/host/CycleComputerTest.cpp

@@ -182,16 +182,15 @@ TEST_F(CycleComputerTest, ResetData) {
   EXPECT_CALL(mockBtnA, isPressed()).WillOnce(Return(true)).WillRepeatedly(Return(false));
   computer->update(); // Change Mode
 
+  // Simulate BTN_BOTH: (A Pressed && B Held) OR (B Pressed && A Held)
+  mockGnss.setSpeed(0.0f);
+
   // Simulate BTN_BOTH: (A Pressed && B Held) OR (B Pressed && A Held)
   EXPECT_CALL(mockBtnA, isPressed()).WillOnce(Return(true)).RetiresOnSaturation();
   EXPECT_CALL(mockBtnB, isHeld()).WillOnce(Return(true)).RetiresOnSaturation();
 
-  computer->update();
-
-  mockGnss.setSpeed(0.0f);
-
+  // Expect the display to update with "0.0" after reset
   EXPECT_CALL(mockDisplay, print(testing::HasSubstr("0.0"))).Times(AtLeast(1));
 
-  delay(1000); // Advance time to trigger display update
   computer->update();
 }

tests/host/TripTest.cpp

@@ -51,12 +51,9 @@ TEST_F(TripTest, IgnoreLowSpeed) {
   EXPECT_FLOAT_EQ(trip.getDistanceKm(), 0.0f);
 }
 
-TEST_F(TripTest, TimeFormatting) {
-  char buffer[32];
-
+TEST_F(TripTest, MovingTimeCalculation) {
   // Test 0
-  trip.getMovingTimeStr(buffer, sizeof(buffer));
-  EXPECT_STREQ(buffer, "00:00");
+  EXPECT_EQ(trip.getMovingTimeMs(), 0);
 
   // Test 1h 1m 1s = 3600 + 60 + 1 = 3661 sec = 3661000 ms
   // Manually injecting moving time via update logic is hard without specific speed profile.
@@ -69,8 +66,7 @@ TEST_F(TripTest, TimeFormatting) {
   now += 3661000;
   trip.update(createNavData(10.0f), now); // Moving, so adds to moving time
 
-  trip.getMovingTimeStr(buffer, sizeof(buffer));
-  EXPECT_STREQ(buffer, "01:01");
+  EXPECT_EQ(trip.getMovingTimeMs(), 3661000);
 }
 
 TEST_F(TripTest, CalculateAvgSpeed) {
@@ -111,15 +107,12 @@ TEST_F(TripTest, StopMovingDoesNotIncreaseMovingTime) {
   trip.update(createNavData(10.0f), now); // Moving for 1 hour
   // movingTime = 1h
 
-  char buffer[32];
-  trip.getMovingTimeStr(buffer, sizeof(buffer));
-  // 1h -> 01:00
-  EXPECT_STREQ(buffer, "01:00");
+  // 1h -> 3600000ms
+  EXPECT_EQ(trip.getMovingTimeMs(), 3600000);
 
   now += 3600000;                        // Another hour passes...
   trip.update(createNavData(0.0f), now); // ...but not moving
   // movingTime should stay 1h
 
-  trip.getMovingTimeStr(buffer, sizeof(buffer));
-  EXPECT_STREQ(buffer, "01:00");
+  EXPECT_EQ(trip.getMovingTimeMs(), 3600000);
 }