Skip to content

✅ Add XCM Transact emulator tests #480

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

Merged
merged 1 commit into from
May 2, 2025

Conversation

dastansam
Copy link
Contributor

@dastansam dastansam commented Apr 30, 2025

This pull request introduces several changes to the integration tests and runtime configurations, primarily focused on replacing rococo-runtime with westend-runtime and adding support for the AssetHubWestend parachain. The changes include updates to dependencies, constants, runtime configurations, and the addition of new test cases for cross-chain transactions.

Key Changes:

Transition from rococo-runtime to westend-runtime:

  • Updated dependencies in Cargo.toml files to replace rococo-runtime and rococo-runtime-constants with westend-runtime and westend-runtime-constants (Cargo.toml, integration-tests/Cargo.toml). [1] [2] [3] [4] [5]
  • Updated runtime constants and session keys in integration-tests/src/constants.rs to use westend-runtime equivalents. [1] [2] [3]

Addition of AssetHubWestend support:

  • Introduced a new module asset_hub in integration-tests/src/constants.rs to define the genesis configuration and constants for AssetHubWestend.
  • Declared the AssetHubWestend parachain in integration-tests/src/lib.rs, including its runtime, core, and pallets. Helper macros for accounts, assets, and XCM were also implemented.
  • Added AssetHubWestend to the test network (PolkadotNet) and updated shortcuts for runtime types to include AssetHubWestend. [1] [2]

New XCM transaction test:

  • Added a new test module transact.rs to validate Cross-Consensus Messaging (XCM) between AssetHubWestend and Polimec. The test ensures that a Polimec Remark call can be successfully sent from AssetHubWestend to Polimec.

Miscellaneous updates:

  • Updated integration-tests/src/lib.rs to include the new asset_hub module and helper imports.
  • Enabled the new transact test module in integration-tests/src/tests/mod.rs.

These changes collectively enhance the integration test framework by introducing support for the westend-runtime and extending the test coverage to include the AssetHubWestend parachain.

@lrazovic lrazovic changed the base branch from 04-30-fix_correct_the_migrations to graphite-base/480 April 30, 2025 15:33
@lrazovic lrazovic force-pushed the 04-30-add_transact_xcm_emulator_tests branch from 2a2cdfc to 99082c4 Compare May 1, 2025 13:26
@lrazovic lrazovic force-pushed the graphite-base/480 branch from 575b593 to 8ee104c Compare May 1, 2025 13:26
@lrazovic lrazovic changed the base branch from graphite-base/480 to 04-30-fix_correct_the_migrations May 1, 2025 13:26
@lrazovic lrazovic force-pushed the 04-30-add_transact_xcm_emulator_tests branch 2 times, most recently from 96fb48b to ec82039 Compare May 1, 2025 13:34
@lrazovic lrazovic force-pushed the 04-30-fix_correct_the_migrations branch from 8ee104c to 296dcb6 Compare May 1, 2025 13:53
@lrazovic lrazovic force-pushed the 04-30-add_transact_xcm_emulator_tests branch from ec82039 to 6d5cec7 Compare May 1, 2025 13:53
@lrazovic lrazovic changed the title Add transact xcm emulator tests ✅ Add Transact XCM emulator tests May 1, 2025
@lrazovic lrazovic changed the title ✅ Add Transact XCM emulator tests ✅ Add XCM Transact emulator tests May 1, 2025
@dastansam dastansam force-pushed the 04-30-fix_correct_the_migrations branch from 296dcb6 to 91aa6fe Compare May 1, 2025 15:54
@dastansam dastansam force-pushed the 04-30-add_transact_xcm_emulator_tests branch from 6d5cec7 to 53c798e Compare May 1, 2025 15:54
@dastansam dastansam marked this pull request as ready for review May 1, 2025 15:54
@dastansam dastansam requested a review from lrazovic May 1, 2025 15:55
Copy link

graphite-app bot commented May 1, 2025

Graphite Automations

"Auto-assign PRs to author" took an action on this PR • (05/01/25)

1 assignee was added to this PR based on Juan Ignacio Rios's automation.

@lrazovic lrazovic force-pushed the 04-30-fix_correct_the_migrations branch from 91aa6fe to 296dcb6 Compare May 2, 2025 07:41
@lrazovic lrazovic force-pushed the 04-30-add_transact_xcm_emulator_tests branch from 53c798e to 3e6e119 Compare May 2, 2025 07:41
Copy link
Member

lrazovic commented May 2, 2025

Merge activity

  • May 2, 7:23 AM EDT: A user started a stack merge that includes this pull request via Graphite.
  • May 2, 7:40 AM EDT: Graphite rebased this pull request as part of a merge.
  • May 2, 7:40 AM EDT: @lrazovic merged this pull request with Graphite.

@lrazovic lrazovic changed the base branch from 04-30-fix_correct_the_migrations to graphite-base/480 May 2, 2025 11:37
@lrazovic lrazovic changed the base branch from graphite-base/480 to main May 2, 2025 11:38
@lrazovic lrazovic force-pushed the 04-30-add_transact_xcm_emulator_tests branch from 3e6e119 to 25b6f1e Compare May 2, 2025 11:39
@lrazovic lrazovic merged commit d61ebb7 into main May 2, 2025
1 check passed
@lrazovic lrazovic deleted the 04-30-add_transact_xcm_emulator_tests branch May 2, 2025 11:40
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.

2 participants