lint: add missing ipc/test to grep_boost_fixture_test_suite
What the developer wrote
Authored by fanquake
lint: add missing ipc/test to grep_boost_fixture_test_suite
What changed, and why it matters
This commit is a minor linting fix. It adds a new test directory (src/ipc/test) to an internal script that checks whether Bitcoin's C++ test files use a specific Boost test macro correctly. It does not change any production code, wallet logic, networking, or consensus rules. There is no security issue here.
No security action required. This is a routine linting/maintenance change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change modifies test/lint/lint-tests.py to include src/ipc/test/**.cpp in the grep_boost_fixture_test_suite() function’s file list. This function searches for BOOST_FIXTURE_TEST_SUITE(…) declarations across test source files as part of the project’s static linting. The omission was causing tests under src/ipc/test to be skipped by this particular lint check. No runtime behavior is affected.
Changed components
test/lint/lint-tests.pyInspect captured patch +1 / −0
diff --git a/test/lint/lint-tests.py b/test/lint/lint-tests.py
index 919912dc..c7354bc4 100755
--- a/test/lint/lint-tests.py
+++ b/test/lint/lint-tests.py
@@ -20,6 +20,7 @@ def grep_boost_fixture_test_suite():
"-E",
r"^BOOST_FIXTURE_TEST_SUITE\(",
"--",
+ "src/ipc/test/**.cpp",
"src/test/**.cpp",
"src/wallet/test/**.cpp",
]
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.