syncer: fix issues when switching to core.Consensus (#418)

- when confirmed blocks passed to core.Consensus
   aren't continuous in position in some chain, the
   pulling would skip those missing blocks.
   - fix: when some block is missing, avoid adding it
      and all blocks after it to core.Consensus.
- we need to avoid the receive channel of network
   module full.
   - fix: during switching to core.Consensus, we
      need to launch a dummy receiver to receive
      from receive channel of network module.
   - fix: between the period during core.Consensus
      created and before running, a dummy receiver is
      also required to receive from receive channel of
      network module.

1 files changed, 1 insertions, 0 deletions

diff --git a/core/consensus_test.go b/core/consensus_test.go
index 938b56d..b53af8a 100644
--- a/core/consensus_test.go
+++ b/core/consensus_test.go
@@ -593,6 +593,7 @@ func (s *ConsensusTestSuite) TestSyncBA() {
 	prvKey := prvKeys[0]
 	_, con := s.prepareConsensus(time.Now().UTC(), gov, prvKey, conn)
 	go con.Run()
+	defer con.Stop()
 	hash := common.NewRandomHash()
 	signers := make([]*utils.Signer, 0, len(prvKeys))
 	for _, prvKey := range prvKeys {