tests: remove existing test pin data before running the test
What the developer wrote
Authored by Jon Griffiths
tests: remove existing test pin data before running the test
This makes e.g. debugging less painful.
What changed, and why it matters
This change only affects the project's test script. It deletes leftover test PIN files before running tests so developers don't get confused by stale data from earlier runs. There is no security issue in the product itself.
No security action needed. Treat as routine test-maintenance cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit modifies test_jade.py to remove any files matching ‘./*.pin’ before test execution. This is a test-hygiene improvement to avoid state leakage between test runs. It does not touch firmware, wallet logic, or production code paths.
Changed components
test_jade.pyInspect captured patch +4 / −0
diff --git a/test_jade.py b/test_jade.py
index 3e29eb8..7c72f98 100644
--- a/test_jade.py
+++ b/test_jade.py
@@ -4238,6 +4238,10 @@ if __name__ == '__main__':
if manage_agents:
btagent = start_agent(args.agentkeyfile)
+ # Remove PINs left over from previous test runs
+ for f in glob.glob("./*.pin"):
+ os.remove(f)
+
try:
info = get_jade_info()
if info:
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.