rest: deduplicate `interface_rest.py` negative tests
What the developer wrote
Authored by Roman Zeyde
rest: deduplicate `interface_rest.py` negative tests
What changed, and why it matters
This commit removes one duplicate line from a test file. It is a code cleanup change with no effect on the actual Bitcoin Core software that users run, and no security relevance.
No action needed. This is a non-functional test cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change deletes a redundant test case in test/functional/interface_rest.py. The removed line was an exact duplicate of the line immediately following it, both calling get_block_part(status=400, query_params={“offset”: len(block_bin) + 1, “size”: 1}). This is purely a test-suite deduplication/refactoring commit.
Changed components
test/functional/interface_rest.pyInspect captured patch +0 / −1
diff --git a/test/functional/interface_rest.py b/test/functional/interface_rest.py
index be5bb78b..6b4bb797 100755
--- a/test/functional/interface_rest.py
+++ b/test/functional/interface_rest.py
@@ -485,7 +485,6 @@ class RESTTest (BitcoinTestFramework):
get_block_part(status=400, query_params={"offset": 0, "size": 0})
get_block_part(status=400, query_params={"offset": len(block_bin), "size": 0})
- get_block_part(status=400, query_params={"offset": len(block_bin) + 1, "size": 1})
get_block_part(status=400, query_params={"offset": len(block_bin), "size": 1})
get_block_part(status=400, query_params={"offset": len(block_bin) + 1, "size": 1})
get_block_part(status=400, query_params={"offset": 0, "size": len(block_bin) + 1})
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.