cleanup

Author: HuangYi

app/app.go

@@ -132,6 +132,7 @@ import (
 	"github.com/evmos/ethermint/app/ante"
 	"github.com/evmos/ethermint/encoding"
 	"github.com/evmos/ethermint/ethereum/eip712"
+	srvconfig "github.com/evmos/ethermint/server/config"
 	srvflags "github.com/evmos/ethermint/server/flags"
 	ethermint "github.com/evmos/ethermint/types"
 	"github.com/evmos/ethermint/x/evm"
@@ -355,12 +356,15 @@ func NewEthermintApp(
 	}
 
 	executor := cast.ToString(appOpts.Get(srvflags.EVMBlockExecutor))
-	if executor == "block-stm" {
+	switch executor {
+	case srvconfig.BlockExecutorBlockSTM:
 		sdk.SetAddrCacheEnabled(false)
 		workers := cast.ToInt(appOpts.Get(srvflags.EVMBlockSTMWorkers))
 		app.SetTxExecutor(STMTxExecutor(app.GetStoreKeys(), workers))
-	} else {
+	case srvconfig.BlockExecutorSequential:
 		app.SetTxExecutor(DefaultTxExecutor)
+	default:
+		panic(fmt.Errorf("unknown EVM block executor: %s", executor))
 	}
 
 	// init params keeper and subspaces

server/config/config.go

@@ -97,13 +97,18 @@ const (
 
 	// DefaultRosettaDenomToSuggest defines the default denom for fee suggestion
 	DefaultRosettaDenomToSuggest = "basecro"
+
+	BlockExecutorSequential = "sequential"
+	BlockExecutorBlockSTM   = "block-stm"
 )
 
 var (
 	// DefaultRosettaGasPrices defines the default list of prices to suggest
 	DefaultRosettaGasPrices = sdk.NewDecCoins(sdk.NewDecCoin(DefaultRosettaDenomToSuggest, sdkmath.NewInt(4_000_000)))
 
 	evmTracers = []string{"json", "markdown", "struct", "access_list"}
+
+	blockExecutors = []string{"sequential", "block-stm"}
 )
 
 // Config defines the server's top level configuration. It includes the default app config
@@ -252,6 +257,10 @@ func (c EVMConfig) Validate() error {
 		return fmt.Errorf("invalid tracer type %s, available types: %v", c.Tracer, evmTracers)
 	}
 
+	if !strings.StringInSlice(c.BlockExecutor, blockExecutors) {
+		return fmt.Errorf("invalid block executor type %s, available types: %v", c.BlockExecutor, blockExecutors)
+	}
+
 	return nil
 }