EIP-1559: Be resilient to target gas used 0 and cap gas used to gas limit

contracts/test/ERC20toCW20PointerTest.js

@@ -86,7 +86,7 @@ describe("ERC20 to CW20 Pointer", function () {
             });
 
             describe("transfer()", function () {
-                it.only("should transfer", async function () {
+                it("should transfer", async function () {
                     let sender = accounts[0];
                     let recipient = accounts[1];
 

x/evm/keeper/fee.go

@@ -11,10 +11,18 @@ func (k *Keeper) AdjustDynamicBaseFeePerGas(ctx sdk.Context, blockGasUsed uint64
 		return nil
 	}
 	currentBaseFee := k.GetDynamicBaseFeePerGas(ctx)
+	targetGasUsed := sdk.NewDec(int64(k.GetTargetGasUsedPerBlock(ctx)))
+	if targetGasUsed.IsZero() {
+		return &currentBaseFee
+	}
 	minimumFeePerGas := k.GetParams(ctx).MinimumFeePerGas
 	blockGasLimit := sdk.NewDec(ctx.ConsensusParams().Block.MaxGas)
 	blockGasUsedDec := sdk.NewDec(int64(blockGasUsed))
-	targetGasUsed := sdk.NewDec(int64(k.GetTargetGasUsedPerBlock(ctx)))
+
+	// cap block gas used to block gas limit
+	if blockGasUsedDec.GT(blockGasLimit) {
+		blockGasUsedDec = blockGasLimit
+	}
 
 	var newBaseFee sdk.Dec
 	if blockGasUsedDec.GT(targetGasUsed) {

x/evm/keeper/fee_test.go

@@ -109,6 +109,29 @@ func TestAdjustBaseFeePerGas(t *testing.T) {
 			targetGasUsed:   500000,
 			expectedBaseFee: 99, // Should not go below the minimum fee
 		},
+		{
+			name:            "target gas used is 0",
+			currentBaseFee:  10000,
+			minimumFee:      10,
+			blockGasUsed:    0,
+			blockGasLimit:   1000000,
+			upwardAdj:       sdk.NewDecWithPrec(5, 1),
+			downwardAdj:     sdk.NewDecWithPrec(5, 1),
+			targetGasUsed:   0,
+			expectedBaseFee: 10000,
+		},
+		{
+			name: "cap block gas used to block gas limit",
+			// block gas used is 1.5x block gas limit
+			currentBaseFee:  10000,
+			minimumFee:      10,
+			blockGasUsed:    1500000,
+			blockGasLimit:   1000000,
+			upwardAdj:       sdk.NewDecWithPrec(5, 1),
+			downwardAdj:     sdk.NewDecWithPrec(5, 1),
+			targetGasUsed:   500000,
+			expectedBaseFee: 15000,
+		},
 	}
 
 	for _, tc := range testCases {

x/evm/types/params.go

@@ -64,7 +64,7 @@
 		paramtypes.NewParamSetPair(KeyMinFeePerGas, &p.MinimumFeePerGas, validateMinFeePerGas),
 		paramtypes.NewParamSetPair(KeyWhitelistedCwCodeHashesForDelegateCall, &p.WhitelistedCwCodeHashesForDelegateCall, validateWhitelistedCwHashesForDelegateCall),
 		paramtypes.NewParamSetPair(KeyDeliverTxHookWasmGasLimit, &p.DeliverTxHookWasmGasLimit, validateDeliverTxHookWasmGasLimit),
-		paramtypes.NewParamSetPair(KeyTargetGasUsedPerBlock, &p.TargetGasUsedPerBlock, validateTargetGasUsedPerBlock),
+		paramtypes.NewParamSetPair(KeyTargetGasUsedPerBlock, &p.TargetGasUsedPerBlock, func(i interface{}) error { return nil} ),
 	}
 }
 
@@ -90,23 +90,9 @@
 	if err := validateBaseFeeAdjustment(p.MaxDynamicBaseFeeDownwardAdjustment); err != nil {
 		return fmt.Errorf("invalid max dynamic base fee downward adjustment: %s, err: %s", p.MaxDynamicBaseFeeDownwardAdjustment, err)
 	}
-	if err := validateTargetGasUsedPerBlock(p.TargetGasUsedPerBlock); err != nil {
-		return err
-	}
 	return validateWhitelistedCwHashesForDelegateCall(p.WhitelistedCwCodeHashesForDelegateCall)
 }
 
-func validateTargetGasUsedPerBlock(i interface{}) error {
-	v, ok := i.(uint64)
-	if !ok {
-		return fmt.Errorf("invalid parameter type: %T", i)
-	}
-	if v == 0 {
-		return fmt.Errorf("invalid target gas used per block: must be greater than 0, got %d", v)
-	}
-	return nil
-}
-
 func validateBaseFeeAdjustment(i interface{}) error {
 	adjustment, ok := i.(sdk.Dec)
 	if !ok {