Skip to content

test: add z->t account detection test case#2237

Draft
nuttycom wants to merge 2 commits intozcash:mainfrom
nuttycom:test/z-to-t-account-detection
Draft

test: add z->t account detection test case#2237
nuttycom wants to merge 2 commits intozcash:mainfrom
nuttycom:test/z-to-t-account-detection

Conversation

@nuttycom
Copy link
Copy Markdown
Collaborator

Summary

  • Add z_to_t_account_detection test covering the shielded-to-transparent account detection logic
  • Tests both Sapling and Orchard variants
  • Verifies transactions sending from a shielded pool to a transparent address are correctly attributed to the funding account, both before and after mining

Test plan

  • Both Sapling and Orchard test variants pass
  • CI passes

Closes #1256

🤖 Generated with Claude Code

Add a test verifying that transactions sending from a shielded pool to
a transparent address are correctly attributed to the funding account.
Covers both Sapling and Orchard variants, exercising the detection
logic in store_decrypted_tx and the scan path.

Closes zcash#1256

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@nuttycom nuttycom force-pushed the test/z-to-t-account-detection branch from 75752e2 to 4b2e321 Compare March 25, 2026 02:17
The API now takes ConfirmationsPolicy rather than raw integers for
spend confirmation requirements.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

Add a test case for z->t account detection logic

1 participant