Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

Add additional integration tests #76

Closed
wants to merge 1 commit into from

Conversation

evilrobot-01
Copy link
Contributor

Adds an integration test for an asset teleport, based on XCM workshop slides.

Currently blocked by the following pull requests, which enable additional checks on events, queries and asserts:

@bernardoaraujor
Copy link
Contributor

Perhaps this PR could also cover #80 ?
That way, all the cases present in the XCM workshop slides would be covered.

@evilrobot-01
Copy link
Contributor Author

Thats what I was hopefully working towards to tackle #53, basing it on the test you originally wrote so I could get familiar with the process and inner workings, but then also have something formal in place so we can easily check things over time as Trappist evolves (or Polkadot is updated).

Sounds like you may have solved #53 with #80, so will take a look at it next.

FYI - for reference, my implementation of the reserve-transfer test based on the XCM workshop slides is currently at https://github.com/paritytech/parachains-integration-tests/blob/frank/trappist/examples/trappist/1_reserve_transfer.yml, which is effectively now the same as yours after learning of the asset registry last night. 🙂

@bernardoaraujor
Copy link
Contributor

bernardoaraujor commented Nov 10, 2022

I had forgotten about #53, perhaps #80 could have simply been appended to it as comments. My bad.

Anyways, I don't have a working solution for #80 yet, feel free to go for it!

In reference to #53, I'd suggest to add chain = "penpal-kusama-3000" to xcm-playground.toml (ref)

that should be a simple solution to the issue of "Rococo Parachain Local" (runtime currently being loaded into parachain 3000) not having an XCM Config.

@bernardoaraujor
Copy link
Contributor

after #78 I think chain = "penpal-kusama-3000" is no longer necessary, since parachain 3000 now has its own runtime with a proper XCM Config.

@evilrobot-01
Copy link
Contributor Author

Working with base-collator from latest master, thanks!

@evilrobot-01 evilrobot-01 changed the title Add asset teleport test Add additional integration tests Nov 16, 2022
@evilrobot-01 evilrobot-01 mentioned this pull request Jan 7, 2023
@hbulgarini
Copy link
Contributor

@stiiifff what should we do with this PR that got opened for quite a while?

@evilrobot-01
Copy link
Contributor Author

I'll close it, as it was dependent on some improvements to the parachains-integration-tests tool which never got merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants