Revert "CLOUDP-300869: Upgrade golangci to v1.64.5 (#3656)"

This reverts commit df33d20.

Author: blva

.github/workflows/code-health.yml

@@ -25,7 +25,7 @@ jobs:
       - name: golangci-lint
         uses: golangci/golangci-lint-action@2226d7cb06a077cd73e56eedd38eecad18e5d837
         with:
-          version: v1.64.5
+          version: v1.61.0
   unit-tests:
     env:
       COVERAGE: coverage.out

.golangci.yml

@@ -119,14 +119,14 @@ linters:
     - rowserrcheck # checks whether Err of rows is checked successfully [fast: false, auto-fix: false]
     - staticcheck # (megacheck) It's a set of rules from staticcheck. It's not the same thing as the staticcheck binary. The author of staticcheck doesn't support or approve the use of staticcheck as a library inside golangci-lint. [fast: false, auto-fix: false]
     - stylecheck # Stylecheck is a replacement for golint [fast: false, auto-fix: false]
+    - tenv # tenv is analyzer that detects using os.Setenv instead of t.Setenv since Go1.17 [fast: false, auto-fix: false]
     - testableexamples # linter checks if examples are testable (have an expected output)
     - testifylint # Checks usage of github.com/stretchr/testify. [fast: false, auto-fix: false]
     - thelper # thelper detects golang test helpers without t.Helper() call and checks the consistency of test helpers [fast: false, auto-fix: false]
     - typecheck # Like the front-end of a Go compiler, parses and type-checks Go code [fast: false, auto-fix: false]
     - unconvert # Remove unnecessary type conversions [fast: false, auto-fix: false]
     - unparam # Reports unused function parameters [fast: false, auto-fix: false]
     - unused # (megacheck) Checks Go code for unused constants, variables, functions and types [fast: false, auto-fix: false]
-    - usetesting # Reports uses of functions with replacement inside the testing package.
     - usestdlibvars # A linter that detect the possibility to use variables/constants from the Go standard library. [fast: true, auto-fix: false]
     - wastedassign # wastedassign finds wasted assignment statements. [fast: false, auto-fix: false]
     - whitespace # Tool for detection of leading and trailing whitespace [fast: true, auto-fix: true]

Makefile

@@ -1,6 +1,6 @@
 # A Self-Documenting Makefile: http://marmelab.com/blog/2016/02/29/auto-documented-makefile.html
 
-GOLANGCI_VERSION=v1.64.5
+GOLANGCI_VERSION=v1.61.0
 COVERAGE?=coverage.out
 GOCOVERDIR?=$(abspath cov)
 

build/ci/evergreen.yml

@@ -21,8 +21,8 @@ include:
 
 variables:
   - &go_linux_version
-      go_root: "/opt/golang/go1.24"
-      go_bin: "/opt/golang/go1.24/bin"
+      go_root: "/opt/golang/go1.23"
+      go_bin: "/opt/golang/go1.23/bin"
       go_base_path: ""
   - &go_env
       XDG_CONFIG_HOME: ${go_base_path}${workdir}

go.mod

@@ -1,6 +1,6 @@
 module github.com/mongodb/mongodb-atlas-cli/atlascli
 
-go 1.24.0
+go 1.23.6
 
 require (
 	cloud.google.com/go/kms v1.20.5

internal/api/watcher_test.go

@@ -457,7 +457,7 @@ func TestWatchInner(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			ctx := t.Context()
+			ctx := context.Background()
 			result, err := watchInner(ctx, tt.executor, tt.expect, tt.commandRequest)
 
 			// Check error

internal/cli/auth/login_test.go

@@ -18,6 +18,7 @@ package auth
 
 import (
 	"bytes"
+	"context"
 	"errors"
 	"fmt"
 	"testing"
@@ -88,7 +89,7 @@ func Test_loginOpts_Run(t *testing.T) {
 		ExpiresIn:       300,
 		Interval:        10,
 	}
-	ctx := t.Context()
+	ctx := context.TODO()
 	mockFlow.
 		EXPECT().
 		RequestCode(ctx).

internal/cli/auth/logout_test.go

@@ -18,6 +18,7 @@ package auth
 
 import (
 	"bytes"
+	"context"
 	"testing"
 
 	"github.com/golang/mock/gomock"
@@ -41,7 +42,7 @@ func Test_logoutOpts_Run(t *testing.T) {
 			Confirm: true,
 		},
 	}
-	ctx := t.Context()
+	ctx := context.TODO()
 	mockFlow.
 		EXPECT().
 		RevokeToken(ctx, gomock.Any(), gomock.Any()).
@@ -71,7 +72,7 @@ func Test_logoutOpts_Run_Keep(t *testing.T) {
 		},
 		keepConfig: true,
 	}
-	ctx := t.Context()
+	ctx := context.TODO()
 	mockFlow.
 		EXPECT().
 		RevokeToken(ctx, gomock.Any(), gomock.Any()).

internal/cli/auth/register_test.go

@@ -18,6 +18,7 @@ package auth
 
 import (
 	"bytes"
+	"context"
 	"testing"
 
 	"github.com/golang/mock/gomock"
@@ -36,7 +37,7 @@ func Test_registerOpts_Run(t *testing.T) {
 	mockStore := mocks.NewMockProjectOrgsLister(ctrl)
 
 	buf := new(bytes.Buffer)
-	ctx := t.Context()
+	ctx := context.TODO()
 
 	opts := &RegisterOpts{}
 	opts.NoBrowser = true

internal/cli/clusters/watch_test.go

@@ -17,6 +17,7 @@
 package clusters
 
 import (
+	"context"
 	"testing"
 
 	"github.com/golang/mock/gomock"
@@ -45,7 +46,7 @@ func TestWatch_Run(t *testing.T) {
 		Return(expected, nil).
 		Times(1)
 
-	require.NoError(t, opts.Run(t.Context()))
+	require.NoError(t, opts.Run(context.Background()))
 }
 
 func TestWatch_Run_FlexCluster(t *testing.T) {
@@ -66,5 +67,5 @@ func TestWatch_Run_FlexCluster(t *testing.T) {
 		Return(expected, nil).
 		Times(1)
 
-	require.NoError(t, opts.Run(t.Context()))
+	require.NoError(t, opts.Run(context.Background()))
 }