miner: fixed race condition in tests (#21664)
This commit is contained in:
		
							parent
							
								
									cef3e2dc5a
								
							
						
					
					
						commit
						6e7137103c
					
				@ -22,7 +22,7 @@ import (
 | 
				
			|||||||
	"time"
 | 
						"time"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/common"
 | 
						"github.com/ethereum/go-ethereum/common"
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/consensus/ethash"
 | 
						"github.com/ethereum/go-ethereum/consensus/clique"
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/core"
 | 
						"github.com/ethereum/go-ethereum/core"
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/core/rawdb"
 | 
						"github.com/ethereum/go-ethereum/core/rawdb"
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/core/state"
 | 
						"github.com/ethereum/go-ethereum/core/state"
 | 
				
			||||||
@ -31,7 +31,6 @@ import (
 | 
				
			|||||||
	"github.com/ethereum/go-ethereum/eth/downloader"
 | 
						"github.com/ethereum/go-ethereum/eth/downloader"
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/ethdb/memorydb"
 | 
						"github.com/ethereum/go-ethereum/ethdb/memorydb"
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/event"
 | 
						"github.com/ethereum/go-ethereum/event"
 | 
				
			||||||
	"github.com/ethereum/go-ethereum/params"
 | 
					 | 
				
			||||||
	"github.com/ethereum/go-ethereum/trie"
 | 
						"github.com/ethereum/go-ethereum/trie"
 | 
				
			||||||
)
 | 
					)
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@ -243,26 +242,20 @@ func createMiner(t *testing.T) (*Miner, *event.TypeMux) {
 | 
				
			|||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		t.Fatalf("can't create new chain config: %v", err)
 | 
							t.Fatalf("can't create new chain config: %v", err)
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
	// Create event Mux
 | 
					 | 
				
			||||||
	mux := new(event.TypeMux)
 | 
					 | 
				
			||||||
	// Create consensus engine
 | 
						// Create consensus engine
 | 
				
			||||||
	engine := ethash.New(ethash.Config{}, []string{}, false)
 | 
						engine := clique.New(chainConfig.Clique, chainDB)
 | 
				
			||||||
	engine.SetThreads(-1)
 | 
					 | 
				
			||||||
	// Create isLocalBlock
 | 
					 | 
				
			||||||
	isLocalBlock := func(block *types.Block) bool {
 | 
					 | 
				
			||||||
		return true
 | 
					 | 
				
			||||||
	}
 | 
					 | 
				
			||||||
	// Create Ethereum backend
 | 
						// Create Ethereum backend
 | 
				
			||||||
	limit := uint64(1000)
 | 
						bc, err := core.NewBlockChain(chainDB, nil, chainConfig, engine, vm.Config{}, nil, nil)
 | 
				
			||||||
	bc, err := core.NewBlockChain(chainDB, new(core.CacheConfig), chainConfig, engine, vm.Config{}, isLocalBlock, &limit)
 | 
					 | 
				
			||||||
	if err != nil {
 | 
						if err != nil {
 | 
				
			||||||
		t.Fatalf("can't create new chain %v", err)
 | 
							t.Fatalf("can't create new chain %v", err)
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
	statedb, _ := state.New(common.Hash{}, state.NewDatabase(rawdb.NewMemoryDatabase()), nil)
 | 
						statedb, _ := state.New(common.Hash{}, state.NewDatabase(chainDB), nil)
 | 
				
			||||||
	blockchain := &testBlockChain{statedb, 10000000, new(event.Feed)}
 | 
						blockchain := &testBlockChain{statedb, 10000000, new(event.Feed)}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	pool := core.NewTxPool(testTxPoolConfig, params.TestChainConfig, blockchain)
 | 
						pool := core.NewTxPool(testTxPoolConfig, chainConfig, blockchain)
 | 
				
			||||||
	backend := NewMockBackend(bc, pool)
 | 
						backend := NewMockBackend(bc, pool)
 | 
				
			||||||
 | 
						// Create event Mux
 | 
				
			||||||
 | 
						mux := new(event.TypeMux)
 | 
				
			||||||
	// Create Miner
 | 
						// Create Miner
 | 
				
			||||||
	return New(backend, &config, chainConfig, mux, engine, isLocalBlock), mux
 | 
						return New(backend, &config, chainConfig, mux, engine, nil), mux
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
		Reference in New Issue
	
	Block a user