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

Support multiple asserts of same type #77

Closed
wants to merge 5 commits into from

Conversation

evilrobot-01
Copy link
Contributor

@evilrobot-01 evilrobot-01 commented Nov 7, 2022

  • rebased to update after checker implementation
  • added support for multiple asserts of the same type (key) by changing asserts from YamlMap to YamlSeq
  • added tests to ensure support works list based asserts (yarn zombienet-test -t tests -c tests/config.toml) as well as a custom assert
  • added missing isNone assert definition to checker interface definition in constants.ts
  • updated the readme to used dashed list syntax

@evilrobot-01 evilrobot-01 requested a review from NachoPal November 7, 2022 10:47
@evilrobot-01 evilrobot-01 linked an issue Nov 7, 2022 that may be closed by this pull request
@evilrobot-01 evilrobot-01 linked an issue Nov 11, 2022 that may be closed by this pull request
Frank Bell added 4 commits January 3, 2023 16:10
Also fixed AssertAction name typo and added missing isNone assert to checker interface in constants.
Only a single collator required.
Only required during debugging.
@evilrobot-01 evilrobot-01 force-pushed the frank/fix-assert-list branch from bcf7aad to 1ea77e4 Compare January 3, 2023 16:13
@evilrobot-01
Copy link
Contributor Author

@NachoPal rebased and updated to use sequences for asserts, along with tests and some typo fixes.

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.

Support multiple asserts of same type
1 participant