test: remove UNKNOWN_ERROR from script_tests
What the developer wrote
Authored by Bruno Garcia
test: remove UNKNOWN_ERROR from script_tests
What changed, and why it matters
This commit removes one line from a test file. It deletes an entry for 'UNKNOWN_ERROR' from a list of script error descriptions used only in unit tests. There is no change to the actual Bitcoin network code, consensus rules, or wallet behavior. It is a test-only cleanup with no security relevance visible in the commit.
No action required. Treat as routine test maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In src/test/script_tests.cpp, the static array script_errors loses the entry {SCRIPT_ERR_UNKNOWN_ERROR, ‘UNKNOWN_ERROR’}. This array maps ScriptError_t enum values to string names for test assertions. The removal likely reflects that SCRIPT_ERR_UNKNOWN_ERROR was removed or unused elsewhere, and the test table is being kept in sync. The diff touches no production code.
Changed components
src/test/script_tests.cppInspect captured patch +0 / −1
diff --git a/src/test/script_tests.cpp b/src/test/script_tests.cpp
index 63136409..22e838f1 100644
--- a/src/test/script_tests.cpp
+++ b/src/test/script_tests.cpp
@@ -52,7 +52,6 @@ struct ScriptErrorDesc
static ScriptErrorDesc script_errors[]={
{SCRIPT_ERR_OK, "OK"},
- {SCRIPT_ERR_UNKNOWN_ERROR, "UNKNOWN_ERROR"},
{SCRIPT_ERR_EVAL_FALSE, "EVAL_FALSE"},
{SCRIPT_ERR_OP_RETURN, "OP_RETURN"},
{SCRIPT_ERR_SCRIPT_SIZE, "SCRIPT_SIZE"},
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.