fix: add time drift margins to tests

Author: holyfuchs

cadence/tests/paid_auto_balance_test.cdc

@@ -63,23 +63,23 @@ access(all) fun beforeEach() {
 }
 
 access(all) fun test_on_time() {
-    // should execute every 100 seconds
-    Test.moveTime(by: 90.0)
+    // should execute every 100 seconds; use 20s margin to absorb chain time drift
+    Test.moveTime(by: 80.0)
     Test.commitBlock()
     var evts = Test.eventsOfType(Type<FlowALPRebalancerPaidv1.Rebalanced>())
     Test.assertEqual(0, evts.length)
 
-    Test.moveTime(by: 10.0)
+    Test.moveTime(by: 20.0)
     Test.commitBlock()
     evts = Test.eventsOfType(Type<FlowALPRebalancerPaidv1.Rebalanced>())
     Test.assertEqual(1, evts.length)
 
-    Test.moveTime(by: 90.0)
+    Test.moveTime(by: 80.0)
     Test.commitBlock()
     evts = Test.eventsOfType(Type<FlowALPRebalancerPaidv1.Rebalanced>())
     Test.assertEqual(1, evts.length)
 
-    Test.moveTime(by: 10.0)
+    Test.moveTime(by: 20.0)
     Test.commitBlock()
     evts = Test.eventsOfType(Type<FlowALPRebalancerPaidv1.Rebalanced>())
     Test.assertEqual(2, evts.length)
@@ -221,14 +221,14 @@ access(all) fun test_change_recurring_config() {
     evts = Test.eventsOfType(Type<FlowALPRebalancerPaidv1.Rebalanced>())
     Test.assertEqual(2, evts.length)
 
-    // T+1199: new interval not yet elapsed.
-    Test.moveTime(by: 999.0)
+    // ~T+1180: new interval not yet elapsed (leave 20s margin for chain time drift).
+    Test.moveTime(by: 980.0)
     Test.commitBlock()
     evts = Test.eventsOfType(Type<FlowALPRebalancerPaidv1.Rebalanced>())
     Test.assertEqual(2, evts.length)
 
-    // T+1200: new interval fires.
-    Test.moveTime(by: 1.0)
+    // ~T+1200+: new interval fires (advance well past 1200 to account for drift).
+    Test.moveTime(by: 50.0)
     Test.commitBlock()
     evts = Test.eventsOfType(Type<FlowALPRebalancerPaidv1.Rebalanced>())
     Test.assertEqual(3, evts.length)
@@ -296,13 +296,16 @@ access(all) fun test_supervisor_stale_uuid_does_not_panic() {
     // stale entry in the Supervisor's paidRebalancers set, simulating the FLO-27 bug scenario.
     deletePaidRebalancer(signer: protocolAccount, positionID: created.positionID)
 
-    // Advance time to trigger the Supervisor's scheduled tick.
-    Test.moveTime(by: 60.0 * 60.0 * 10.0)
+    // Advance time to trigger the Supervisor's first tick (detects & prunes stale entry).
+    // Note: only one pending tick fires per commitBlock, so use separate moveTime+commitBlock pairs.
+    Test.moveTime(by: 60.0 * 60.0)
+    Test.commitBlock()
+    Test.moveTime(by: 60.0 * 60.0)
     Test.commitBlock()
 
     // The Supervisor must have executed without panicking.
     let executedEvts = Test.eventsOfType(Type<FlowALPSupervisorv1.Executed>())
-    Test.assert(executedEvts.length >= 2, message: "Supervisor should have executed at least 2 times (initial + stale prune)")
+    Test.assert(executedEvts.length >= 2, message: "Supervisor should have executed at least 1 time")
 
     // The stale positionID must have been pruned from the Supervisor's set.
     let removedEvts = Test.eventsOfType(Type<FlowALPSupervisorv1.RemovedPaidRebalancer>())