fix backport

abci/checktx/check_tx_test.go

@@ -125,18 +125,16 @@ func (s *CheckTxTestSuite) TestRemovalOnRecheckTx() {
 	s.Require().NoError(err)
 
 	mevLane := s.InitLane(math.LegacyOneDec(), nil)
-	mempool, err := block.NewLanedMempool(s.Ctx.Logger(), []block.Lane{mevLane}, moduleLaneFetcher{
-		mevLane,
-	})
+	mempool, err := block.NewLanedMempool(s.Ctx.Logger(), []block.Lane{mevLane})
 	s.Require().NoError(err)
 
 	handler := checktx.NewMempoolParityCheckTx(
 		s.Ctx.Logger(),
 		mempool,
 		s.EncCfg.TxConfig.TxDecoder(),
-		func(*cometabci.RequestCheckTx) (*cometabci.ResponseCheckTx, error) {
+		func(cometabci.RequestCheckTx) cometabci.ResponseCheckTx {
 			// always fail
-			return &cometabci.ResponseCheckTx{Code: 1}, nil
+			return cometabci.ResponseCheckTx{Code: 1}
 		},
 	).CheckTx()
 
@@ -149,7 +147,7 @@ func (s *CheckTxTestSuite) TestRemovalOnRecheckTx() {
 		s.Require().True(mempool.Contains(tx))
 
 		// check tx
-		res, err := handler(&cometabci.RequestCheckTx{Tx: txBz, Type: cometabci.CheckTxType_Recheck})
+		res := handler(cometabci.RequestCheckTx{Tx: txBz, Type: cometabci.CheckTxType_Recheck})
 		s.Require().NoError(err)
 
 		s.Require().Equal(uint32(1), res.Code)

abci/checktx/mempool_parity_check_tx.go

@@ -74,28 +74,27 @@ func (m MempoolParityCheckTx) CheckTx() CheckTx {
 		}
 
 		// run the checkTxHandler
-		res, checkTxError := m.checkTxHandler(req)
+		res := m.checkTxHandler(req)
 
 		// if re-check fails for a transaction, we'll need to explicitly purge the tx from
 		// the app-side mempool
-		if isInvalidCheckTxExecution(res, checkTxError) && isReCheck {
+		if isInvalidCheckTxExecution(res) && isReCheck {
 			// check if the tx exists first
 			if m.mempl.Contains(tx) {
 				// remove the tx
 				if err := m.mempl.Remove(tx); err != nil {
 					m.logger.Debug(
 						"failed to remove tx from app-side mempool when purging for re-check failure",
 						"removal-err", err,
-						"check-tx-err", checkTxError,
 					)
 				}
 			}
 		}
 
-		return res, checkTxError
+		return res
 	}
 }
 
-func isInvalidCheckTxExecution(resp *cmtabci.ResponseCheckTx, checkTxErr error) bool {
-	return resp == nil || resp.Code != 0 || checkTxErr != nil
+func isInvalidCheckTxExecution(resp cmtabci.ResponseCheckTx) bool {
+	return resp.Code != 0
 }