From 7c562994b2527829b15083fa59af809fdc22035b Mon Sep 17 00:00:00 2001 From: MariusVanDerWijden Date: Thu, 17 Jul 2025 11:03:15 +0200 Subject: [PATCH] core: add state processor errors --- core/state_processor_test.go | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/core/state_processor_test.go b/core/state_processor_test.go index d175b5eac5..9d6cbdbc8b 100644 --- a/core/state_processor_test.go +++ b/core/state_processor_test.go @@ -132,6 +132,7 @@ func TestStateProcessorErrors(t *testing.T) { bigNumber := new(big.Int).SetBytes(common.MaxHash.Bytes()) tooBigNumber := new(big.Int).Set(bigNumber) tooBigNumber.Add(tooBigNumber, common.Big1) + gasLimit := blockchain.CurrentHeader().GasLimit for i, tt := range []struct { txs []*types.Transaction want string @@ -157,9 +158,9 @@ func TestStateProcessorErrors(t *testing.T) { }, { // ErrGasLimitReached txs: []*types.Transaction{ - makeTx(key1, 0, common.Address{}, big.NewInt(0), 21000000, big.NewInt(875000000), nil), + makeTx(key1, 0, common.Address{}, big.NewInt(0), gasLimit+1, big.NewInt(875000000), nil), }, - want: "could not apply tx 0 [0xbd49d8dadfd47fb846986695f7d4da3f7b2c48c8da82dbc211a26eb124883de9]: gas limit reached", + want: "could not apply tx 0 [0xd0fb3ea181e800cd55c4637c55c1f2f78137efb6bb9723e50bda3cad97208db2]: gas limit reached", }, { // ErrInsufficientFundsForTransfer txs: []*types.Transaction{ @@ -185,9 +186,9 @@ func TestStateProcessorErrors(t *testing.T) { }, { // ErrGasLimitReached txs: []*types.Transaction{ - makeTx(key1, 0, common.Address{}, big.NewInt(0), params.TxGas*1000, big.NewInt(875000000), nil), + makeTx(key1, 0, common.Address{}, big.NewInt(0), gasLimit+1, big.NewInt(875000000), nil), }, - want: "could not apply tx 0 [0xbd49d8dadfd47fb846986695f7d4da3f7b2c48c8da82dbc211a26eb124883de9]: gas limit reached", + want: "could not apply tx 0 [0xd0fb3ea181e800cd55c4637c55c1f2f78137efb6bb9723e50bda3cad97208db2]: gas limit reached", }, { // ErrFeeCapTooLow txs: []*types.Transaction{ @@ -256,6 +257,12 @@ func TestStateProcessorErrors(t *testing.T) { }, // ErrSetCodeTxCreate cannot be tested here: it is impossible to create a SetCode-tx with nil `to`. // The EstimateGas API tests test this case. + { // ErrGasLimitTooHigh + txs: []*types.Transaction{ + makeTx(key1, 0, common.Address{}, big.NewInt(0), params.MaxTxGas+1, big.NewInt(875000000), nil), + }, + want: "could not apply tx 0 [0x16505812a6da0b0150593e4d4eb90190ba64816a04b27d19ca926ebd6aff8aa0]: transaction gas limit too high (cap: 16777216, tx: 16777217)", + }, } { block := GenerateBadBlock(gspec.ToBlock(), beacon.New(ethash.NewFaker()), tt.txs, gspec.Config, false) _, err := blockchain.InsertChain(types.Blocks{block})