From ce2a984ee324d37ba1dd7c2c4e27e40e0508bedc Mon Sep 17 00:00:00 2001
From: l0rinc <pap.lorinc@gmail.com>
Date: Tue, 10 Feb 2026 16:58:17 -0500
Subject: [PATCH] test: cleanup, use HasReason in threadpool_tests.cpp

---
 src/test/threadpool_tests.cpp | 15 ++++++---------
 1 file changed, 6 insertions(+), 9 deletions(-)

diff --git a/src/test/threadpool_tests.cpp b/src/test/threadpool_tests.cpp
index c7878c3523b..647b0f7c28d 100644
--- a/src/test/threadpool_tests.cpp
+++ b/src/test/threadpool_tests.cpp
@@ -5,6 +5,7 @@
 #include <common/system.h>
 #include <logging.h>
 #include <random.h>
+#include <test/util/common.h>
 #include <util/string.h>
 #include <util/threadpool.h>
 #include <util/time.h>
@@ -164,21 +165,17 @@ BOOST_AUTO_TEST_CASE(task_exception_propagates_to_future)
     ThreadPool threadPool(POOL_NAME);
     threadPool.Start(NUM_WORKERS_DEFAULT);
 
-    int num_tasks = 5;
-    std::string err_msg{"something wrong happened"};
+    const auto make_err{[&](size_t n) { return strprintf("error on thread #%s", n); }};
+
+    const int num_tasks = 5;
     std::vector<std::future<void>> futures;
     futures.reserve(num_tasks);
     for (int i = 0; i < num_tasks; i++) {
-        futures.emplace_back(Submit(threadPool, [err_msg, i]() {
-            throw std::runtime_error(err_msg + util::ToString(i));
-        }));
+        futures.emplace_back(Submit(threadPool, [&make_err, i] { throw std::runtime_error(make_err(i)); }));
     }
 
     for (int i = 0; i < num_tasks; i++) {
-        BOOST_CHECK_EXCEPTION(futures.at(i).get(), std::runtime_error, [&](const std::runtime_error& e) {
-            BOOST_CHECK_EQUAL(e.what(), err_msg + util::ToString(i));
-            return true;
-        });
+        BOOST_CHECK_EXCEPTION(futures[i].get(), std::runtime_error, HasReason{make_err(i)});
     }
 }