search

paritytech / polkadot-launch

Simple CLI tool to launch a local Polkadot test network
MIT License
460 stars 93 forks source link

Add testing #132

Open joelamouche opened 3 years ago

joelamouche commented 3 years ago

As done in https://github.com/PureStake/moonbeam/tree/master/tests/para-tests/moonriver in the moonbeam repo and in https://github.com/PureStake/polkadot-launch/tree/moonbeam-launch/tests, in the moonbeam fork of polkadot-launch.

joelamouche commented 3 years ago

@shawntabrizi What type of tests should we have by default in that folder?

shawntabrizi commented 3 years ago

I think something basic like a reserve transfer XCM is good enough? I am sure we will build out more tests as we think of them.

Also feel free to bring over any tests which you think are not moonbeam specific :)

cc @bkchr @tomaka

joelamouche commented 3 years ago

So now we have to figure out what binaries the tests will use. At Moonbeam we download binaries from docker (only linux) and when developing localy on my mac I build polkadot nd moonbeam locally. We could do the same but with cumulus instead of moonbeam, as per the README. The question is: do we currently have a docker image of cumulus build with parachain-collator ? @shawntabrizi

joelamouche commented 3 years ago

Oh and also: does this parachain binary come with prefunded addresses?

shawntabrizi commented 3 years ago

I think we can just use the docker files of Polakdot and Cumulus?

https://github.com/paritytech/polkadot/tree/master/docker

https://github.com/paritytech/cumulus/tree/master/docker

As for funds, when running in --dev alice is always prefunded.

Let me know if I am communicating on the right frequency here :)

joelamouche commented 3 years ago

Ok great thats all the info I needed :)

joelamouche commented 3 years ago

Sorry I have been busy with other tasks but I didn't forget this!

joelamouche commented 3 years ago

@shaunxw Do the docker images linked support xcm?

joelamouche commented 3 years ago

As for funds, when running in --dev alice is always prefunded.

According to https://github.com/paritytech/cumulus, adding --alice and --bob respectively will suffice. I wonder if I need to add the force-authoring flag?

shawntabrizi commented 3 years ago

Some comments on force-authoring here: https://github.com/paritytech/polkadot-launch/pull/143

Would there be any problems with adding it?

joelamouche commented 3 years ago

Thanks! yes I could easily solve #143 at the same time