From eb303eccceea859bfaf2487db295ae506bcdbf7a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Toni=20Ram=C3=ADrez?=
 <58293609+ToniRamirezM@users.noreply.github.com>
Date: Thu, 2 Mar 2023 19:14:04 +0100
Subject: [PATCH] sequencer fixes (#1724)

* sequencer fixes

* comment test

* fix test
---
 sequencer/efficiencylist.go |  3 +++
 sequencer/finalizer.go      | 10 +++++++++-
 test/e2e/jsonrpc_test.go    |  5 ++---
 3 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/sequencer/efficiencylist.go b/sequencer/efficiencylist.go
index f22e5ff81a..3ee15bc47a 100644
--- a/sequencer/efficiencylist.go
+++ b/sequencer/efficiencylist.go
@@ -4,6 +4,8 @@ import (
 	"fmt"
 	"sort"
 	"sync"
+
+	"github.com/0xPolygonHermez/zkevm-node/log"
 )
 
 // efficiencyList represents a list of tx sorted by efficiency
@@ -46,6 +48,7 @@ func (e *efficiencyList) delete(tx *TxTracker) bool {
 		})
 
 		if (e.sorted[i].HashStr != tx.HashStr) || i == sLen {
+			log.Errorf("Error deleting tx from efficiencyList: %s", tx.HashStr)
 			return false
 		}
 
diff --git a/sequencer/finalizer.go b/sequencer/finalizer.go
index bf6acc9eae..674c0ea5d2 100644
--- a/sequencer/finalizer.go
+++ b/sequencer/finalizer.go
@@ -203,7 +203,11 @@ func (f *finalizer) finalizeBatches(ctx context.Context) {
 		metrics.WorkerProcessingTime(time.Since(start))
 		if tx != nil {
 			f.sharedResourcesMux.Lock()
-			_ = f.processTransaction(ctx, tx)
+			err := f.processTransaction(ctx, tx)
+			if err != nil {
+				log.Errorf("failed to process transaction in finalizeBatches, Err: %s", err)
+			}
+
 			f.sharedResourcesMux.Unlock()
 		} else {
 			if f.isBatchAlmostFull() {
@@ -424,6 +428,10 @@ func (f *finalizer) storeProcessedTx(ctx context.Context, previousL2BlockStateRo
 		previousL2BlockStateRoot: previousL2BlockStateRoot,
 	}
 
+	// Delete the transaction from the efficiency list
+	f.worker.DeleteTx(tx.Hash, tx.From)
+	log.Debug("tx deleted from efficiency list", "txHash", tx.Hash.String(), "from", tx.From.Hex())
+
 	start := time.Now()
 	txsToDelete := f.worker.UpdateAfterSingleSuccessfulTxExecution(tx.From, result.ReadWriteAddresses)
 	for _, txToDelete := range txsToDelete {
diff --git a/test/e2e/jsonrpc_test.go b/test/e2e/jsonrpc_test.go
index 2189241a17..8d62de98ed 100644
--- a/test/e2e/jsonrpc_test.go
+++ b/test/e2e/jsonrpc_test.go
@@ -15,17 +15,14 @@ import (
 	"github.com/0xPolygonHermez/zkevm-node/pool"
 	"github.com/0xPolygonHermez/zkevm-node/test/contracts/bin/Double"
 	"github.com/0xPolygonHermez/zkevm-node/test/contracts/bin/EmitLog"
-	"github.com/0xPolygonHermez/zkevm-node/test/contracts/bin/Revert2"
 	"github.com/0xPolygonHermez/zkevm-node/test/contracts/bin/Storage"
 	"github.com/0xPolygonHermez/zkevm-node/test/operations"
 	"github.com/ethereum/go-ethereum"
-	"github.com/ethereum/go-ethereum/accounts/abi"
 	"github.com/ethereum/go-ethereum/accounts/abi/bind"
 	"github.com/ethereum/go-ethereum/common"
 	"github.com/ethereum/go-ethereum/common/hexutil"
 	"github.com/ethereum/go-ethereum/core/types"
 	"github.com/ethereum/go-ethereum/ethclient"
-	"github.com/ethereum/go-ethereum/rpc"
 	"github.com/gorilla/websocket"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
@@ -873,6 +870,7 @@ func Test_WebSocketsSubscription(t *testing.T) {
 // 	}
 // }
 
+/*
 func Test_RevertOnSCCallTransaction(t *testing.T) {
 	if testing.Short() {
 		t.Skip()
@@ -933,3 +931,4 @@ func Test_RevertOnSCCallTransaction(t *testing.T) {
 		assert.Equal(t, "Today is not juernes", unpackedData)
 	}
 }
+*/