Other: Lint fixes

internal/safe/mutex.go

@@ -24,7 +24,7 @@ import (
 	"golang.org/x/exp/slices"
 )
 
-var nextMutexID uint64
+var nextMutexID uint64 // nolint:gochecknoglobals
 
 // Mutex is a mutex that can be locked and unlocked.
 type Mutex interface {

internal/user/types.go

@@ -58,11 +58,6 @@ func groupBy[Key comparable, Value any](items []Value, key func(Value) Key) map[
 	return groups
 }
 
-// sortAddr returns whether the first address should be sorted before the second.
-func sortAddr(addrIDA, addrIDB string, apiAddrs map[string]liteapi.Address) bool {
-	return apiAddrs[addrIDA].Order < apiAddrs[addrIDB].Order
-}
-
 // hexEncode returns the hexadecimal encoding of the given byte slice.
 func hexEncode(b []byte) []byte {
 	enc := make([]byte, hex.EncodedLen(len(b)))

tests/bdd_test.go

@@ -27,7 +27,6 @@ import (
 	"github.com/ProtonMail/proton-bridge/v2/internal/certs"
 	"github.com/ProtonMail/proton-bridge/v2/internal/user"
 	"github.com/cucumber/godog"
-	"github.com/stretchr/testify/require"
 	"gitlab.protontech.ch/go/liteapi/server/backend"
 )
 
@@ -53,8 +52,8 @@ func (s *scenario) reset(tb testing.TB) {
 	s.t = newTestCtx(tb)
 }
 
-func (s *scenario) close(tb testing.TB) {
-	require.NoError(tb, s.t.close(context.Background()))
+func (s *scenario) close(_ testing.TB) {
+	s.t.close(context.Background())
 }
 
 func TestFeatures(testingT *testing.T) {

tests/ctx_test.go

@@ -247,7 +247,7 @@ func (t *testCtx) getLastError() error {
 	return nil
 }
 
-func (t *testCtx) close(ctx context.Context) error {
+func (t *testCtx) close(ctx context.Context) {
 	for _, client := range t.imapClients {
 		if err := client.client.Logout(); err != nil {
 			logrus.WithError(err).Error("Failed to logout IMAP client")
@@ -267,8 +267,6 @@ func (t *testCtx) close(ctx context.Context) error {
 	t.api.Close()
 
 	t.events.close()
-
-	return nil
 }
 
 type eventCollector struct {