Skip to content

Snowbridge - V1 - Adds 2 hop transfer to Rococo#7956

Merged
acatangiu merged 26 commits intoparitytech:masterfrom
claravanstaden:2-hop-transfer-to-rococo
Apr 25, 2025
Merged

Snowbridge - V1 - Adds 2 hop transfer to Rococo#7956
acatangiu merged 26 commits intoparitytech:masterfrom
claravanstaden:2-hop-transfer-to-rococo

Conversation

@claravanstaden
Copy link
Copy Markdown
Contributor

@claravanstaden claravanstaden commented Mar 18, 2025

Adds a test to demonstrate a 2-hop (Ethereum to Westend AH, second transaction from Westend AH to Rococo AH) transfer to Rococo AH (and back to Ethereum in a 2-hop fashion as well).

# Conflicts:
#	cumulus/parachains/integration-tests/emulated/chains/parachains/assets/asset-hub-rococo/src/genesis.rs
#	cumulus/parachains/integration-tests/emulated/chains/parachains/assets/asset-hub-westend/src/genesis.rs
#	cumulus/parachains/integration-tests/emulated/common/src/lib.rs
#	cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge.rs
@claravanstaden claravanstaden marked this pull request as ready for review March 26, 2025 07:15
@claravanstaden claravanstaden changed the title Snowbridge - Adds 2 hop transfer to Rococo Snowbridge - V1 - Adds 2 hop transfer to Rococo [Draft] Mar 26, 2025
@claravanstaden claravanstaden marked this pull request as draft March 26, 2025 09:34
@claravanstaden claravanstaden changed the title Snowbridge - V1 - Adds 2 hop transfer to Rococo [Draft] Snowbridge - V1 - Adds 2 hop transfer to Rococo Apr 16, 2025
@claravanstaden claravanstaden marked this pull request as ready for review April 16, 2025 08:25
Copy link
Copy Markdown
Contributor

@acatangiu acatangiu left a comment

Choose a reason for hiding this comment

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

Code changes look good, but some tests are failing - without running them to debug, and by just looking at diff, my best guess is WETH registration in AHR genesis maybe conflicting with explicit registration attempts from older tests

@claravanstaden
Copy link
Copy Markdown
Contributor Author

Requires labels R0-silent T10-tests.

@acatangiu acatangiu added R0-no-crate-publish-required The change does not require any crates to be re-published. T10-tests This PR/Issue is related to tests. labels Apr 23, 2025
@acatangiu acatangiu enabled auto-merge April 23, 2025 10:43
auto-merge was automatically disabled April 23, 2025 12:14

Head branch was pushed to by a user without write access

# Conflicts:
#	cumulus/parachains/integration-tests/emulated/chains/parachains/assets/asset-hub-rococo/src/genesis.rs
#	cumulus/parachains/integration-tests/emulated/chains/parachains/assets/asset-hub-westend/src/genesis.rs
#	cumulus/parachains/integration-tests/emulated/common/src/lib.rs
#	cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/asset_transfers.rs
#	cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/mod.rs
#	cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/register_bridged_assets.rs
#	cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge.rs
#	cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge_edge_case.rs
#	cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/transact.rs
@acatangiu acatangiu added this pull request to the merge queue Apr 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 24, 2025
@acatangiu
Copy link
Copy Markdown
Contributor

@claravanstaden this needs rebase and fixing some imports since your other PR got it that moved some stuff around

@acatangiu
Copy link
Copy Markdown
Contributor

maybe you can also address #8289 (comment) in this PR

@claravanstaden
Copy link
Copy Markdown
Contributor Author

maybe you can also address #8289 (comment) in this PR

@acatangiu added a comment over here: #8289 (comment)

@acatangiu acatangiu added this pull request to the merge queue Apr 25, 2025
Merged via the queue into paritytech:master with commit 8da42e8 Apr 25, 2025
233 of 243 checks passed
wassimans pushed a commit to wassimans/polkadot-sdk that referenced this pull request Apr 27, 2025
Adds a test to demonstrate a 2-hop (Ethereum to Westend AH, second
transaction from Westend AH to Rococo AH) transfer to Rococo AH (and
back to Ethereum in a 2-hop fashion as well).
ordian added a commit that referenced this pull request Apr 28, 2025
* master: (120 commits)
  [CI] Improve GH build status checking (#8331)
  [CI/CD] Use original PR name in prdoc check for the backport PR's to the stable branches (#8329)
  Add new host APIs set_storage_or_clear and get_storage_or_zero (#7857)
  push to dockerhub (#8322)
  Snowbridge - V1 - Adds 2 hop transfer to Rococo (#7956)
  [AHM] Prepare `election-provider-multi-block` for full lazy data deletion (#8304)
  Check umbrella version (#8250)
  [AHM] Fully bound staking async (#8303)
  migrate parachain-templates tests to `gha` (#8226)
  staking-async: add missing new_session_genesis (#8310)
  New NFT traits: granular and abstract interface (#5620)
  Extract create_pool_with_native_on macro to common crate (#8289)
  XCMP: use batching when enqueuing inbound messages (#8021)
  Snowbridge - Tests refactor (#8014)
  Allow configuration of worst case buy execution weight (#7944)
  Fix faulty pre-upgrade migration check in pallet-session (#8294)
  [pallet-revive] add get_storage_var_key for variable-sized keys (#8274)
  add poke_deposit extrinsic to pallet-recovery (#7882)
  `txpool`: use tracing for structured logging (#8001)
  [revive] eth-rpc refactoring (#8148)
  ...
@claravanstaden claravanstaden deleted the 2-hop-transfer-to-rococo branch April 29, 2025 07:54
castillax pushed a commit that referenced this pull request May 12, 2025
Adds a test to demonstrate a 2-hop (Ethereum to Westend AH, second
transaction from Westend AH to Rococo AH) transfer to Rococo AH (and
back to Ethereum in a 2-hop fashion as well).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published. T10-tests This PR/Issue is related to tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants