-
Notifications
You must be signed in to change notification settings - Fork 119
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
test: added suppport for version specific actions e2e test #2319
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
4e7773b
to
9cae0fa
Compare
9cae0fa
to
8388dd4
Compare
8388dd4
to
c59766e
Compare
// wait for inclusion in a block -> '--broadcast-mode block' is deprecated | ||
tr.waitBlocks(ChainID("provi"), 2, 30*time.Second) | ||
//tr.waitBlocks(ChainID("provi"), 2, 30*time.Second) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this can be deleted?
// wait for the transaction | ||
start := time.Now() | ||
for { | ||
res, err := tr.target.QueryTransaction(chain, response.TxHash) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe we can add some sleep of one second(or commit time), just to avoid querying too often
|
||
func (tr Commands) getValidatorNode(chain ChainID, validator ValidatorID) string { | ||
// for CometMock, validatorNodes are all the same address as the query node (which is CometMocks address) | ||
// TODO: @bermuell Fix this !!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this comment still valid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Closes: #2198
Refactoring of tests to support execution of tests against target versions (provider, consumer) where execution
of action differs.
action handler
and commands for v5tested with compatibility tests on versions v5.2.0 (consumer/provider) v6.1.0 (consumer/provider)
compatibility of consumer v5.x with provider version v6.2.x is broken (issue #2359)
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...