Add minimal button debounce 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

@@ -119,7 +119,7 @@ async fn main(spawner: Spawner) {
     // Create a power button instance
     let button_a = Input::new(p.PIO1_1, Pull::Up, Inverter::Disabled);
     // Create a debouncer instance
-    let debouncer = Debouncer::new(3, Duration::from_millis(10), ActiveState::ActiveLow);
+    let debouncer = Debouncer::new(3, 10, ActiveState::ActiveLow);
     // Create a custom button configuration instance
     let config_a = ButtonConfig::new(debouncer, Duration::from_millis(1000), Duration::from_millis(2000));
 

examples/rt685s-evk/src/bin/power_button.rs

@@ -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()),
         }