fix usestdlibvars linter

Signed-off-by: Spencer Schrock <[email protected]>

.golangci.yml

@@ -61,6 +61,7 @@ linters:
     - typecheck
     - unconvert
     - unused
+    - usestdlibvars
     - whitespace
     - wrapcheck
 linters-settings:

clients/cii_http_client.go

@@ -50,7 +50,7 @@ func (transport *expBackoffTransport) RoundTrip(req *http.Request) (*http.Respon
 func (client *httpClientCIIBestPractices) GetBadgeLevel(ctx context.Context, uri string) (BadgeLevel, error) {
 	repoURI := fmt.Sprintf("https://%s", uri)
 	url := fmt.Sprintf("https://www.bestpractices.dev/projects.json?url=%s", repoURI)
-	req, err := http.NewRequestWithContext(ctx, "GET", url, nil)
+	req, err := http.NewRequestWithContext(ctx, http.MethodGet, url, nil)
 	if err != nil {
 		return Unknown, fmt.Errorf("error during http.NewRequestWithContext: %w", err)
 	}

clients/gitlabrepo/branches.go

@@ -16,6 +16,7 @@ package gitlabrepo
 
 import (
 	"fmt"
+	"net/http"
 	"strings"
 	"sync"
 
@@ -84,10 +85,10 @@ func (handler *branchesHandler) setup() error {
 		if branch.Protected {
 			protectedBranch, resp, err := handler.getProtectedBranch(
 				handler.repourl.projectID, branch.Name)
-			if err != nil && resp.StatusCode != 403 {
+			if err != nil && resp.StatusCode != http.StatusForbidden {
 				handler.errSetup = fmt.Errorf("request for protected branch failed with error %w", err)
 				return
-			} else if resp.StatusCode == 403 {
+			} else if resp.StatusCode == http.StatusForbidden {
 				handler.errSetup = fmt.Errorf("incorrect permissions to fully check branch protection %w", err)
 				return
 			}

clients/gitlabrepo/branches_test.go

@@ -52,7 +52,7 @@ func TestGetBranches(t *testing.T) {
 			},
 			returnStatus: &gitlab.Response{
 				Response: &http.Response{
-					StatusCode: 200,
+					StatusCode: http.StatusOK,
 				},
 			},
 			branchReturn:        &gitlab.ProtectedBranch{},
@@ -69,7 +69,7 @@ func TestGetBranches(t *testing.T) {
 			},
 			returnStatus: &gitlab.Response{
 				Response: &http.Response{
-					StatusCode: 200,
+					StatusCode: http.StatusOK,
 				},
 			},
 		},

clients/gitlabrepo/issues.go

@@ -16,6 +16,7 @@ package gitlabrepo
 
 import (
 	"fmt"
+	"net/http"
 	"sync"
 
 	"github.com/xanzy/go-gitlab"
@@ -53,7 +54,7 @@ func (handler *issuesHandler) setup() error {
 		if err != nil && resp.StatusCode != 401 {
 			handler.errSetup = fmt.Errorf("unable to find access tokens associated with the project id: %w", err)
 			return
-		} else if resp.StatusCode == 401 {
+		} else if resp.StatusCode == http.StatusUnauthorized {
 			handler.errSetup = fmt.Errorf("insufficient permissions to check issue author associations %w", err)
 			return
 		}

cmd/internal/nuget/client_test.go

@@ -608,7 +608,7 @@ func testResult(wantErr bool, responseFileName string) (*http.Response, error) {
 	}
 	if wantErr && responseFileName == "text" {
 		return &http.Response{
-			StatusCode: 200,
+			StatusCode: http.StatusOK,
 			Body:       io.NopCloser(bytes.NewBufferString("text")),
 		}, nil
 	}
@@ -617,7 +617,7 @@ func testResult(wantErr bool, responseFileName string) (*http.Response, error) {
 		return nil, fmt.Errorf("%w", err)
 	}
 	return &http.Response{
-		StatusCode: 200,
+		StatusCode: http.StatusOK,
 		Body:       io.NopCloser(bytes.NewBufferString(string(content))),
 	}, nil
 }

cmd/package_managers_test.go

@@ -146,7 +146,7 @@ func Test_fetchGitRepositoryFromNPM(t *testing.T) {
 					}
 
 					return &http.Response{
-						StatusCode: 200,
+						StatusCode: http.StatusOK,
 						Body:       io.NopCloser(bytes.NewBufferString(tt.args.result)),
 					}, nil
 				}).AnyTimes()
@@ -456,7 +456,7 @@ func Test_fetchGitRepositoryFromPYPI(t *testing.T) {
 					}
 
 					return &http.Response{
-						StatusCode: 200,
+						StatusCode: http.StatusOK,
 						Body:       io.NopCloser(bytes.NewBufferString(tt.args.result)),
 					}, nil
 				}).AnyTimes()
@@ -725,7 +725,7 @@ func Test_fetchGitRepositoryFromRubyGems(t *testing.T) {
 					}
 
 					return &http.Response{
-						StatusCode: 200,
+						StatusCode: http.StatusOK,
 						Body:       io.NopCloser(bytes.NewBufferString(tt.args.result)),
 					}, nil
 				}).AnyTimes()