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

Enhance test_starknet_ics20_contract to fully simulate relaying from Cosmos to Starknet #170

Open
Tracked by #123
Farhad-Shabani opened this issue Jan 7, 2025 · 0 comments · May be fixed by #172
Open
Tracked by #123
Assignees
Milestone

Comments

@Farhad-Shabani
Copy link
Member

Farhad-Shabani commented Jan 7, 2025

Context

In the test_starknet_ics20_contract test, once the Cosmos and Starknet chains are connected (via the handshake process), a dummy msg_recv_packet is created. To fully simulate the relaying of a packet from Cosmos to Starknet, the msg_recv_packet should be constructed based on a previously submitted MsgTransfer to the Cosmos chain.

Additionally, the packet should be acknowledged on the Cosmos chain and should cover two scenarios: first, when the incoming token into Starknet is an IBC-minted token, and second, when it is a native token previously sent to Cosmos and now being unescrowed.

@Farhad-Shabani Farhad-Shabani added this to the MVP Demo #2 milestone Jan 7, 2025
@Farhad-Shabani Farhad-Shabani changed the title Replace dummy msg_recv_packet with actual message in test_starknet_ics20_contract Enhance test_starknet_ics20_contract to fully simulate packet relay from Cosmos to Starknet Jan 7, 2025
@Farhad-Shabani Farhad-Shabani changed the title Enhance test_starknet_ics20_contract to fully simulate packet relay from Cosmos to Starknet Enhance test_starknet_ics20_contract to fully simulate relaying from Cosmos to Starknet Jan 7, 2025
@rnbguy rnbguy self-assigned this Jan 8, 2025
@rnbguy rnbguy linked a pull request Jan 8, 2025 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants