avoid deadlock

internal/consensus/common_test.go

@@ -227,7 +227,6 @@ func sortVValidatorStubsByPower(ctx context.Context, t *testing.T, vss []*valida
 
 func startTestRound(ctx context.Context, cs *State, height int64, round int32) {
 	cs.enterNewRound(ctx, height, round)
-	cs.startRoutines(ctx, 0)
 }
 
 // Create proposal block from cs1 but sign it with vs.

internal/consensus/state.go

@@ -2097,7 +2097,11 @@ func (cs *State) defaultSetProposal(proposal *types.Proposal, recvTime time.Time
 	// TODO: We can check if Proposal is for a different block as this is a sign of misbehavior!
 	if cs.ProposalBlockParts == nil {
 		cs.metrics.MarkBlockGossipStarted()
-		cs.ProposalBlockParts = types.NewPartSetFromHeader(proposal.BlockID.PartSetHeader)
+		func() {
+			cs.mtx.Lock()
+			defer cs.mtx.Unlock()
+			cs.ProposalBlockParts = types.NewPartSetFromHeader(proposal.BlockID.PartSetHeader)
+		}()
 	}
 
 	cs.logger.Info("received proposal", "proposal", proposal)