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.

Sign up for GitHub

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

Jump to bottom

Iv thea tests #748

Merged
merged 29 commits into from
May 16, 2023
Merged

Iv thea tests #748

merged 29 commits into from
May 16, 2023

Conversation

35359595
Copy link
Contributor

@35359595 35359595 commented May 15, 2023 •
edited

Describe your changes

Thea client testsing based on sp_network

Issue ticket number and link

717 of Thea

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • I removed all Clippy and Formatting Warnings.
  • I added required Copyrights.

35359595 and others added 26 commits April 20, 2023 12:03
@35359595
some warnings fixes
a5f952f
@35359595
formatting applied
6e87f62
@35359595
partial grandpa
08f43c9
@35359595
partial grandpa finality
68a56a7
@35359595
grandpa peers tests failing
2524ded
@35359595
finality works
dac4368
@35359595
all tests pass
32e22eb
@35359595
extensions
23f8f85
@35359595
Merge branch 'Develop' of github.com:Polkadex-Substrate/Polkadex into…
ab0a02e 
… iv-thea-tests
@35359595
log renamed to thea target in thea worker
5b3ec66
@35359595
manual gosip in test
0faccec
@35359595
assertion fixes
92733bb
@35359595
joined networks, grandpa errors
f6edb80
@35359595
keys are not injected?
a1d6630
@35359595
worker loop unblocked, still no gossiping
65a4a97
@35359595
Merge branch 'Develop' of github.com:Polkadex-Substrate/Polkadex into…
712f31a 
… iv-thea-tests
@35359595
one dropped validator test works with gossip network
520ccc9
@35359595
actually dropping one validator
a91e212
@35359595
test timeout reduced + ocex test fix
cba76fb
@35359595
post-merge fixes and partial build issues
4715cb5
@35359595
warnings fixes
db18498
@35359595
formatting and minor fixes
5b8e256
@35359595
initial set id change test
0eab8a3
@35359595
flow updates
6c6f8a3
@Gauthamastro
Update client RPC
39eeeff
@Gauthamastro
Fixed the RPC error
9e0eb35
@35359595 35359595 self-assigned this May 15, 2023
This was referenced May 16, 2023
Merged
Merged
@35359595 35359595 added the A0-PleaseReview Pull request needs code review. label May 16, 2023
@35359595
Copy link
Contributor Author

Thea tests (automated) are commented out temporarily and will be fixed under #717

@Gauthamastro Gauthamastro merged commit 222e637 into Develop May 16, 2023
@Gauthamastro Gauthamastro deleted the iv-thea-tests branch May 16, 2023 10:07
@Gauthamastro Gauthamastro mentioned this pull request May 16, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauthamastro Gauthamastro Awaiting requested review from Gauthamastro

@serhii-temchenko serhii-temchenko Awaiting requested review from serhii-temchenko

Assignees

@35359595 35359595

Labels
A0-PleaseReview Pull request needs code review.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@35359595 @Gauthamastro @serhii-temchenko