Skip to content

test: log dynamic EvoNode helper exceptions#7313

Draft
thepastaclaw wants to merge 1 commit intodashpay:developfrom
thepastaclaw:fix-dip3-v19-exception-logging
Draft

test: log dynamic EvoNode helper exceptions#7313
thepastaclaw wants to merge 1 commit intodashpay:developfrom
thepastaclaw:fix-dip3-v19-exception-logging

Conversation

@thepastaclaw
Copy link
Copy Markdown

Summary

  • include helper arguments and structured RPC error payloads when dynamic masternode/EvoNode setup is rejected
  • log unexpected dynamic helper exceptions with tracebacks before the existing accepted/rejected assertion fails
  • keep expected rejection cases as info-level logs

Context

This does not claim to fix the underlying feature_dip3_v19.py flake from #6702, but it should make the next failure more actionable instead of reducing it to dynamically_prepare_masternode failed or protx_evo rejected.

Testing

  • python3 -m py_compile test/functional/test_framework/test_framework.py
  • git diff --check
  • pre-PR code review: ship

@thepastaclaw
Copy link
Copy Markdown
Author

thepastaclaw commented May 8, 2026

Review Gate

Commit: 1bbfbf99

  • Debounce: 947m ago (need 30m)

  • CI checks: builds passed, 6/6 tests passed

  • CodeRabbit review: no comment yet

  • Off-peak hours: off-peak (12:45 PM PT Friday)

  • Run review now (check to override)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 8, 2026

✅ No Merge Conflicts Detected

This PR currently has no conflicts with other open PRs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant