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

[SPIKE] integration test of smart_rollup_execute_outbox_message #2815

Open
hui-an-yang opened this issue Feb 6, 2024 · 2 comments
Open

[SPIKE] integration test of smart_rollup_execute_outbox_message #2815

hui-an-yang opened this issue Feb 6, 2024 · 2 comments
Assignees
Labels
testing🧪 Work related to testing, manual or automated

Comments

@hui-an-yang
Copy link
Collaborator

Subsequence ticket after user's contribution pr2807 that supports smart_rollup_execute_outbox_message op in contractAPI, we'd like to add integration test possibly using etherlink on ghostnet sr1A5txMPrpjY9d4qCUq1Ke4PGBQ2WNiB7LJ.

@hui-an-yang hui-an-yang added this to the v19.2 milestone Mar 1, 2024
@hui-an-yang hui-an-yang added the testing🧪 Work related to testing, manual or automated label Mar 1, 2024
@hui-an-yang hui-an-yang self-assigned this Mar 1, 2024
@hui-an-yang hui-an-yang changed the title Add integration test of smart_rollup_execute_outbox_message [SPIKE] integration test of smart_rollup_execute_outbox_message Mar 1, 2024
@hui-an-yang
Copy link
Collaborator Author

this test will require making recursive call curl -i "${ROLLUP_NODE_ENDPOINT}/global/block/cemented/outbox/${L}/messages" to a rollup node to get the level that message is cemented (40 blocks after smart_rollup_add_messages).
Then another rpc call to rollup node curl -i "${ROLLUP_NODE_ENDPOINT}/global/block/head/helpers/\ proofs/outbox/${L}/messages?index=0 to get the proof and commitment hash.
In order to form smart_rollup_execute_outbox_message operation request parameter

@hui-an-yang hui-an-yang modified the milestones: v19.2, NEXT Mar 21, 2024
@hui-an-yang
Copy link
Collaborator Author

new discovery, only the rollup owner can execute the outbox message hence adding integration test will also need to have a rollup node running will difficult to test in testnet integration test bump to low priority in backlog

@hui-an-yang hui-an-yang removed this from the NEXT milestone Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing🧪 Work related to testing, manual or automated
Projects
Status: Low priority
Development

No branches or pull requests

1 participant