diff --git a/core/txpool/legacypool/legacypool.go b/core/txpool/legacypool/legacypool.go index 6d5ef63f6f..3034291f19 100644 --- a/core/txpool/legacypool/legacypool.go +++ b/core/txpool/legacypool/legacypool.go @@ -282,7 +282,7 @@ func New(config Config, chain BlockChain) *LegacyPool { // pool, specifically, whether it is a Legacy, AccessList or Dynamic transaction. func (pool *LegacyPool) Filter(tx *types.Transaction) bool { switch tx.Type() { - case types.LegacyTxType, types.AccessListTxType, types.DynamicFeeTxType: + case types.LegacyTxType, types.AccessListTxType, types.DynamicFeeTxType, types.SetCodeTxType: return true default: return false diff --git a/core/txpool/legacypool/legacypool_test.go b/core/txpool/legacypool/legacypool_test.go index 06c2dac49e..84afeb83c4 100644 --- a/core/txpool/legacypool/legacypool_test.go +++ b/core/txpool/legacypool/legacypool_test.go @@ -2586,19 +2586,6 @@ func TestSetCodeTransactions(t *testing.T) { testAddBalance(pool, addrB, big.NewInt(params.Ether)) testAddBalance(pool, addrC, big.NewInt(params.Ether)) - // A few situations to test: - // 1. Accounts with delegation set can only have one in-flight transaction. - // 2. Setcode tx should be rejected if any authority has a known pooled tx. - // 3. New txs from senders with pooled delegations should not be accepted. - // 4. Ensure setcode tx can replace itself provided the fee bump is enough. - // 5. Make sure that if a setcode tx is replaced, the auths associated with - // the tx are removed. - // 5.1. This should also work when a self-sponsored setcode tx attempts - // to replace itself. - - // make sure auth list recreated correctly after full reorg? - // (in different test?) verify that a setcode tx cannot invalidate a blob tx. - for _, tt := range []struct { name string pending int