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

Launch Chopsticks forking Kusama and an indexer pointed at it in tests #436

Merged
merged 47 commits into from
Nov 29, 2023

Conversation

Tbaut
Copy link
Collaborator

@Tbaut Tbaut commented Nov 9, 2023

closes #394

The CI now has

  • a Postgres for the indexer
  • Chopsticks funding 4 accounts and forking Kusama
  • Subsquid indexer pointing at Chopsticks, and running alongside a graphQL server
  • a minimal test with the full creation of a multisig, with proxy, with 3 of the funded accounts

We need to make this test encompass many more scenarii.
Unfortunately, all these things running in parallel means that the output in CI is very messy.
To test locally, I also made sure we have a simple command. It's npm run start:chopsticks-test-build-and-launch-all which will (quoting the README)

# Launch the full setup. It will:
# - delete any previous Chopsticks database
# - launch a docker with the indexer db
# - launch a Chopsticks node
# - build and launch a subsquid indexer pointed at Chopsticks
# - launch a graphQL server for the squid
# - launch the UI with local env pointed at Chopsticks

@Tbaut Tbaut changed the title Launch Chopstick forking Kusama and an indexer pointed at it in tests Launch Chopsticks forking Kusama and an indexer pointed at it in tests Nov 28, 2023
Copy link
Member

@asnaith asnaith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left a few minor comments that's all.

This is great, the closest we can get to live testing and with no mocks. 🙌

Thank you for all the effort with this, it's well worth it 🚀

.github/workflows/cypress-tests.yaml Outdated Show resolved Hide resolved
Copy link
Collaborator

@Lykhoyda Lykhoyda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for such a great improvement!

@Tbaut Tbaut merged commit 857db39 into main Nov 29, 2023
8 checks passed
@Tbaut Tbaut deleted the tbaut-chopsticks-ci branch November 29, 2023 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Experiment running our own fork of any network in the CI with Chopsticks
4 participants