Fix linter warning

stellar · Nov 3, 2023 · cb51c0e · cb51c0e
1 parent b8efa95
commit cb51c0e
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/services/horizon/internal/db2/history/claimable_balances_test.go b/services/horizon/internal/db2/history/claimable_balances_test.go
@@ -3,9 +3,9 @@ package history
 import (
 	"database/sql"
 	"fmt"
-	"github.com/guregu/null"
 	"testing"
 
+	"github.com/guregu/null"
 	"github.com/stellar/go/services/horizon/internal/db2"
 	"github.com/stellar/go/services/horizon/internal/test"
 	"github.com/stellar/go/xdr"
@@ -264,8 +264,10 @@ func TestFindClaimableBalancesByDestinationWithLimit(t *testing.T) {
 	test.ResetHorizonDB(t, tt.HorizonDB)
 	q := &Q{tt.HorizonSession()}
 
-	q.SessionInterface.BeginTx(tt.Ctx, &sql.TxOptions{})
-	defer q.Rollback()
+	tt.Assert.NoError(q.SessionInterface.BeginTx(tt.Ctx, &sql.TxOptions{}))
+	defer func() {
+		_ = q.SessionInterface.Rollback()
+	}()
 
 	assetIssuer := "GA25GQLHJU3LPEJXEIAXK23AWEA5GWDUGRSHTQHDFT6HXHVMRULSQJUJ"
 	asset1 := xdr.MustNewCreditAsset("ASSET1", assetIssuer)

diff --git a/services/horizon/internal/db2/history/mock_claimable_balance_batch_insert_builder.go b/services/horizon/internal/db2/history/mock_claimable_balance_batch_insert_builder.go
@@ -2,8 +2,8 @@ package history
 
 import (
 	"context"
-	"github.com/stellar/go/support/db"
 
+	"github.com/stellar/go/support/db"
 	"github.com/stretchr/testify/mock"
 )
 

diff --git a/services/horizon/internal/ingest/verify_test.go b/services/horizon/internal/ingest/verify_test.go
@@ -366,7 +366,7 @@ func TestStateVerifier(t *testing.T) {
 	tt.Assert.NoError(changeProcessor.Commit(tt.Ctx))
 	tt.Assert.Equal(len(xdr.LedgerEntryTypeMap), len(coverage))
 
-	q.SessionInterface.Commit()
+	tt.Assert.NoError(q.SessionInterface.Commit())
 
 	q.UpdateLastLedgerIngest(tt.Ctx, checkpointLedger)