From 2324850265d30e835b6a81f5becd794b0917148b Mon Sep 17 00:00:00 2001 From: Ryan Fowler Date: Sun, 1 Feb 2026 07:36:31 -0800 Subject: [PATCH] Check conn_for_each results in pool test The test was discarding the Vec> returned by conn_for_each, so a failed attach operation would go unnoticed. Unwrap each result to ensure the test fails if any connection errors. --- tests/tests.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tests/tests.rs b/tests/tests.rs index 4eaf002..ad15a20 100644 --- a/tests/tests.rs +++ b/tests/tests.rs @@ -206,7 +206,10 @@ async fn test_pool_conn_for_each() { ) }; // attach to the dummy db via conn_for_each - pool.conn_for_each(attach_fn).await; + let results = pool.conn_for_each(attach_fn).await; + for result in results { + result.unwrap(); + } // check that the dummy db is attached fn check_fn(conn: &rusqlite::Connection) -> Result, rusqlite::Error> {