rpctest: ignore redundant create test chain harness option

Since the blocks are generated only when both CreateTestChain and
NumMatureOutputs are set, there's no obvious reason to keep both. The
code was simplified by ignoring the CreateTestChain option.

Author: allocz

integration/rpctest/rpc_harness.go

@@ -364,7 +364,7 @@ func (h *Harness) SetUp(opts *SetUpOpts) error {
 
 	// Create a test chain with the desired number of mature coinbase
 	// outputs.
-	if opts.CreateTestChain && opts.NumMatureOutputs != 0 {
+	if opts.NumMatureOutputs > 0 {
 		coinbaseMaturity := uint32(h.ActiveNet.CoinbaseMaturity)
 		numToGenerate := coinbaseMaturity + opts.NumMatureOutputs
 		_, err := h.Client.Generate(numToGenerate)