Handle large time spikesHEADmain

1 files changed, 33 insertions, 5 deletions

diff --git a/simloop/test/simloop_test.c b/simloop/test/simloop_test.c
index 61e7dff..bcf9d57 100644
--- a/simloop/test/simloop_test.c
+++ b/simloop/test/simloop_test.c
@@ -222,8 +222,14 @@ TEST_CASE(simloop_determinism) {
 }
 
 /// The simulation loop attempts to catch up with the clock in the event of a
-/// time spike. This is possible only if the simulation loops with a frequency
-/// higher than the requested update frequency given by the update delta time.
+/// time spike.
+///
+/// Catch-up is possible only if the simulation loops with a frequency higher
+/// than the requested update frequency given by the update delta time.
+///
+/// Catch-up is performed only for sufficiently small time spikes. For large
+/// time spikes, the simulation clock is warped. This test is for the small
+/// time spike case.
 static void simloop_catch_up(
     struct test_case_metadata* metadata, int update_ddt_ms, int loop_step_ms,
     bool expect_catchup) {
@@ -241,9 +247,10 @@ static void simloop_catch_up(
   int        frames = 0;
 
   // Simulate a time spike.
-  // Advance time to t=10s. That is a lag of 10,000ms / 100ms = 100 frames.
-  // The simulation now has 20s to catch up.
-  simloop_time_t dt = time_delta_from_sec(10);
+  // Advance time to t=1s. That is a lag of 1,000ms / 100ms = 10 frames.
+  // 10 frames is the maximum allowed catch-up.
+  // The simulation now has 29s to catch up.
+  simloop_time_t dt = time_delta_from_sec(1);
   for (simloop_time_t t = dt; t <= SIM_DURATION_SEC;) {
     simloop_update(&simloop, dt, &simout);
 
@@ -275,4 +282,25 @@ TEST_CASE(simloop_catch_up_failure) {
   simloop_catch_up(metadata, UPDATE_DDT_MS, LOOP_DDT_MS, false);
 }
 
+/// This tests the large time spike case, where the simulation clock is warped
+/// to the wall clock.
+TEST_CASE(simloop_warp) {
+  const int            UPDATE_FPS = 50;
+  const simloop_time_t UPDATE_DDT =
+      time_delta_from_sec(1.0 / (double)UPDATE_FPS);
+
+  Simloop    simloop = simloop_make(&(SimloopArgs){.update_fps = UPDATE_FPS});
+  SimloopOut simout;
+
+  // The maximum allowed catch-up is 10 frames. Simulate a time spike larger
+  // than that.
+  const simloop_time_t TIME_SPIKE = UPDATE_DDT * 20;
+  simloop_update(&simloop, TIME_SPIKE, &simout);
+  TEST_TRUE(simout.should_update); // Warp should still request update.
+
+  // Now "advance" by 0.
+  simloop_update(&simloop, 0, &simout);
+  TEST_TRUE(!simout.should_update); // No more updates after warp.
+}
+
 int main() { return 0; }