Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

[Test] Integration tests for Statemint via XCM-Emulator #2440

Closed
wants to merge 2 commits into from

Conversation

ruseinov
Copy link
Contributor

@ruseinov ruseinov commented Apr 10, 2023

WORK IN PROGRESS

Needs to be rebased once the below PR has been merged, then we can get rid of patch overrides as xcm-emulator is now cumulus-based.

Depends on: #2447.
Note, that this does not work without path overrides, until the above PR has been merged, due to the fact that we import Polkadot master across the board in cumulus, but xcm-emulator does not exist there yet.

This is an initial take on writing XCM integration tests for live runtimes via XCM-Emulator. In this case Statemint and Polkadot.

Current trade-offs
integration-tests is a separate module as opposed to something that lives under specific runtime's directory, to avoid runtime recompilation when tests change. That saves a lot of dev time.

@ruseinov ruseinov added D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit. B1-note_worthy Changes should be noted in the release notes labels Apr 10, 2023
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: test-linux-stable
Logs: https://gitlab.parity.io/parity/mirrors/cumulus/-/jobs/2660781

@ruseinov
Copy link
Contributor Author

@gilescope this can be used as reference going forward, closing this PR for now.

@ruseinov ruseinov closed this Apr 12, 2023
@ruseinov
Copy link
Contributor Author

cc @NachoPal - some initial ideas on folder structure.

This must be rebased against xcm-emulator branch in cumulus, then there won't be any more trade-offs.
Furthermore after implementing all the tests we can try and make it more DRY by introducing helpers/macros. And possibly making similar tests runtime-agnostic.

@ruseinov ruseinov mentioned this pull request Apr 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B1-note_worthy Changes should be noted in the release notes D2-notlive 💤 PR contains changes in a runtime directory that is not deployed to a chain that requires an audit.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants