test: Rename shuffled_indeces to shuffled_indices
What changed, and why it matters
This commit is a simple typo fix in a test file. It renames a local variable from 'shuffled_indeces' to the correct spelling 'shuffled_indices'. There is no change to Bitcoin Core's actual network code, consensus rules, wallet handling, or any user-facing behavior. It only affects an internal variable name in a functional test.
No security action needed. This is a cosmetic/test-quality change and can be treated as routine maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff changes four occurrences of the misspelled identifier ‘shuffled_indeces’ to ‘shuffled_indices’ in test/functional/rpc_packages.py. The variable is used only within a single helper method to generate a random permutation for testmempoolaccept package ordering tests. No logic, semantics, or runtime behavior changes.
Changed components
test/functional/rpc_packages.pyInspect captured patch +4 / −4
diff --git a/test/functional/rpc_packages.py b/test/functional/rpc_packages.py
index d52fe33a..3c467394 100755
--- a/test/functional/rpc_packages.py
+++ b/test/functional/rpc_packages.py
@@ -44,10 +44,10 @@ class RPCPackagesTest(BitcoinTestFramework):
be used to test packages where the order does not matter. The ordering of transactions in package_hex and
testres_expected must match.
"""
- shuffled_indeces = list(range(len(package_hex)))
- random.shuffle(shuffled_indeces)
- shuffled_package = [package_hex[i] for i in shuffled_indeces]
- shuffled_testres = [testres_expected[i] for i in shuffled_indeces]
+ shuffled_indices = list(range(len(package_hex)))
+ random.shuffle(shuffled_indices)
+ shuffled_package = [package_hex[i] for i in shuffled_indices]
+ shuffled_testres = [testres_expected[i] for i in shuffled_indices]
assert_equal(shuffled_testres, self.nodes[0].testmempoolaccept(shuffled_package))
def run_test(self):
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.