2a24ae90c1
fixes from Jae's review
...
1. remove pointer
2. add Quit() method to Service interface
2018-02-12 14:32:09 +04:00
11b68f1934
rewrite broadcastTxRoutine to use channels
...
https://play.golang.org/p/gN21yO9IRs3
```
func waitWithCancel(f func() *clist.CElement, ctx context.Context) *clist.CElement {
el := make(chan *clist.CElement, 1)
select {
case el <- f():
```
will just run f() blockingly, so this doesn't change much in terms of behavior.
2018-02-05 16:36:26 +04:00
4051391039
blockchain: test wip for hard to test functionality [ci skip]
2018-01-23 22:27:33 -05:00
a30315276b
Formatting and documentation
2018-01-21 12:32:23 -05:00
3368eeb03e
fix tests
2018-01-14 01:19:07 -05:00
a17105fd46
p2p: peer.Key -> peer.ID
2018-01-01 22:39:05 -05:00
1d6f00859d
fixes from review
2017-12-28 23:15:54 -05:00
537b0dfa1a
use NopEventBus
2017-12-28 23:15:54 -05:00
0acca7fe69
final updates for state
2017-12-28 23:15:54 -05:00
bac60f2067
blockchain: update for new state
2017-12-28 23:15:54 -05:00
3d00c477fc
separate block vs state based validation
2017-12-21 16:49:47 -05:00
843e1ed400
Updates -> ValidatoSetUpdates
2017-12-19 13:03:39 -06:00
960b25408f
Store LastConsensusHash in State as well
...
Update all BlockValidation that it matches the last state
2017-12-19 12:28:08 -05:00
d151e36ea8
Add ConsensusHash to header
2017-12-19 12:28:08 -05:00
068d83bce8
Merge pull request #677 from tendermint/blockchain-test-store
...
blockchain: add tests for BlockStore
2017-12-15 13:33:55 -05:00
78a682e4b6
blockchain: test fixes
2017-12-15 12:07:48 -05:00
dedf03bb81
Add TotalTx to block header, issue #952
...
Update state to keep track of this info.
Change function args as needed.
Make NumTx also an int64 for consistency.
2017-12-13 12:20:53 +01:00
2da5299924
blockchain: less fragile and involved tests for blockstore
...
With feedback from @ebuchman, to make the tests nicer
and less fragile.
2017-12-10 19:58:22 -05:00
922af7c405
int64 height
...
uint64 is considered dangerous. the details will follow in a blog post.
2017-12-01 19:04:53 -06:00
b3492356e6
uint64 height (Refs #911 )
2017-12-01 17:17:22 -05:00
e06bbaf303
refactor TestNoBlockMessageResponse to eliminate a race condition
2017-10-24 15:32:01 +04:00
716364182d
[state] expose ChainID and Params
...
```
jaekwon
Yeah we should definitely expose ChainID.
ConsensusParams is small enough, we can just write it.
```
https://github.com/tendermint/tendermint/pull/676#discussion_r144123203
2017-10-16 10:34:02 +04:00
659768783f
blockchain: fixing reactor tests
2017-10-04 22:26:00 -04:00
068f01368f
blockchain/reactor: respondWithNoResponseMessage for missing height
...
Fixes #514
Replaces #540
If a peer requests a block with a height that we don't have
respond with a bcNoBlockResponseMessage.
However, according to the Tendermint spec, if all nodes are honest
this condition shouldn't occur, so this is a possible hint of an
dishonest node.
2017-10-04 17:27:10 -04:00
Anton Kaliaev
Ethan Buchman
Adrian Brink
Ethan Frey
Emmanuel Odeke