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

Chopsticks tests initial set-up and documentation #6227

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

x3c41a
Copy link
Contributor

@x3c41a x3c41a commented Oct 24, 2024

Created an initial version of chopstick tests structure with the corresponding folders and READMEs.
Added an example of chopsticks tests for Trusted Query API - PR6212
I went through my instructions in a new workspace and made sure that they work

The purpose of the folder is to serve as a reference documentation and a central place for all the tests with chopsticks.
Eventually these tests will be integrated into CI/CD pipeline and will be run as a part of the release process.

@x3c41a x3c41a marked this pull request as draft October 24, 2024 18:01
@bkchr
Copy link
Member

bkchr commented Oct 24, 2024

What you want to achieve with this? (not just say "writing tests with chopsticks")

@x3c41a x3c41a changed the title WIP: Chopsticks tests set-up and documentation Chopsticks tests initial set-up and documentation Oct 28, 2024
@x3c41a
Copy link
Contributor Author

x3c41a commented Oct 28, 2024

What you want to achieve with this? (not just say "writing tests with chopsticks")

I believe this comment is obsolete since it concerned a draft PR...

@x3c41a x3c41a marked this pull request as ready for review October 29, 2024 07:41
@x3c41a x3c41a marked this pull request as draft October 29, 2024 07:41
@x3c41a x3c41a added the R0-silent Changes should not be mentioned in any release notes label Oct 29, 2024
@x3c41a x3c41a marked this pull request as ready for review October 29, 2024 08:59
@x3c41a x3c41a changed the base branch from master to xcm-v5 October 29, 2024 10:24
@x3c41a x3c41a requested review from a team as code owners October 29, 2024 10:24
@paritytech-review-bot paritytech-review-bot bot requested a review from a team October 29, 2024 10:25
@x3c41a x3c41a changed the base branch from xcm-v5 to master October 29, 2024 10:31
@x3c41a x3c41a removed request for a team October 29, 2024 10:34
@bkchr
Copy link
Member

bkchr commented Nov 5, 2024

We have the ecosystem test. Tests should be put there. Also they provide a lot of infrastructure, setup code etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants