From a54996d8d76d7168ec458597cee276de975bd699 Mon Sep 17 00:00:00 2001
From: 1-rafael-1 <rafael.koch@gmx.net>
Date: Tue, 16 Sep 2025 05:49:14 +0200
Subject: [PATCH] RTC alarm test timing was off

---
 tests/rp/src/bin/rtc.rs | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/tests/rp/src/bin/rtc.rs b/tests/rp/src/bin/rtc.rs
index 8d1d4ee14..c66981d95 100644
--- a/tests/rp/src/bin/rtc.rs
+++ b/tests/rp/src/bin/rtc.rs
@@ -36,6 +36,7 @@ async fn main(_spawner: Spawner) {
             second: 0,
         };
         rtc.set_datetime(now).unwrap();
+        Timer::after_millis(100).await;
     }
 
     // Test 1: Basic RTC functionality - read current time
@@ -53,8 +54,18 @@ async fn main(_spawner: Spawner) {
     // Test 2: Schedule and wait for alarm
     info!("Testing alarm scheduling");
 
-    // Schedule alarm for 3 seconds from now
-    let alarm_second = (initial_time.second + 3) % 60;
+    // Wait until we're at a predictable second, then schedule for a future second
+    loop {
+        let current = rtc.now().unwrap();
+        if current.second <= 55 {
+            break;
+        }
+        Timer::after_millis(100).await;
+    }
+
+    // Now schedule alarm for 3 seconds from current time
+    let current_time = rtc.now().unwrap();
+    let alarm_second = (current_time.second + 3) % 60;
     let alarm_filter = DateTimeFilter::default().second(alarm_second);
 
     info!("Scheduling alarm for second: {}", alarm_second);
@@ -69,13 +80,13 @@ async fn main(_spawner: Spawner) {
     let alarm_start = Instant::now();
     match select(Timer::after_secs(5), rtc.wait_for_alarm()).await {
         Either::First(_) => {
-            core::panic!("Alarm timeout - alarm should have triggered within 5 seconds");
+            core::panic!("Alarm timeout - alarm should have triggered by now");
         }
         Either::Second(_) => {
             let alarm_duration = Instant::now() - alarm_start;
             info!("ALARM TRIGGERED after {:?}", alarm_duration);
 
-            // Verify timing is reasonable (should be around 3 seconds, allow some margin)
+            // Verify timing is reasonable (should be around 3 seconds)
             assert!(
                 alarm_duration >= Duration::from_secs(2) && alarm_duration <= Duration::from_secs(4),
                 "Alarm timing incorrect: {:?}",
@@ -107,6 +118,7 @@ async fn main(_spawner: Spawner) {
         post_alarm_scheduled.is_none(),
         "Alarm should not be scheduled after triggering"
     );
+    info!("Alarm correctly cleared after triggering");
 
     info!("Test OK");
     cortex_m::asm::bkpt();