Fix integration tests to match updated server::run signature

Previous: Integration tests were calling spkrd::server::run with three
parameters (port, retry_timeout, device_path), following the function
signature that existed before debug logging was added to the server
module. All four test functions used this outdated calling convention.

Changed: Updated all server::run function calls in integration tests to
include the fourth debug parameter set to false, maintaining clean test
output while matching the current function signature that requires
(port, retry_timeout, device_path, debug).

See: changelog/20250829-cargo-test-fixes.md

Author: knz

changelog/20250829-cargo-test-fixes.md

@@ -0,0 +1,24 @@
+# Cargo Test Fixes - 2025-08-29
+
+## Task Specification
+Fix failing `cargo test` command in the SPKRD project.
+
+## Current Status
+COMPLETED - All cargo tests now pass successfully
+
+## Files Modified
+- tests/integration_tests.rs (updated all 4 server::run calls to include debug: false parameter)
+
+## High-Level Decisions
+- Tests should use `debug: false` since they don't need debug output during testing
+
+## Requirements Changes
+(None currently)
+
+## Rationales and Alternatives
+- Using `false` for debug parameter in tests keeps test output clean and focused
+- Alternative would be `true` but that would add unnecessary verbosity to test runs
+
+## Obstacles and Solutions
+- Issue: Function signature mismatch between server::run definition and test calls
+- Solution: Add `false` as 4th parameter to all test calls

tests/integration_tests.rs

@@ -13,7 +13,7 @@ async fn test_server_with_file_device() {
     // Start the server on a random available port
     let port = find_available_port().await;
     let server_handle = tokio::spawn(async move {
-        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path).await;
+        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path, false).await;
     });
 
     // Wait a moment for the server to start
@@ -52,7 +52,7 @@ async fn test_melody_validation() {
     // Start the server on a random available port
     let port = find_available_port().await;
     let server_handle = tokio::spawn(async move {
-        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path).await;
+        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path, false).await;
     });
 
     // Wait a moment for the server to start
@@ -92,7 +92,7 @@ async fn test_multiple_requests() {
     // Start the server on a random available port
     let port = find_available_port().await;
     let server_handle = tokio::spawn(async move {
-        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path).await;
+        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path, false).await;
     });
 
     // Wait a moment for the server to start
@@ -138,7 +138,7 @@ async fn test_invalid_utf8() {
     // Start the server on a random available port
     let port = find_available_port().await;
     let server_handle = tokio::spawn(async move {
-        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path).await;
+        let _ = spkrd::server::run(port, Duration::from_secs(30), device_path, false).await;
     });
 
     // Wait a moment for the server to start