Skip to content
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

[xcm-emulator] Restructure Parachains Integration Tests #1693

Closed

Conversation

bkontur
Copy link
Contributor

@bkontur bkontur commented Sep 25, 2023

Description

Fixes #1389
Closes #1389

TODO

@bkontur bkontur added T9-cumulus This PR/Issue is related to cumulus. T14-system_parachains This PR/Issue is related to system parachains. labels Sep 25, 2023
@bkontur
Copy link
Contributor Author

bkontur commented Oct 9, 2023

closing, integration tests will be moved out and fixed elsewhere

@bkontur bkontur closed this Oct 9, 2023
bkontur added a commit that referenced this pull request Oct 11, 2023
# Desription

## Summary 

This PR introduces several nits and tweaks to xcm emulator tests for
system parachains.

## Explanation

**Deduplicate `XcmPallet::send(` with root origin code**
- Introduced `send_transact_to_parachain` which could be easily reuse
for scenarios like _governance call from relay chain to parachain_.

**Refactor `send_transact_sudo_from_relay_to_system_para_works`**
- Test covered just one use-case which was moved to the
`do_force_create_asset_from_relay_to_system_para`, so now we can extend
this test with more _governance-like_ senarios.
- Renamed to
`send_transact_as_superuser_from_relay_to_system_para_works`.

**Remove `send_transact_native_from_relay_to_system_para_fails` test**
- This test and/or description is kind of misleading, because system
paras support Native from relay chain by `RelayChainAsNative` with
correct xcm origin.
- It tested only sending on relay chain which should go directly to the
relay chain unit-tests (does not even need to be in xcm emulator level).

## Future directions

Check restructure parachains integration tests
[issue](#1389) and [PR
with more TODOs](#1693).

---------

Co-authored-by: Ignacio Palacios <ignacio.palacios.santos@gmail.com>
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Mar 26, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Mar 27, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 8, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 8, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 8, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 8, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 8, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 9, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 9, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 9, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 9, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 9, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 9, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 10, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
serban300 pushed a commit to serban300/polkadot-sdk that referenced this pull request Apr 10, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
bkchr pushed a commit that referenced this pull request Apr 10, 2024
* removed MintReward and added PayLaneRewardFromAccount

* clippy

* added issue ref to TODO
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T9-cumulus This PR/Issue is related to cumulus. T14-system_parachains This PR/Issue is related to system parachains.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant