You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

372 lines
10 KiB

blockchain: Reorg reactor (#3561) * go routines in blockchain reactor * Added reference to the go routine diagram * Initial commit * cleanup * Undo testing_logger change, committed by mistake * Fix the test loggers * pulled some fsm code into pool.go * added pool tests * changes to the design added block requests under peer moved the request trigger in the reactor poolRoutine, triggered now by a ticker in general moved everything required for making block requests smarter in the poolRoutine added a simple map of heights to keep track of what will need to be requested next added a few more tests * send errors to FSM in a different channel than blocks send errors (RemovePeer) from switch on a different channel than the one receiving blocks renamed channels added more pool tests * more pool tests * lint errors * more tests * more tests * switch fast sync to new implementation * fixed data race in tests * cleanup * finished fsm tests * address golangci comments :) * address golangci comments :) * Added timeout on next block needed to advance * updating docs and cleanup * fix issue in test from previous cleanup * cleanup * Added termination scenarios, tests and more cleanup * small fixes to adr, comments and cleanup * Fix bug in sendRequest() If we tried to send a request to a peer not present in the switch, a missing continue statement caused the request to be blackholed in a peer that was removed and never retried. While this bug was manifesting, the reactor kept asking for other blocks that would be stored and never consumed. Added the number of unconsumed blocks in the math for requesting blocks ahead of current processing height so eventually there will be no more blocks requested until the already received ones are consumed. * remove bpPeer's didTimeout field * Use distinct err codes for peer timeout and FSM timeouts * Don't allow peers to update with lower height * review comments from Ethan and Zarko * some cleanup, renaming, comments * Move block execution in separate goroutine * Remove pool's numPending * review comments * fix lint, remove old blockchain reactor and duplicates in fsm tests * small reorg around peer after review comments * add the reactor spec * verify block only once * review comments * change to int for max number of pending requests * cleanup and godoc * Add configuration flag fast sync version * golangci fixes * fix config template * move both reactor versions under blockchain * cleanup, golint, renaming stuff * updated documentation, fixed more golint warnings * integrate with behavior package * sync with master * gofmt * add changelog_pending entry * move to improvments * suggestion to changelog entry
6 years ago
blockchain: Reorg reactor (#3561) * go routines in blockchain reactor * Added reference to the go routine diagram * Initial commit * cleanup * Undo testing_logger change, committed by mistake * Fix the test loggers * pulled some fsm code into pool.go * added pool tests * changes to the design added block requests under peer moved the request trigger in the reactor poolRoutine, triggered now by a ticker in general moved everything required for making block requests smarter in the poolRoutine added a simple map of heights to keep track of what will need to be requested next added a few more tests * send errors to FSM in a different channel than blocks send errors (RemovePeer) from switch on a different channel than the one receiving blocks renamed channels added more pool tests * more pool tests * lint errors * more tests * more tests * switch fast sync to new implementation * fixed data race in tests * cleanup * finished fsm tests * address golangci comments :) * address golangci comments :) * Added timeout on next block needed to advance * updating docs and cleanup * fix issue in test from previous cleanup * cleanup * Added termination scenarios, tests and more cleanup * small fixes to adr, comments and cleanup * Fix bug in sendRequest() If we tried to send a request to a peer not present in the switch, a missing continue statement caused the request to be blackholed in a peer that was removed and never retried. While this bug was manifesting, the reactor kept asking for other blocks that would be stored and never consumed. Added the number of unconsumed blocks in the math for requesting blocks ahead of current processing height so eventually there will be no more blocks requested until the already received ones are consumed. * remove bpPeer's didTimeout field * Use distinct err codes for peer timeout and FSM timeouts * Don't allow peers to update with lower height * review comments from Ethan and Zarko * some cleanup, renaming, comments * Move block execution in separate goroutine * Remove pool's numPending * review comments * fix lint, remove old blockchain reactor and duplicates in fsm tests * small reorg around peer after review comments * add the reactor spec * verify block only once * review comments * change to int for max number of pending requests * cleanup and godoc * Add configuration flag fast sync version * golangci fixes * fix config template * move both reactor versions under blockchain * cleanup, golint, renaming stuff * updated documentation, fixed more golint warnings * integrate with behavior package * sync with master * gofmt * add changelog_pending entry * move to improvments * suggestion to changelog entry
6 years ago
mempool: move interface into mempool package (#3524) ## Description Refs #2659 Breaking changes in the mempool package: [mempool] #2659 Mempool now an interface old Mempool renamed to CListMempool NewMempool renamed to NewCListMempool Option renamed to CListOption MempoolReactor renamed to Reactor NewMempoolReactor renamed to NewReactor unexpose TxID method TxInfo.PeerID renamed to SenderID unexpose MempoolReactor.Mempool Breaking changes in the state package: [state] #2659 Mempool interface moved to mempool package MockMempool moved to top-level mock package and renamed to Mempool Non Breaking changes in the node package: [node] #2659 Add Mempool method, which allows you to access mempool ## Commits * move Mempool interface into mempool package Refs #2659 Breaking changes in the mempool package: - Mempool now an interface - old Mempool renamed to CListMempool Breaking changes to state package: - MockMempool moved to mempool/mock package and renamed to Mempool - Mempool interface moved to mempool package * assert CListMempool impl Mempool * gofmt code * rename MempoolReactor to Reactor - combine everything into one interface - rename TxInfo.PeerID to TxInfo.SenderID - unexpose MempoolReactor.Mempool * move mempool mock into top-level mock package * add a fixme TxsFront should not be a part of the Mempool interface because it leaks implementation details. Instead, we need to come up with general interface for querying the mempool so the MempoolReactor can fetch and broadcast txs to peers. * change node#Mempool to return interface * save commit = new reactor arch * Revert "save commit = new reactor arch" This reverts commit 1bfceacd9d65a720574683a7f22771e69af9af4d. * require CListMempool in mempool.Reactor * add two changelog entries * fixes after my own review * quote interfaces, structs and functions * fixes after Ismail's review * make node's mempool an interface * make InitWAL/CloseWAL methods a part of Mempool interface * fix merge conflicts * make node's mempool an interface
6 years ago
blockchain: Reorg reactor (#3561) * go routines in blockchain reactor * Added reference to the go routine diagram * Initial commit * cleanup * Undo testing_logger change, committed by mistake * Fix the test loggers * pulled some fsm code into pool.go * added pool tests * changes to the design added block requests under peer moved the request trigger in the reactor poolRoutine, triggered now by a ticker in general moved everything required for making block requests smarter in the poolRoutine added a simple map of heights to keep track of what will need to be requested next added a few more tests * send errors to FSM in a different channel than blocks send errors (RemovePeer) from switch on a different channel than the one receiving blocks renamed channels added more pool tests * more pool tests * lint errors * more tests * more tests * switch fast sync to new implementation * fixed data race in tests * cleanup * finished fsm tests * address golangci comments :) * address golangci comments :) * Added timeout on next block needed to advance * updating docs and cleanup * fix issue in test from previous cleanup * cleanup * Added termination scenarios, tests and more cleanup * small fixes to adr, comments and cleanup * Fix bug in sendRequest() If we tried to send a request to a peer not present in the switch, a missing continue statement caused the request to be blackholed in a peer that was removed and never retried. While this bug was manifesting, the reactor kept asking for other blocks that would be stored and never consumed. Added the number of unconsumed blocks in the math for requesting blocks ahead of current processing height so eventually there will be no more blocks requested until the already received ones are consumed. * remove bpPeer's didTimeout field * Use distinct err codes for peer timeout and FSM timeouts * Don't allow peers to update with lower height * review comments from Ethan and Zarko * some cleanup, renaming, comments * Move block execution in separate goroutine * Remove pool's numPending * review comments * fix lint, remove old blockchain reactor and duplicates in fsm tests * small reorg around peer after review comments * add the reactor spec * verify block only once * review comments * change to int for max number of pending requests * cleanup and godoc * Add configuration flag fast sync version * golangci fixes * fix config template * move both reactor versions under blockchain * cleanup, golint, renaming stuff * updated documentation, fixed more golint warnings * integrate with behavior package * sync with master * gofmt * add changelog_pending entry * move to improvments * suggestion to changelog entry
6 years ago
cs/replay: execCommitBlock should not read from state.lastValidators (#3067) * execCommitBlock should not read from state.lastValidators * fix height 1 * fix blockchain/reactor_test * fix consensus/mempool_test * fix consensus/reactor_test * fix consensus/replay_test * add CHANGELOG * fix consensus/reactor_test * fix consensus/replay_test * add a test for replay validators change * fix mem_pool test * fix byzantine test * remove a redundant code * reduce validator change blocks to 6 * fix * return peer0 config * seperate testName * seperate testName 1 * seperate testName 2 * seperate app db path * seperate app db path 1 * add a lock before startNet * move the lock to reactor_test * simulate just once * try to find problem * handshake only saveState when app version changed * update gometalinter to 3.0.0 (#3233) in the attempt to fix https://circleci.com/gh/tendermint/tendermint/43165 also code is simplified by running gofmt -s . remove unused vars enable linters we're currently passing remove deprecated linters (cherry picked from commit d47094550315c094512a242445e0dde24b5a03f5) * gofmt code * goimport code * change the bool name to testValidatorsChange * adjust receive kvstore.ProtocolVersion * adjust receive kvstore.ProtocolVersion 1 * adjust receive kvstore.ProtocolVersion 3 * fix merge execution.go * fix merge develop * fix merge develop 1 * fix run cleanupFunc * adjust code according to reviewers' opinion * modify the func name match the convention * simplify simulate a chain containing some validator change txs 1 * test CI error * Merge remote-tracking branch 'upstream/develop' into fixReplay 1 * fix pubsub_test * subscribeUnbuffered vote channel
6 years ago
mempool: move interface into mempool package (#3524) ## Description Refs #2659 Breaking changes in the mempool package: [mempool] #2659 Mempool now an interface old Mempool renamed to CListMempool NewMempool renamed to NewCListMempool Option renamed to CListOption MempoolReactor renamed to Reactor NewMempoolReactor renamed to NewReactor unexpose TxID method TxInfo.PeerID renamed to SenderID unexpose MempoolReactor.Mempool Breaking changes in the state package: [state] #2659 Mempool interface moved to mempool package MockMempool moved to top-level mock package and renamed to Mempool Non Breaking changes in the node package: [node] #2659 Add Mempool method, which allows you to access mempool ## Commits * move Mempool interface into mempool package Refs #2659 Breaking changes in the mempool package: - Mempool now an interface - old Mempool renamed to CListMempool Breaking changes to state package: - MockMempool moved to mempool/mock package and renamed to Mempool - Mempool interface moved to mempool package * assert CListMempool impl Mempool * gofmt code * rename MempoolReactor to Reactor - combine everything into one interface - rename TxInfo.PeerID to TxInfo.SenderID - unexpose MempoolReactor.Mempool * move mempool mock into top-level mock package * add a fixme TxsFront should not be a part of the Mempool interface because it leaks implementation details. Instead, we need to come up with general interface for querying the mempool so the MempoolReactor can fetch and broadcast txs to peers. * change node#Mempool to return interface * save commit = new reactor arch * Revert "save commit = new reactor arch" This reverts commit 1bfceacd9d65a720574683a7f22771e69af9af4d. * require CListMempool in mempool.Reactor * add two changelog entries * fixes after my own review * quote interfaces, structs and functions * fixes after Ismail's review * make node's mempool an interface * make InitWAL/CloseWAL methods a part of Mempool interface * fix merge conflicts * make node's mempool an interface
6 years ago
cs/replay: execCommitBlock should not read from state.lastValidators (#3067) * execCommitBlock should not read from state.lastValidators * fix height 1 * fix blockchain/reactor_test * fix consensus/mempool_test * fix consensus/reactor_test * fix consensus/replay_test * add CHANGELOG * fix consensus/reactor_test * fix consensus/replay_test * add a test for replay validators change * fix mem_pool test * fix byzantine test * remove a redundant code * reduce validator change blocks to 6 * fix * return peer0 config * seperate testName * seperate testName 1 * seperate testName 2 * seperate app db path * seperate app db path 1 * add a lock before startNet * move the lock to reactor_test * simulate just once * try to find problem * handshake only saveState when app version changed * update gometalinter to 3.0.0 (#3233) in the attempt to fix https://circleci.com/gh/tendermint/tendermint/43165 also code is simplified by running gofmt -s . remove unused vars enable linters we're currently passing remove deprecated linters (cherry picked from commit d47094550315c094512a242445e0dde24b5a03f5) * gofmt code * goimport code * change the bool name to testValidatorsChange * adjust receive kvstore.ProtocolVersion * adjust receive kvstore.ProtocolVersion 1 * adjust receive kvstore.ProtocolVersion 3 * fix merge execution.go * fix merge develop * fix merge develop 1 * fix run cleanupFunc * adjust code according to reviewers' opinion * modify the func name match the convention * simplify simulate a chain containing some validator change txs 1 * test CI error * Merge remote-tracking branch 'upstream/develop' into fixReplay 1 * fix pubsub_test * subscribeUnbuffered vote channel
6 years ago
abci: Refactor tagging events using list of lists (#3643) ## PR This PR introduces a fundamental breaking change to the structure of ABCI response and tx tags and the way they're processed. Namely, the SDK can support more complex and aggregated events for distribution and slashing. In addition, block responses can include duplicate keys in events. Implement new Event type. An event has a type and a list of KV pairs (ie. list-of-lists). Typical events may look like: "rewards": [{"amount": "5000uatom", "validator": "...", "recipient": "..."}] "sender": [{"address": "...", "balance": "100uatom"}] The events are indexed by {even.type}.{even.attribute[i].key}/.... In this case a client would subscribe or query for rewards.recipient='...' ABCI response types and related types now include Events []Event instead of Tags []cmn.KVPair. PubSub logic now publishes/matches against map[string][]string instead of map[string]string to support duplicate keys in response events (from #1385). A match is successful if the value is found in the slice of strings. closes: #1859 closes: #2905 ## Commits: * Implement Event ABCI type and updates responses to use events * Update messages_test.go * Update kvstore.go * Update event_bus.go * Update subscription.go * Update pubsub.go * Update kvstore.go * Update query logic to handle slice of strings in events * Update Empty#Matches and unit tests * Update pubsub logic * Update EventBus#Publish * Update kv tx indexer * Update godocs * Update ResultEvent to use slice of strings; update RPC * Update more tests * Update abci.md * Check for key in validateAndStringifyEvents * Fix KV indexer to skip empty keys * Fix linting errors * Update CHANGELOG_PENDING.md * Update docs/spec/abci/abci.md Co-Authored-By: Federico Kunze <31522760+fedekunze@users.noreply.github.com> * Update abci/types/types.proto Co-Authored-By: Ethan Buchman <ethan@coinculture.info> * Update docs/spec/abci/abci.md Co-Authored-By: Ethan Buchman <ethan@coinculture.info> * Update libs/pubsub/query/query.go Co-Authored-By: Ethan Buchman <ethan@coinculture.info> * Update match function to match if ANY value matches * Implement TestSubscribeDuplicateKeys * Update TestMatches to include multi-key test cases * Update events.go * Update Query interface godoc * Update match godoc * Add godoc for matchValue * DRY-up tx indexing * Return error from PublishWithEvents in EventBus#Publish * Update PublishEventNewBlockHeader to return an error * Fix build * Update events doc in ABCI * Update ABCI events godoc * Implement TestEventBusPublishEventTxDuplicateKeys * Update TestSubscribeDuplicateKeys to be table-driven * Remove mod file * Remove markdown from events godoc * Implement TestTxSearchDeprecatedIndexing test
6 years ago
  1. package v0
  2. import (
  3. "os"
  4. "sort"
  5. "testing"
  6. "time"
  7. "github.com/pkg/errors"
  8. "github.com/tendermint/tendermint/store"
  9. "github.com/stretchr/testify/assert"
  10. abci "github.com/tendermint/tendermint/abci/types"
  11. cfg "github.com/tendermint/tendermint/config"
  12. "github.com/tendermint/tendermint/libs/log"
  13. "github.com/tendermint/tendermint/mock"
  14. "github.com/tendermint/tendermint/p2p"
  15. "github.com/tendermint/tendermint/proxy"
  16. sm "github.com/tendermint/tendermint/state"
  17. "github.com/tendermint/tendermint/types"
  18. tmtime "github.com/tendermint/tendermint/types/time"
  19. dbm "github.com/tendermint/tm-db"
  20. )
  21. var config *cfg.Config
  22. func randGenesisDoc(numValidators int, randPower bool, minPower int64) (*types.GenesisDoc, []types.PrivValidator) {
  23. validators := make([]types.GenesisValidator, numValidators)
  24. privValidators := make([]types.PrivValidator, numValidators)
  25. for i := 0; i < numValidators; i++ {
  26. val, privVal := types.RandValidator(randPower, minPower)
  27. validators[i] = types.GenesisValidator{
  28. PubKey: val.PubKey,
  29. Power: val.VotingPower,
  30. }
  31. privValidators[i] = privVal
  32. }
  33. sort.Sort(types.PrivValidatorsByAddress(privValidators))
  34. return &types.GenesisDoc{
  35. GenesisTime: tmtime.Now(),
  36. ChainID: config.ChainID(),
  37. Validators: validators,
  38. }, privValidators
  39. }
  40. type BlockchainReactorPair struct {
  41. reactor *BlockchainReactor
  42. app proxy.AppConns
  43. }
  44. func newBlockchainReactor(logger log.Logger, genDoc *types.GenesisDoc, privVals []types.PrivValidator, maxBlockHeight int64) BlockchainReactorPair {
  45. if len(privVals) != 1 {
  46. panic("only support one validator")
  47. }
  48. app := &testApp{}
  49. cc := proxy.NewLocalClientCreator(app)
  50. proxyApp := proxy.NewAppConns(cc)
  51. err := proxyApp.Start()
  52. if err != nil {
  53. panic(errors.Wrap(err, "error start app"))
  54. }
  55. blockDB := dbm.NewMemDB()
  56. stateDB := dbm.NewMemDB()
  57. blockStore := store.NewBlockStore(blockDB)
  58. state, err := sm.LoadStateFromDBOrGenesisDoc(stateDB, genDoc)
  59. if err != nil {
  60. panic(errors.Wrap(err, "error constructing state from genesis file"))
  61. }
  62. // Make the BlockchainReactor itself.
  63. // NOTE we have to create and commit the blocks first because
  64. // pool.height is determined from the store.
  65. fastSync := true
  66. db := dbm.NewMemDB()
  67. blockExec := sm.NewBlockExecutor(db, log.TestingLogger(), proxyApp.Consensus(),
  68. mock.Mempool{}, sm.MockEvidencePool{})
  69. sm.SaveState(db, state)
  70. // let's add some blocks in
  71. for blockHeight := int64(1); blockHeight <= maxBlockHeight; blockHeight++ {
  72. lastCommit := types.NewCommit(types.BlockID{}, nil)
  73. if blockHeight > 1 {
  74. lastBlockMeta := blockStore.LoadBlockMeta(blockHeight - 1)
  75. lastBlock := blockStore.LoadBlock(blockHeight - 1)
  76. vote, err := types.MakeVote(lastBlock.Header.Height, lastBlockMeta.BlockID, state.Validators, privVals[0], lastBlock.Header.ChainID)
  77. if err != nil {
  78. panic(err)
  79. }
  80. voteCommitSig := vote.CommitSig()
  81. lastCommit = types.NewCommit(lastBlockMeta.BlockID, []*types.CommitSig{voteCommitSig})
  82. }
  83. thisBlock := makeBlock(blockHeight, state, lastCommit)
  84. thisParts := thisBlock.MakePartSet(types.BlockPartSizeBytes)
  85. blockID := types.BlockID{Hash: thisBlock.Hash(), PartsHeader: thisParts.Header()}
  86. state, err = blockExec.ApplyBlock(state, blockID, thisBlock)
  87. if err != nil {
  88. panic(errors.Wrap(err, "error apply block"))
  89. }
  90. blockStore.SaveBlock(thisBlock, thisParts, lastCommit)
  91. }
  92. bcReactor := NewBlockchainReactor(state.Copy(), blockExec, blockStore, fastSync)
  93. bcReactor.SetLogger(logger.With("module", "blockchain"))
  94. return BlockchainReactorPair{bcReactor, proxyApp}
  95. }
  96. func TestNoBlockResponse(t *testing.T) {
  97. config = cfg.ResetTestRoot("blockchain_reactor_test")
  98. defer os.RemoveAll(config.RootDir)
  99. genDoc, privVals := randGenesisDoc(1, false, 30)
  100. maxBlockHeight := int64(65)
  101. reactorPairs := make([]BlockchainReactorPair, 2)
  102. reactorPairs[0] = newBlockchainReactor(log.TestingLogger(), genDoc, privVals, maxBlockHeight)
  103. reactorPairs[1] = newBlockchainReactor(log.TestingLogger(), genDoc, privVals, 0)
  104. p2p.MakeConnectedSwitches(config.P2P, 2, func(i int, s *p2p.Switch) *p2p.Switch {
  105. s.AddReactor("BLOCKCHAIN", reactorPairs[i].reactor)
  106. return s
  107. }, p2p.Connect2Switches)
  108. defer func() {
  109. for _, r := range reactorPairs {
  110. r.reactor.Stop()
  111. r.app.Stop()
  112. }
  113. }()
  114. tests := []struct {
  115. height int64
  116. existent bool
  117. }{
  118. {maxBlockHeight + 2, false},
  119. {10, true},
  120. {1, true},
  121. {100, false},
  122. }
  123. for {
  124. if reactorPairs[1].reactor.pool.IsCaughtUp() {
  125. break
  126. }
  127. time.Sleep(10 * time.Millisecond)
  128. }
  129. assert.Equal(t, maxBlockHeight, reactorPairs[0].reactor.store.Height())
  130. for _, tt := range tests {
  131. block := reactorPairs[1].reactor.store.LoadBlock(tt.height)
  132. if tt.existent {
  133. assert.True(t, block != nil)
  134. } else {
  135. assert.True(t, block == nil)
  136. }
  137. }
  138. }
  139. // NOTE: This is too hard to test without
  140. // an easy way to add test peer to switch
  141. // or without significant refactoring of the module.
  142. // Alternatively we could actually dial a TCP conn but
  143. // that seems extreme.
  144. func TestBadBlockStopsPeer(t *testing.T) {
  145. config = cfg.ResetTestRoot("blockchain_reactor_test")
  146. defer os.RemoveAll(config.RootDir)
  147. genDoc, privVals := randGenesisDoc(1, false, 30)
  148. maxBlockHeight := int64(148)
  149. otherChain := newBlockchainReactor(log.TestingLogger(), genDoc, privVals, maxBlockHeight)
  150. defer func() {
  151. otherChain.reactor.Stop()
  152. otherChain.app.Stop()
  153. }()
  154. reactorPairs := make([]BlockchainReactorPair, 4)
  155. reactorPairs[0] = newBlockchainReactor(log.TestingLogger(), genDoc, privVals, maxBlockHeight)
  156. reactorPairs[1] = newBlockchainReactor(log.TestingLogger(), genDoc, privVals, 0)
  157. reactorPairs[2] = newBlockchainReactor(log.TestingLogger(), genDoc, privVals, 0)
  158. reactorPairs[3] = newBlockchainReactor(log.TestingLogger(), genDoc, privVals, 0)
  159. switches := p2p.MakeConnectedSwitches(config.P2P, 4, func(i int, s *p2p.Switch) *p2p.Switch {
  160. s.AddReactor("BLOCKCHAIN", reactorPairs[i].reactor)
  161. return s
  162. }, p2p.Connect2Switches)
  163. defer func() {
  164. for _, r := range reactorPairs {
  165. r.reactor.Stop()
  166. r.app.Stop()
  167. }
  168. }()
  169. for {
  170. if reactorPairs[3].reactor.pool.IsCaughtUp() {
  171. break
  172. }
  173. time.Sleep(1 * time.Second)
  174. }
  175. //at this time, reactors[0-3] is the newest
  176. assert.Equal(t, 3, reactorPairs[1].reactor.Switch.Peers().Size())
  177. //mark reactorPairs[3] is an invalid peer
  178. reactorPairs[3].reactor.store = otherChain.reactor.store
  179. lastReactorPair := newBlockchainReactor(log.TestingLogger(), genDoc, privVals, 0)
  180. reactorPairs = append(reactorPairs, lastReactorPair)
  181. switches = append(switches, p2p.MakeConnectedSwitches(config.P2P, 1, func(i int, s *p2p.Switch) *p2p.Switch {
  182. s.AddReactor("BLOCKCHAIN", reactorPairs[len(reactorPairs)-1].reactor)
  183. return s
  184. }, p2p.Connect2Switches)...)
  185. for i := 0; i < len(reactorPairs)-1; i++ {
  186. p2p.Connect2Switches(switches, i, len(reactorPairs)-1)
  187. }
  188. for {
  189. if lastReactorPair.reactor.pool.IsCaughtUp() || lastReactorPair.reactor.Switch.Peers().Size() == 0 {
  190. break
  191. }
  192. time.Sleep(1 * time.Second)
  193. }
  194. assert.True(t, lastReactorPair.reactor.Switch.Peers().Size() < len(reactorPairs)-1)
  195. }
  196. func TestBcBlockRequestMessageValidateBasic(t *testing.T) {
  197. testCases := []struct {
  198. testName string
  199. requestHeight int64
  200. expectErr bool
  201. }{
  202. {"Valid Request Message", 0, false},
  203. {"Valid Request Message", 1, false},
  204. {"Invalid Request Message", -1, true},
  205. }
  206. for _, tc := range testCases {
  207. t.Run(tc.testName, func(t *testing.T) {
  208. request := bcBlockRequestMessage{Height: tc.requestHeight}
  209. assert.Equal(t, tc.expectErr, request.ValidateBasic() != nil, "Validate Basic had an unexpected result")
  210. })
  211. }
  212. }
  213. func TestBcNoBlockResponseMessageValidateBasic(t *testing.T) {
  214. testCases := []struct {
  215. testName string
  216. nonResponseHeight int64
  217. expectErr bool
  218. }{
  219. {"Valid Non-Response Message", 0, false},
  220. {"Valid Non-Response Message", 1, false},
  221. {"Invalid Non-Response Message", -1, true},
  222. }
  223. for _, tc := range testCases {
  224. t.Run(tc.testName, func(t *testing.T) {
  225. nonResponse := bcNoBlockResponseMessage{Height: tc.nonResponseHeight}
  226. assert.Equal(t, tc.expectErr, nonResponse.ValidateBasic() != nil, "Validate Basic had an unexpected result")
  227. })
  228. }
  229. }
  230. func TestBcStatusRequestMessageValidateBasic(t *testing.T) {
  231. testCases := []struct {
  232. testName string
  233. requestHeight int64
  234. expectErr bool
  235. }{
  236. {"Valid Request Message", 0, false},
  237. {"Valid Request Message", 1, false},
  238. {"Invalid Request Message", -1, true},
  239. }
  240. for _, tc := range testCases {
  241. t.Run(tc.testName, func(t *testing.T) {
  242. request := bcStatusRequestMessage{Height: tc.requestHeight}
  243. assert.Equal(t, tc.expectErr, request.ValidateBasic() != nil, "Validate Basic had an unexpected result")
  244. })
  245. }
  246. }
  247. func TestBcStatusResponseMessageValidateBasic(t *testing.T) {
  248. testCases := []struct {
  249. testName string
  250. responseHeight int64
  251. expectErr bool
  252. }{
  253. {"Valid Response Message", 0, false},
  254. {"Valid Response Message", 1, false},
  255. {"Invalid Response Message", -1, true},
  256. }
  257. for _, tc := range testCases {
  258. t.Run(tc.testName, func(t *testing.T) {
  259. response := bcStatusResponseMessage{Height: tc.responseHeight}
  260. assert.Equal(t, tc.expectErr, response.ValidateBasic() != nil, "Validate Basic had an unexpected result")
  261. })
  262. }
  263. }
  264. //----------------------------------------------
  265. // utility funcs
  266. func makeTxs(height int64) (txs []types.Tx) {
  267. for i := 0; i < 10; i++ {
  268. txs = append(txs, types.Tx([]byte{byte(height), byte(i)}))
  269. }
  270. return txs
  271. }
  272. func makeBlock(height int64, state sm.State, lastCommit *types.Commit) *types.Block {
  273. block, _ := state.MakeBlock(height, makeTxs(height), lastCommit, nil, state.Validators.GetProposer().Address)
  274. return block
  275. }
  276. type testApp struct {
  277. abci.BaseApplication
  278. }
  279. var _ abci.Application = (*testApp)(nil)
  280. func (app *testApp) Info(req abci.RequestInfo) (resInfo abci.ResponseInfo) {
  281. return abci.ResponseInfo{}
  282. }
  283. func (app *testApp) BeginBlock(req abci.RequestBeginBlock) abci.ResponseBeginBlock {
  284. return abci.ResponseBeginBlock{}
  285. }
  286. func (app *testApp) EndBlock(req abci.RequestEndBlock) abci.ResponseEndBlock {
  287. return abci.ResponseEndBlock{}
  288. }
  289. func (app *testApp) DeliverTx(req abci.RequestDeliverTx) abci.ResponseDeliverTx {
  290. return abci.ResponseDeliverTx{Events: []abci.Event{}}
  291. }
  292. func (app *testApp) CheckTx(req abci.RequestCheckTx) abci.ResponseCheckTx {
  293. return abci.ResponseCheckTx{}
  294. }
  295. func (app *testApp) Commit() abci.ResponseCommit {
  296. return abci.ResponseCommit{}
  297. }
  298. func (app *testApp) Query(reqQuery abci.RequestQuery) (resQuery abci.ResponseQuery) {
  299. return
  300. }