Skip to content

Commit be3fa57

Browse files
committed
chore: fix tests
1 parent 64aa3cc commit be3fa57

File tree

1 file changed

+10
-6
lines changed

1 file changed

+10
-6
lines changed

state/execution_test.go

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -596,7 +596,7 @@ func TestFinalizeBlockValidatorUpdates(t *testing.T) {
596596
mock.Anything,
597597
mock.Anything,
598598
mock.Anything).Return(nil)
599-
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(types.Txs{})
599+
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return([]*types.CachedTx{})
600600

601601
blockStore := store.NewBlockStore(dbm.NewMemDB())
602602
blockExec := sm.NewBlockExecutor(
@@ -812,8 +812,9 @@ func TestPrepareProposalReorderTxs(t *testing.T) {
812812
evpool.On("PendingEvidence", mock.Anything).Return([]types.Evidence{}, int64(0))
813813

814814
txs := test.MakeNTxs(height, 10)
815+
cachedTxs := types.CachedTxFromTxs(txs)
815816
mp := &mpmocks.Mempool{}
816-
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(txs)
817+
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(cachedTxs)
817818

818819
txs = txs[2:]
819820
txs = append(txs[len(txs)/2:], txs[:len(txs)/2]...)
@@ -871,8 +872,9 @@ func TestPrepareProposalErrorOnTooManyTxs(t *testing.T) {
871872
var bytesPerTx int64 = 3
872873
maxDataBytes := types.MaxDataBytes(state.ConsensusParams.Block.MaxBytes, 0, nValidators)
873874
txs := test.MakeNTxs(height, maxDataBytes/bytesPerTx+2) // +2 so that tx don't fit
875+
cachedTxs := types.CachedTxFromTxs(txs)
874876
mp := &mpmocks.Mempool{}
875-
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(txs)
877+
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(cachedTxs)
876878

877879
app := &abcimocks.Application{}
878880
app.On("PrepareProposal", mock.Anything, mock.Anything).Return(&abci.ResponsePrepareProposal{
@@ -928,8 +930,9 @@ func TestPrepareProposalCountSerializationOverhead(t *testing.T) {
928930
evpool.On("PendingEvidence", mock.Anything).Return([]types.Evidence{}, int64(0))
929931

930932
txs := test.MakeNTxs(height, maxDataBytes/bytesPerTx)
933+
cachedTxs := types.CachedTxFromTxs(txs)
931934
mp := &mpmocks.Mempool{}
932-
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(txs)
935+
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(cachedTxs)
933936

934937
app := &abcimocks.Application{}
935938
app.On("PrepareProposal", mock.Anything, mock.Anything).Return(&abci.ResponsePrepareProposal{
@@ -977,8 +980,9 @@ func TestPrepareProposalErrorOnPrepareProposalError(t *testing.T) {
977980
evpool.On("PendingEvidence", mock.Anything).Return([]types.Evidence{}, int64(0))
978981

979982
txs := test.MakeNTxs(height, 10)
983+
cachedTxs := types.CachedTxFromTxs(txs)
980984
mp := &mpmocks.Mempool{}
981-
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(txs)
985+
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(cachedTxs)
982986

983987
cm := &abciclientmocks.Client{}
984988
cm.On("SetLogger", mock.Anything).Return()
@@ -1080,7 +1084,7 @@ func TestCreateProposalAbsentVoteExtensions(t *testing.T) {
10801084
mock.Anything,
10811085
mock.Anything,
10821086
mock.Anything).Return(nil)
1083-
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return(types.Txs{})
1087+
mp.On("ReapMaxBytesMaxGas", mock.Anything, mock.Anything).Return([]*types.CachedTx{})
10841088

10851089
blockStore := store.NewBlockStore(dbm.NewMemDB())
10861090
blockExec := sm.NewBlockExecutor(

0 commit comments

Comments
 (0)