Skip to content

add test for sending asset witness functionality #77

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
Closed
Arshia-r-m wants to merge 0 commits into from
Closed

add test for sending asset witness functionality #77

Arshia-r-m wants to merge 0 commits into from

Conversation

@Arshia-r-m
Copy link
Contributor

@Arshia-r-m Arshia-r-m commented Oct 23, 2025 •
edited
Loading

This test replicate the issue on #78

@Arshia-r-m Arshia-r-m force-pushed the test-send-asset-witness branch 2 times, most recently from 5a34ed8 to d5bc83b Compare October 23, 2025 10:45
@zoedberg
Copy link
Member

zoedberg commented Oct 24, 2025

The test in this PR is failing because of an UnknownContractId error, not an InvalidRecipientData error as reported by #78. But changing the call to rgb_invoice to avoid passing the asset ID shows the issue, so thanks for opening this. I think we can just add the transfer at the end of the send_receive test, could you please do that and rebase this on top of the latest master tip? Thank you

@Arshia-r-m Arshia-r-m closed this Oct 24, 2025
@Arshia-r-m Arshia-r-m force-pushed the test-send-asset-witness branch from d5bc83b to a546002 Compare October 24, 2025 18:35
@Arshia-r-m Arshia-r-m mentioned this pull request Oct 24, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Arshia-r-m @zoedberg