From b08e0c67e40d167ccfa2a90bea2caefc9cea4d0b Mon Sep 17 00:00:00 2001 From: Marcin Maliszkiewicz Date: Wed, 29 Apr 2026 17:44:17 +0200 Subject: [PATCH] test/boost: add dummy case to table_helper_test for non-injection modes The only test requires SCYLLA_ENABLE_ERROR_INJECTION. In modes without it (e.g. release) the suite was empty, so pytest exited with code 5 ("no tests collected") and CI failed. Add a no-op case in that branch so collection always yields at least one test. --- test/boost/table_helper_test.cc | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/test/boost/table_helper_test.cc b/test/boost/table_helper_test.cc index df5cc66b53..0f08a9b73f 100644 --- a/test/boost/table_helper_test.cc +++ b/test/boost/table_helper_test.cc @@ -98,4 +98,14 @@ SEASTAR_TEST_CASE(test_concurrent_invalidation) { #endif // SCYLLA_ENABLE_ERROR_INJECTION +#ifndef SCYLLA_ENABLE_ERROR_INJECTION +// The only test in this suite requires error injection support. Without this +// dummy case the suite would be empty, which causes boost to report +// "test tree is empty" and pytest to exit with code 5 ("no tests collected"), +// failing CI in modes (e.g. release) where error injection is disabled. +BOOST_AUTO_TEST_CASE(test_skipped_no_error_injection) { + BOOST_TEST_MESSAGE("table_helper_test requires SCYLLA_ENABLE_ERROR_INJECTION; skipping"); +} +#endif + BOOST_AUTO_TEST_SUITE_END()