Skip to content

Add revive substrate runtime-api integration tests for call & instantiate #10919

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
Open
pgherveou wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add revive substrate runtime-api integration tests for call & instantiate #10919

pgherveou wants to merge 4 commits into from

Conversation

@pgherveou
Copy link
Contributor

@pgherveou pgherveou commented Jan 28, 2026

Summary

  • Add integration tests for revive runtime API
  • Test Fibonacci contract deployment and execution via substrate APIs

Changes

  • Add test for Fibonacci contract call via runtime API
  • Add test to verify large Fibonacci values run out of gas as expected
  • Update dev-node runtime configuration for testing

Test plan

  • Run new integration tests
  • Verify runtime API correctly handles contract deployment
  • Verify gas limits are enforced correctly

@pgherveou pgherveou changed the title Add revive runtime-api integration tests Add revive substrate runtime-api integration tests for call & instantiate Jan 28, 2026
@pgherveou
Copy link
Contributor Author

pgherveou commented Jan 28, 2026

/cmd prdoc --audience runtime_dev --bump patch

@pgherveou pgherveou added the T7-smart_contracts This PR/Issue is related to smart contracts. label Jan 28, 2026
@pgherveou pgherveou requested a review from marian-radu January 28, 2026 08:59
@pgherveou pgherveou marked this pull request as ready for review January 28, 2026 08:59
@paritytech-workflow-stopper
Copy link

paritytech-workflow-stopper bot commented Jan 28, 2026

All GitHub workflows were cancelled due to failure one of the required jobs.
Failed workflow url: https://github.com/paritytech/polkadot-sdk/actions/runs/21431772765
Failed job name: cargo-clippy

@pgherveou @claude
Fix test_fibonacci_call_via_runtime_api signature
428e599 
Remove unused client parameter from test function to match the
test macro which calls tests without arguments.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@cla-bot-2021
Copy link

cla-bot-2021 bot commented Jan 28, 2026

User @claude, please sign the CLA here.

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

Reviewers

@marian-radu marian-radu marian-radu approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

T7-smart_contracts This PR/Issue is related to smart contracts.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@pgherveou @marian-radu