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

adds multisig RPC integration test #4681

Merged
merged 4 commits into from
Feb 7, 2024
Merged

Conversation

hughy
Copy link
Contributor

@hughy hughy commented Feb 5, 2024

Summary

integration test replicates the wallet test, but uses RPC requests and responses

the response for each step of the flow is used as an input to the request of the next step

Testing Plan

Documentation

Does this change require any updates to the Iron Fish Docs (ex. the RPC API
Reference
)? If yes, link a
related documentation pull request for the website.

[ ] Yes

Breaking Change

Is this a breaking change? If yes, add notes below on why this is breaking and
what additional work is required, if any.

[ ] Yes

@hughy hughy requested a review from a team as a code owner February 5, 2024 23:25
@hughy hughy force-pushed the chore/hughy/rpc-integration-test branch from c64efd3 to 6c2eb89 Compare February 6, 2024 04:20
@hughy hughy marked this pull request as draft February 6, 2024 19:51
@hughy hughy marked this pull request as ready for review February 6, 2024 21:15
@hughy hughy force-pushed the chore/hughy/rpc-integration-test branch from 6c2eb89 to 5d35df5 Compare February 6, 2024 23:34
integration test replicates the wallet test, but uses RPC requests and responses

the response for each step of the flow is used as an input to the request of the
next step
@hughy hughy force-pushed the chore/hughy/rpc-integration-test branch from 5d35df5 to 45b17e6 Compare February 7, 2024 00:19
@hughy hughy merged commit a10c9b4 into staging Feb 7, 2024
9 checks passed
@hughy hughy deleted the chore/hughy/rpc-integration-test branch February 7, 2024 17:27
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