Add minimal button tests

This commit adds a minimal set of tests for the button logic. Notice
that this is, indeed, a breaking a change and coordination with
dependent crates is required.

Author: Felipe Balbi

Cargo.lock

@@ -13,3 +13,7 @@ embassy-time.workspace = true
 embedded-hal-async.workspace = true
 embedded-hal.workspace = true
 log = { workspace = true, optional = true }
+
+[dev-dependencies]
+embedded-hal-mock = { version = "0.11.1", features = ["eh1", "embedded-hal-async"] }
+tokio = { version = "1.45.0", features = ["rt", "macros"] }

power-button-service/Cargo.toml

@@ -1,6 +1,6 @@
 //! Button Service Definitions
 
-use embassy_time::{with_timeout, Duration, Instant, TimeoutError, Timer};
+use embassy_time::{with_timeout, Delay, Duration, Instant, TimeoutError, Timer};
 use embedded_hal::digital::InputPin;
 use embedded_hal_async::digital::Wait;
 
@@ -48,7 +48,9 @@ impl<I: InputPin + Wait> Button<I> {
 
     /// Checks button state.
     pub async fn get_button_state(&mut self) -> ButtonState {
-        match self.config.debouncer.debounce(&mut self.gpio).await {
+        let mut delay = Delay;
+
+        match self.config.debouncer.debounce(&mut self.gpio, &mut delay).await {
             true => ButtonState::ButtonPressed(Instant::now()),
             false => ButtonState::ButtonReleased(Instant::now()),
         }