Skip to content

Add test cases to test vector #59

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add test cases to test vector #59

wants to merge 2 commits into from

Conversation

@cygnet3
Copy link
Owner

@cygnet3 cygnet3 commented Sep 23, 2023 •
edited
Loading

Two changes to the test vectors:

  • Make the 'multiple outputs, same recipient' test have 3 outputs instead of 2
  • Add a test case where same recipient receives twice for un-labeled address & twice for labeled address

@cygnet3 cygnet3 force-pushed the add-test-cases branch 3 times, most recently from 425dfb7 to 35b41a7 Compare September 23, 2023 22:01
@Sosthene00
Copy link
Contributor

Sosthene00 commented Sep 24, 2023

tACK

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

Reviewers

1 more reviewer

@QuannWright QuannWright QuannWright approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@cygnet3 @Sosthene00 @QuannWright