Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test/LDG-517--nano-app-implement-transfertopublic-test #33

Merged
merged 4 commits into from
Dec 10, 2024

Conversation

keiff3r
Copy link

@keiff3r keiff3r commented Dec 10, 2024

  • Add test P1 enum values for transfer to public testing:
    • P1_INITIAL_TRANSFER_TO_PUBLIC
    • P1_REMAINING_AMOUNT_TRANSFER_TO_PUBLIC
    • P1_PROOF_TRANSFER_TO_PUBLIC
  • Implement sign_transfer_to_public test method in BoilerplateCommandSender
  • Add test infrastructure for multi-part data verification:
    • Test path and header data transmission
    • Test remaining amount and transaction details
    • Test proof data chunking for large proofs
  • Include debug prints for test response validation

- Add test P1 enum values for transfer to public testing:
  - P1_INITIAL_TRANSFER_TO_PUBLIC
  - P1_REMAINING_AMOUNT_TRANSFER_TO_PUBLIC
  - P1_PROOF_TRANSFER_TO_PUBLIC
- Implement sign_transfer_to_public test method in BoilerplateCommandSender
- Add test infrastructure for multi-part data verification:
  - Test path and header data transmission
  - Test remaining amount and transaction details
  - Test proof data chunking for large proofs
- Include debug prints for test response validation
@keiff3r keiff3r requested review from a team and removed request for a team December 10, 2024 12:33
@keiff3r keiff3r requested a review from a team December 10, 2024 12:51
@n4l5u0r n4l5u0r merged commit e8bdebc into main Dec 10, 2024
29 of 30 checks passed
@n4l5u0r n4l5u0r deleted the test/LDG-517--nano-app-implement-transfertopublic-test branch December 10, 2024 12:55
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.

2 participants