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

add : TestValidatorWentOffline #507

Merged
merged 10 commits into from
Sep 5, 2022
Merged

add : TestValidatorWentOffline #507

merged 10 commits into from
Sep 5, 2022

Conversation

0xsharma
Copy link
Contributor

In this PR, we add a red test where we test a validator coming back online after a crash and reorging their backup blocks generated by the secondary validator.

@codecov-commenter
Copy link

Codecov Report

Base: 56.93% // Head: 56.85% // Decreases project coverage by -0.07% ⚠️

Coverage data is based on head (a2dc545) compared to base (9c370f4).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #507      +/-   ##
===========================================
- Coverage    56.93%   56.85%   -0.08%     
===========================================
  Files          610      607       -3     
  Lines        69999    69976      -23     
===========================================
- Hits         39854    39785      -69     
- Misses       26748    26788      +40     
- Partials      3397     3403       +6     
Impacted Files Coverage Δ
node/defaults.go 24.13% <0.00%> (-6.90%) ⬇️
p2p/simulations/mocker.go 30.00% <0.00%> (-5.56%) ⬇️
rpc/server.go 72.50% <0.00%> (-5.00%) ⬇️
les/distributor.go 79.68% <0.00%> (-3.13%) ⬇️
rlp/typecache.go 89.53% <0.00%> (-2.33%) ⬇️
trie/proof.go 73.59% <0.00%> (-2.12%) ⬇️
accounts/keystore/account_cache.go 90.20% <0.00%> (-2.10%) ⬇️
consensus/ethash/ethash.go 54.19% <0.00%> (-1.53%) ⬇️
miner/worker.go 70.94% <0.00%> (-1.49%) ⬇️
consensus/ethash/algorithm.go 91.44% <0.00%> (-1.32%) ⬇️
... and 18 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

tests/bor/bor_reorg_test.go Outdated Show resolved Hide resolved
@0xsharma 0xsharma merged commit 85890fe into develop Sep 5, 2022
@0xsharma 0xsharma deleted the shiva/pos-730 branch September 27, 2022 06:01
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.

5 participants