Skip to content

test: add integration tests for foreign chain requests - #2820

Merged
gilcu3 merged 11 commits into
mainfrom
dsharifi/integration-tests-foreign-chain-requests
Apr 13, 2026
Merged

test: add integration tests for foreign chain requests#2820
gilcu3 merged 11 commits into
mainfrom
dsharifi/integration-tests-foreign-chain-requests

Conversation

@DSharifi

@DSharifi DSharifi commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

closes #2821

@DSharifi
DSharifi marked this pull request as ready for review April 10, 2026 11:44
@DSharifi

Copy link
Copy Markdown
Contributor Author

@claude review

@near near deleted a comment from claude Bot Apr 10, 2026
@near near deleted a comment from claude Bot Apr 10, 2026
@DSharifi

Copy link
Copy Markdown
Contributor Author

@claude review

@claude

claude Bot commented Apr 10, 2026

Copy link
Copy Markdown

Code Review

No critical issues found. This is a clean, test-only PR that adds integration tests for foreign chain requests covering three important scenarios:

  1. Happy path — submit request, sign response, verify result
  2. Policy rejection — request fails without a chain policy configured
  3. Timeout — request times out when no response is submitted

Good use of rstest parameterization to cover all supported chains (Ethereum, Abstract, Bitcoin, Starknet, BNB) without duplicating test logic.

✅ Approved

Generated by Claude Code

@DSharifi
DSharifi enabled auto-merge April 10, 2026 12:25
anodar
anodar previously approved these changes Apr 10, 2026
Comment thread crates/contract/tests/sandbox/common.rs Outdated
Comment thread crates/contract/tests/sandbox/common.rs
anodar
anodar previously approved these changes Apr 10, 2026

@gilcu3 gilcu3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

In the current PR we are adding duplicate code, and not following the current code organization. We should sort these out before merging.

Comment thread crates/contract/tests/sandbox/common.rs
Comment thread crates/contract/tests/sandbox/common.rs
Comment thread crates/contract/tests/sandbox/common.rs
gilcu3
gilcu3 previously approved these changes Apr 13, 2026

@gilcu3 gilcu3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@DSharifi
DSharifi added this pull request to the merge queue Apr 13, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Apr 13, 2026
@DSharifi
DSharifi dismissed stale reviews from gilcu3 and anodar via b982b40 April 13, 2026 16:22
@DSharifi
DSharifi requested review from anodar and gilcu3 April 13, 2026 16:22
@gilcu3
gilcu3 added this pull request to the merge queue Apr 13, 2026
Merged via the queue into main with commit 776be28 Apr 13, 2026
27 checks passed
@gilcu3
gilcu3 deleted the dsharifi/integration-tests-foreign-chain-requests branch April 13, 2026 20:39
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.

test: add sandbox integration tests for foreign chain verification requests

3 participants