polkadot-runtime-template icon indicating copy to clipboard operation
polkadot-runtime-template copied to clipboard

Make xcm-simulator mocks match runtimes

Open 4meta5 opened this issue 1 year ago • 1 comments

Follow up issue for #363 raised during its review

  1. use XCM abstractions for all mock configs
  2. add tests to ensure that the XCM config for the mock runtimes match the actual runtime configs exactly

4meta5 avatar Nov 26 '24 18:11 4meta5

Requires declaring these pallets in xcm_tests::parachain::construct_runtime!:

  • pallet-message-queue
  • cumulus_pallet_xcmp_queue
  • XcmExecutorConfig, xcm_executor::Config
  • cumulus_pallet_xcm
  • pallet_xcm_weight_trader
  • orml_xtokens
  • pallet_xcm_transactor

The pallets listed in the previous comment could be functionally excluded while still using the macros if https://github.com/OpenZeppelin/openzeppelin-pallet-abstractions/issues/15 and https://github.com/OpenZeppelin/openzeppelin-pallet-abstractions/issues/40 were implemented and closed.

However, those issues do not block this one if we just include the pallets listed above. Including them is sensible because the goal of this issue is to better match the actual runtimes when it comes to the XCM configuration.

4meta5 avatar Dec 09 '24 16:12 4meta5