From 05d9b9c1f2dae2fe41f4c4f0aa6dcec4ff816245 Mon Sep 17 00:00:00 2001 From: Nicholas Dudfield Date: Wed, 20 Aug 2025 13:38:49 +0700 Subject: [PATCH] refactor: simplify getBackends() with early return for default case --- src/test/rdb/RelationalDatabase_test.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/test/rdb/RelationalDatabase_test.cpp b/src/test/rdb/RelationalDatabase_test.cpp index b083cbbed..96cd5554f 100644 --- a/src/test/rdb/RelationalDatabase_test.cpp +++ b/src/test/rdb/RelationalDatabase_test.cpp @@ -699,10 +699,12 @@ public: // Valid backends static const std::set validBackends = {"sqlite", "rwdb"}; + // Default to all valid backends if no arg specified + if (unittest_arg.empty()) + return {validBackends.begin(), validBackends.end()}; + std::set backends; // Use set to avoid duplicates - std::string argStr = - unittest_arg.empty() ? "sqlite,rwdb" : unittest_arg; - std::stringstream ss(argStr); + std::stringstream ss(unittest_arg); std::string backend; while (std::getline(ss, backend, ','))