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
Fix 2457 #2731
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.
😋
} | ||
} | ||
} | ||
|
||
/* | ||
// NOTE: This is too hard to test without | ||
// an easy way to add test peer to switch | ||
// or without significant refactoring of the module. | ||
// Alternatively we could actually dial a TCP conn but | ||
// that seems extreme. | ||
func TestBadBlockStopsPeer(t *testing.T) { |
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.
these tests remind me of our consensus tests - hard to read, hard to modify, hard to debug
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.
Agree, will have to do for now. In future we'll need to restructure the reactors for better testing.
Codecov Report
@@ Coverage Diff @@
## develop #2731 +/- ##
===========================================
+ Coverage 62.19% 62.75% +0.55%
===========================================
Files 212 212
Lines 17362 17388 +26
===========================================
+ Hits 10799 10911 +112
+ Misses 5654 5561 -93
- Partials 909 916 +7
|
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.
🥑
* develop: types: Emit tags from BeginBlock/EndBlock (tendermint#2747) Fix fast sync stack with wrong block tendermint#2457 (tendermint#2731) It's better read from genDoc than from state.validators when appHeight==0 in replay (tendermint#2893) update encoding spec (tendermint#2903) return back initially allowed level if we encounter allowed key (tendermint#2889) Handling integer IDs in JSON-RPC requests -- fixes tendermint#2366 (tendermint#2811) # Conflicts: # blockchain/reactor_test.go # blockchain/store_test.go # node/node_test.go # types/events.go
Replaces #2621
ref #2457