From 141abce2d5f37856465087d83fa099e464a24dda Mon Sep 17 00:00:00 2001 From: Ludovic Fernandez Date: Wed, 20 Mar 2024 10:26:03 +0100 Subject: [PATCH] chore: update linter --- .github/workflows/validate.yaml | 2 +- .golangci.yml | 7 ++++--- .semaphore/semaphore.yml | 2 +- cmd/traefik/traefik_test.go | 1 - integration/https_test.go | 4 ---- pkg/api/dashboard/dashboard_test.go | 2 -- pkg/api/handler_entrypoint_test.go | 1 - pkg/api/handler_http_test.go | 1 - pkg/api/handler_overview_test.go | 1 - pkg/api/handler_tcp_test.go | 1 - pkg/api/handler_test.go | 2 -- pkg/api/handler_udp_test.go | 1 - pkg/collector/collector.go | 2 +- pkg/config/kv/kv_node_test.go | 1 - pkg/config/kv/kv_test.go | 1 - pkg/config/runtime/runtime_http_test.go | 1 - pkg/config/runtime/runtime_tcp_test.go | 1 - pkg/config/runtime/runtime_test.go | 2 -- pkg/config/runtime/runtime_udp_test.go | 1 - pkg/config/static/static_config_test.go | 1 - pkg/healthcheck/healthcheck.go | 3 +-- pkg/healthcheck/healthcheck_test.go | 3 --- pkg/ip/checker_test.go | 3 --- pkg/ip/strategy_test.go | 3 --- pkg/log/log_test.go | 1 - pkg/metrics/prometheus_test.go | 2 -- .../accesslog/logger_formatters_test.go | 2 -- pkg/middlewares/accesslog/logger_test.go | 3 --- pkg/middlewares/accesslog/parser_test.go | 1 - pkg/middlewares/accesslog/save_retries_test.go | 2 -- pkg/middlewares/addprefix/add_prefix_test.go | 2 -- pkg/middlewares/auth/basic_auth_test.go | 1 - pkg/middlewares/auth/digest_auth_test.go | 1 - pkg/middlewares/buffering/buffering_test.go | 1 - pkg/middlewares/capture/capture_test.go | 1 - pkg/middlewares/compress/compress_test.go | 3 --- .../connectionheader/connectionheader_test.go | 1 - .../customerrors/custom_errors_test.go | 2 -- .../empty_backend_handler_test.go | 1 - .../forwardedheaders/forwarded_header_test.go | 2 -- pkg/middlewares/headers/header_test.go | 1 - pkg/middlewares/headers/headers_test.go | 1 - pkg/middlewares/headers/secure_test.go | 1 - pkg/middlewares/ipallowlist/ip_allowlist_test.go | 2 -- pkg/middlewares/ipwhitelist/ip_whitelist_test.go | 2 -- pkg/middlewares/metrics/metrics_test.go | 2 -- .../pass_tls_client_cert_test.go | 4 ---- pkg/middlewares/pipelining/pipelining_test.go | 1 - pkg/middlewares/ratelimiter/rate_limiter_test.go | 2 -- pkg/middlewares/redirect/redirect_regex_test.go | 1 - pkg/middlewares/redirect/redirect_scheme_test.go | 2 -- .../requestdecorator/hostresolver_test.go | 1 - .../requestdecorator/request_decorator_test.go | 3 --- pkg/middlewares/retry/retry_test.go | 2 -- pkg/middlewares/snicheck/snicheck_test.go | 1 - pkg/middlewares/stripprefix/strip_prefix_test.go | 1 - .../stripprefixregex/strip_prefix_regex_test.go | 1 - .../tcp/ipallowlist/ip_allowlist_test.go | 2 -- .../tcp/ipwhitelist/ip_whitelist_test.go | 2 -- pkg/muxer/http/mux_test.go | 7 ------- pkg/muxer/tcp/mux_test.go | 16 ---------------- pkg/provider/acme/local_store_test.go | 1 - pkg/provider/acme/provider_test.go | 6 ------ pkg/provider/aggregator/aggregator.go | 1 - .../constraints/constraints_labels_test.go | 1 - .../constraints/constraints_tags_test.go | 1 - pkg/provider/consulcatalog/config_test.go | 5 ----- pkg/provider/consulcatalog/convert_types_test.go | 1 - pkg/provider/docker/config_test.go | 8 -------- pkg/provider/docker/docker.go | 3 +-- pkg/provider/docker/swarm_test.go | 3 --- pkg/provider/ecs/config_test.go | 3 --- pkg/provider/ecs/ecs_test.go | 1 - pkg/provider/kubernetes/crd/kubernetes_test.go | 14 -------------- pkg/provider/kubernetes/gateway/client_test.go | 2 -- .../kubernetes/gateway/kubernetes_test.go | 11 ----------- .../kubernetes/ingress/annotations_test.go | 3 --- pkg/provider/kubernetes/ingress/client_test.go | 2 -- .../kubernetes/ingress/kubernetes_test.go | 7 ------- .../kubernetes/k8s/event_handler_test.go | 1 - pkg/provider/kv/consul/consul_test.go | 2 -- pkg/provider/marathon/builder_test.go | 3 +-- pkg/provider/marathon/config_test.go | 3 --- pkg/provider/marathon/label_test.go | 1 - pkg/provider/marathon/readiness_test.go | 1 - pkg/provider/nomad/config_test.go | 2 -- pkg/provider/nomad/nomad_test.go | 1 - pkg/provider/nomad/tag_test.go | 1 - pkg/provider/rancher/config_test.go | 2 -- pkg/provider/traefik/internal_test.go | 1 - pkg/redactor/redactor_doOnJSON_test.go | 1 - pkg/redactor/redactor_doOnStruct_test.go | 1 - pkg/rules/parser.go | 1 - pkg/rules/parser_test.go | 2 -- pkg/safe/routine_test.go | 1 - pkg/server/aggregator_test.go | 6 ------ pkg/server/cookie/cookie_test.go | 2 -- pkg/server/middleware/middlewares_test.go | 2 -- pkg/server/provider/provider_test.go | 2 -- pkg/server/router/router.go | 1 - pkg/server/router/router_test.go | 2 -- pkg/server/router/tcp/manager.go | 2 -- pkg/server/router/tcp/manager_test.go | 2 -- pkg/server/router/tcp/router_test.go | 2 -- pkg/server/router/udp/router.go | 2 -- pkg/server/router/udp/router_test.go | 2 -- pkg/server/routerfactory_test.go | 2 -- pkg/server/service/roundtripper_test.go | 2 -- pkg/server/service/service_test.go | 3 --- pkg/server/service/tcp/service_test.go | 1 - pkg/server/service/udp/service_test.go | 1 - pkg/tcp/proxy_test.go | 3 --- pkg/tcp/wrr_load_balancer_test.go | 2 -- pkg/tls/certificate_store_test.go | 1 - pkg/tls/tlsmanager_test.go | 2 -- pkg/tracing/operation_name_test.go | 3 --- pkg/types/domain_test.go | 3 --- pkg/types/tls_test.go | 2 -- 118 files changed, 10 insertions(+), 265 deletions(-) diff --git a/.github/workflows/validate.yaml b/.github/workflows/validate.yaml index 076a36b64..1ba3a6d61 100644 --- a/.github/workflows/validate.yaml +++ b/.github/workflows/validate.yaml @@ -7,7 +7,7 @@ on: env: GO_VERSION: '1.22' - GOLANGCI_LINT_VERSION: v1.56.2 + GOLANGCI_LINT_VERSION: v1.57.0 MISSSPELL_VERSION: v0.4.1 jobs: diff --git a/.golangci.yml b/.golangci.yml index 1a7860b91..743021e87 100644 --- a/.golangci.yml +++ b/.golangci.yml @@ -1,8 +1,5 @@ run: timeout: 10m - skip-files: [] - skip-dirs: - - pkg/provider/kubernetes/crd/generated/ linters-settings: govet: @@ -208,11 +205,15 @@ linters: - maintidx # kind of duplicate of gocyclo - nonamedreturns # Too strict - gosmopolitan # not relevant + - exportloopref # Useless with go1.22 + - musttag issues: exclude-use-default: false max-issues-per-linter: 0 max-same-issues: 0 + exclude-dirs: + - pkg/provider/kubernetes/crd/generated/ exclude: - 'Error return value of .((os\.)?std(out|err)\..*|.*Close|.*Flush|os\.Remove(All)?|.*printf?|os\.(Un)?Setenv). is not checked' - "should have a package comment, unless it's in another file for this package" diff --git a/.semaphore/semaphore.yml b/.semaphore/semaphore.yml index e45bca92c..ede5885d5 100644 --- a/.semaphore/semaphore.yml +++ b/.semaphore/semaphore.yml @@ -25,7 +25,7 @@ global_job_config: - export "PATH=${GOPATH}/bin:${PATH}" - mkdir -vp "${SEMAPHORE_GIT_DIR}" "${GOPATH}/bin" - export GOPROXY=https://proxy.golang.org,direct - - curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b "${GOPATH}/bin" v1.56.2 + - curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b "${GOPATH}/bin" v1.57.0 - curl -sSfL https://gist.githubusercontent.com/traefiker/6d7ac019c11d011e4f131bb2cca8900e/raw/goreleaser.sh | bash -s -- -b "${GOPATH}/bin" - checkout - cache restore traefik-$(checksum go.sum) diff --git a/cmd/traefik/traefik_test.go b/cmd/traefik/traefik_test.go index 26bc8643c..4b6cdcef3 100644 --- a/cmd/traefik/traefik_test.go +++ b/cmd/traefik/traefik_test.go @@ -94,7 +94,6 @@ func TestAppendCertMetric(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/integration/https_test.go b/integration/https_test.go index a4f7dc527..87a05480c 100644 --- a/integration/https_test.go +++ b/integration/https_test.go @@ -1133,8 +1133,6 @@ func (s *HTTPSSuite) TestWithDomainFronting() { } for _, test := range testCases { - test := test - req, err := http.NewRequest(http.MethodGet, "https://127.0.0.1:4443", nil) require.NoError(s.T(), err) req.Host = test.hostHeader @@ -1178,8 +1176,6 @@ func (s *HTTPSSuite) TestWithInvalidTLSOption() { } for _, test := range testCases { - test := test - tlsConfig := &tls.Config{ InsecureSkipVerify: true, } diff --git a/pkg/api/dashboard/dashboard_test.go b/pkg/api/dashboard/dashboard_test.go index ef0ffc635..28c734d94 100644 --- a/pkg/api/dashboard/dashboard_test.go +++ b/pkg/api/dashboard/dashboard_test.go @@ -42,7 +42,6 @@ func Test_safePrefix(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -91,7 +90,6 @@ func Test_ContentSecurityPolicy(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/api/handler_entrypoint_test.go b/pkg/api/handler_entrypoint_test.go index 36010cebe..d5d6a5d1f 100644 --- a/pkg/api/handler_entrypoint_test.go +++ b/pkg/api/handler_entrypoint_test.go @@ -210,7 +210,6 @@ func TestHandler_EntryPoints(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/api/handler_http_test.go b/pkg/api/handler_http_test.go index b814c69ab..7ed879a97 100644 --- a/pkg/api/handler_http_test.go +++ b/pkg/api/handler_http_test.go @@ -920,7 +920,6 @@ func TestHandler_HTTP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/api/handler_overview_test.go b/pkg/api/handler_overview_test.go index 428687e69..422c35f21 100644 --- a/pkg/api/handler_overview_test.go +++ b/pkg/api/handler_overview_test.go @@ -275,7 +275,6 @@ func TestHandler_Overview(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/api/handler_tcp_test.go b/pkg/api/handler_tcp_test.go index 85e5fdc19..af79bbce8 100644 --- a/pkg/api/handler_tcp_test.go +++ b/pkg/api/handler_tcp_test.go @@ -791,7 +791,6 @@ func TestHandler_TCP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/api/handler_test.go b/pkg/api/handler_test.go index 7bca3921a..9a2e3ae76 100644 --- a/pkg/api/handler_test.go +++ b/pkg/api/handler_test.go @@ -127,7 +127,6 @@ func TestHandler_RawData(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -252,7 +251,6 @@ func TestHandler_GetMiddleware(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/api/handler_udp_test.go b/pkg/api/handler_udp_test.go index 56c9b3a6f..9f230c466 100644 --- a/pkg/api/handler_udp_test.go +++ b/pkg/api/handler_udp_test.go @@ -530,7 +530,6 @@ func TestHandler_UDP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/collector/collector.go b/pkg/collector/collector.go index 2ecec4d1d..50ce0666d 100644 --- a/pkg/collector/collector.go +++ b/pkg/collector/collector.go @@ -65,7 +65,7 @@ func createBody(staticConfiguration *static.Configuration) (*bytes.Buffer, error } buf := new(bytes.Buffer) - err = json.NewEncoder(buf).Encode(data) //nolint:musttag // cannot be changed for historical reasons. + err = json.NewEncoder(buf).Encode(data) if err != nil { return nil, err } diff --git a/pkg/config/kv/kv_node_test.go b/pkg/config/kv/kv_node_test.go index c4a76b843..86b7d6ea2 100644 --- a/pkg/config/kv/kv_node_test.go +++ b/pkg/config/kv/kv_node_test.go @@ -244,7 +244,6 @@ func TestDecodeToNode(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/config/kv/kv_test.go b/pkg/config/kv/kv_test.go index 9dafcaab4..2a951cc51 100644 --- a/pkg/config/kv/kv_test.go +++ b/pkg/config/kv/kv_test.go @@ -87,7 +87,6 @@ func TestDecode(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/config/runtime/runtime_http_test.go b/pkg/config/runtime/runtime_http_test.go index 9ac028bfb..7ca957472 100644 --- a/pkg/config/runtime/runtime_http_test.go +++ b/pkg/config/runtime/runtime_http_test.go @@ -208,7 +208,6 @@ func TestGetRoutersByEntryPoints(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() runtimeConfig := NewConfig(test.conf) diff --git a/pkg/config/runtime/runtime_tcp_test.go b/pkg/config/runtime/runtime_tcp_test.go index 067e9ee09..b30509325 100644 --- a/pkg/config/runtime/runtime_tcp_test.go +++ b/pkg/config/runtime/runtime_tcp_test.go @@ -208,7 +208,6 @@ func TestGetTCPRoutersByEntryPoints(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() runtimeConfig := NewConfig(test.conf) diff --git a/pkg/config/runtime/runtime_test.go b/pkg/config/runtime/runtime_test.go index 3a7588be3..7b309b53d 100644 --- a/pkg/config/runtime/runtime_test.go +++ b/pkg/config/runtime/runtime_test.go @@ -663,8 +663,6 @@ func TestPopulateUsedBy(t *testing.T) { }, } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/config/runtime/runtime_udp_test.go b/pkg/config/runtime/runtime_udp_test.go index 87fe2476f..c43972674 100644 --- a/pkg/config/runtime/runtime_udp_test.go +++ b/pkg/config/runtime/runtime_udp_test.go @@ -189,7 +189,6 @@ func TestGetUDPRoutersByEntryPoints(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() runtimeConfig := NewConfig(test.conf) diff --git a/pkg/config/static/static_config_test.go b/pkg/config/static/static_config_test.go index 0fbd83056..691540580 100644 --- a/pkg/config/static/static_config_test.go +++ b/pkg/config/static/static_config_test.go @@ -26,7 +26,6 @@ func TestHasEntrypoint(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/healthcheck/healthcheck.go b/pkg/healthcheck/healthcheck.go index 68210e821..8bd8d1e17 100644 --- a/pkg/healthcheck/healthcheck.go +++ b/pkg/healthcheck/healthcheck.go @@ -137,9 +137,8 @@ func (hc *HealthCheck) SetBackendsConfiguration(parentCtx context.Context, backe hc.cancel = cancel for _, backend := range backends { - currentBackend := backend safe.Go(func() { - hc.execute(ctx, currentBackend) + hc.execute(ctx, backend) }) } } diff --git a/pkg/healthcheck/healthcheck_test.go b/pkg/healthcheck/healthcheck_test.go index 5b1ba2f39..73254b27e 100644 --- a/pkg/healthcheck/healthcheck_test.go +++ b/pkg/healthcheck/healthcheck_test.go @@ -96,7 +96,6 @@ func TestSetBackendsConfiguration(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -268,7 +267,6 @@ func TestNewRequest(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -358,7 +356,6 @@ func TestRequestOptions(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/ip/checker_test.go b/pkg/ip/checker_test.go index da2f07861..53bcb1311 100644 --- a/pkg/ip/checker_test.go +++ b/pkg/ip/checker_test.go @@ -36,7 +36,6 @@ func TestIsAuthorized(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -117,7 +116,6 @@ func TestNew(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -289,7 +287,6 @@ func TestContainsIsAllowed(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/ip/strategy_test.go b/pkg/ip/strategy_test.go index 1409b1d54..8bc5d285e 100644 --- a/pkg/ip/strategy_test.go +++ b/pkg/ip/strategy_test.go @@ -21,7 +21,6 @@ func TestRemoteAddrStrategy_GetIP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -61,7 +60,6 @@ func TestDepthStrategy_GetIP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -109,7 +107,6 @@ func TestTrustedIPsStrategy_GetIP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/log/log_test.go b/pkg/log/log_test.go index c81724ff3..704f2a83b 100644 --- a/pkg/log/log_test.go +++ b/pkg/log/log_test.go @@ -38,7 +38,6 @@ func TestLog(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { var buffer bytes.Buffer SetOutput(&buffer) diff --git a/pkg/metrics/prometheus_test.go b/pkg/metrics/prometheus_test.go index 33b6ba80d..7be5cbf64 100644 --- a/pkg/metrics/prometheus_test.go +++ b/pkg/metrics/prometheus_test.go @@ -64,7 +64,6 @@ func TestRegisterPromState(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { actualNbRegistries := 0 for _, prom := range test.prometheusSlice { @@ -419,7 +418,6 @@ func TestPrometheus(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.name, func(t *testing.T) { family := findMetricFamily(test.name, metricsFamilies) if family == nil { diff --git a/pkg/middlewares/accesslog/logger_formatters_test.go b/pkg/middlewares/accesslog/logger_formatters_test.go index cfa4281ec..028e3dbf7 100644 --- a/pkg/middlewares/accesslog/logger_formatters_test.go +++ b/pkg/middlewares/accesslog/logger_formatters_test.go @@ -86,7 +86,6 @@ func TestCommonLogFormatter_Format(t *testing.T) { t.Setenv("TZ", "Etc/GMT+9") for _, test := range testCases { - test := test t.Run(test.name, func(t *testing.T) { t.Parallel() @@ -150,7 +149,6 @@ func Test_toLog(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/accesslog/logger_test.go b/pkg/middlewares/accesslog/logger_test.go index b599e3e8d..ebec9173e 100644 --- a/pkg/middlewares/accesslog/logger_test.go +++ b/pkg/middlewares/accesslog/logger_test.go @@ -178,7 +178,6 @@ func TestLoggerHeaderFields(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { logFile, err := os.CreateTemp(t.TempDir(), "*.log") require.NoError(t, err) @@ -466,7 +465,6 @@ func TestLoggerJSON(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -684,7 +682,6 @@ func TestNewLogHandlerOutputStdout(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { // NOTE: It is not possible to run these cases in parallel because we capture Stdout diff --git a/pkg/middlewares/accesslog/parser_test.go b/pkg/middlewares/accesslog/parser_test.go index 1712f1a8c..1e194dd7e 100644 --- a/pkg/middlewares/accesslog/parser_test.go +++ b/pkg/middlewares/accesslog/parser_test.go @@ -60,7 +60,6 @@ func TestParseAccessLog(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/accesslog/save_retries_test.go b/pkg/middlewares/accesslog/save_retries_test.go index add4cc28f..ca296b2a5 100644 --- a/pkg/middlewares/accesslog/save_retries_test.go +++ b/pkg/middlewares/accesslog/save_retries_test.go @@ -28,8 +28,6 @@ func TestSaveRetries(t *testing.T) { } for _, test := range tests { - test := test - t.Run(fmt.Sprintf("%d retries", test.requestAttempt), func(t *testing.T) { t.Parallel() saveRetries := &SaveRetries{} diff --git a/pkg/middlewares/addprefix/add_prefix_test.go b/pkg/middlewares/addprefix/add_prefix_test.go index 9ec4df5fa..e56a13d9f 100644 --- a/pkg/middlewares/addprefix/add_prefix_test.go +++ b/pkg/middlewares/addprefix/add_prefix_test.go @@ -29,7 +29,6 @@ func TestNewAddPrefix(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -75,7 +74,6 @@ func TestAddPrefix(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/auth/basic_auth_test.go b/pkg/middlewares/auth/basic_auth_test.go index 893443b96..196bd3669 100644 --- a/pkg/middlewares/auth/basic_auth_test.go +++ b/pkg/middlewares/auth/basic_auth_test.go @@ -210,7 +210,6 @@ func TestBasicAuthUsersFromFile(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/auth/digest_auth_test.go b/pkg/middlewares/auth/digest_auth_test.go index a972223b4..1494205cd 100644 --- a/pkg/middlewares/auth/digest_auth_test.go +++ b/pkg/middlewares/auth/digest_auth_test.go @@ -88,7 +88,6 @@ func TestDigestAuthUsersFromFile(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/buffering/buffering_test.go b/pkg/middlewares/buffering/buffering_test.go index 2d8eec37e..adf2c6daa 100644 --- a/pkg/middlewares/buffering/buffering_test.go +++ b/pkg/middlewares/buffering/buffering_test.go @@ -48,7 +48,6 @@ func TestBuffering(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/capture/capture_test.go b/pkg/middlewares/capture/capture_test.go index 2254b0fea..7ef7fcade 100644 --- a/pkg/middlewares/capture/capture_test.go +++ b/pkg/middlewares/capture/capture_test.go @@ -217,7 +217,6 @@ func TestCloseNotifier(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/compress/compress_test.go b/pkg/middlewares/compress/compress_test.go index 3bdaf9678..353174fa7 100644 --- a/pkg/middlewares/compress/compress_test.go +++ b/pkg/middlewares/compress/compress_test.go @@ -124,7 +124,6 @@ func TestShouldNotCompressWhenSpecificContentType(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -326,8 +325,6 @@ func TestMinResponseBodyBytes(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.name, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/connectionheader/connectionheader_test.go b/pkg/middlewares/connectionheader/connectionheader_test.go index 7ee047bd0..bd41d58d8 100644 --- a/pkg/middlewares/connectionheader/connectionheader_test.go +++ b/pkg/middlewares/connectionheader/connectionheader_test.go @@ -47,7 +47,6 @@ func TestRemover(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/customerrors/custom_errors_test.go b/pkg/middlewares/customerrors/custom_errors_test.go index 437c560cb..046ecb665 100644 --- a/pkg/middlewares/customerrors/custom_errors_test.go +++ b/pkg/middlewares/customerrors/custom_errors_test.go @@ -157,7 +157,6 @@ func TestHandler(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -293,7 +292,6 @@ func TestNewResponseRecorder(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/emptybackendhandler/empty_backend_handler_test.go b/pkg/middlewares/emptybackendhandler/empty_backend_handler_test.go index fddb9c768..299fe9c31 100644 --- a/pkg/middlewares/emptybackendhandler/empty_backend_handler_test.go +++ b/pkg/middlewares/emptybackendhandler/empty_backend_handler_test.go @@ -28,7 +28,6 @@ func TestEmptyBackendHandler(t *testing.T) { } for _, test := range testCases { - test := test t.Run(fmt.Sprintf("amount servers %d", test.amountServer), func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/forwardedheaders/forwarded_header_test.go b/pkg/middlewares/forwardedheaders/forwarded_header_test.go index 4d2dab2fc..8e1d10925 100644 --- a/pkg/middlewares/forwardedheaders/forwarded_header_test.go +++ b/pkg/middlewares/forwardedheaders/forwarded_header_test.go @@ -272,7 +272,6 @@ func TestServeHTTP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -369,7 +368,6 @@ func Test_isWebsocketRequest(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/headers/header_test.go b/pkg/middlewares/headers/header_test.go index 363b7e737..17745e826 100644 --- a/pkg/middlewares/headers/header_test.go +++ b/pkg/middlewares/headers/header_test.go @@ -52,7 +52,6 @@ func TestNewHeader_customRequestHeader(t *testing.T) { emptyHandler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {}) for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/headers/headers_test.go b/pkg/middlewares/headers/headers_test.go index ceef014dd..41ec273a2 100644 --- a/pkg/middlewares/headers/headers_test.go +++ b/pkg/middlewares/headers/headers_test.go @@ -59,7 +59,6 @@ func TestNew_allowedHosts(t *testing.T) { require.NoError(t, err) for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/headers/secure_test.go b/pkg/middlewares/headers/secure_test.go index ce2b93a7c..050288dfa 100644 --- a/pkg/middlewares/headers/secure_test.go +++ b/pkg/middlewares/headers/secure_test.go @@ -180,7 +180,6 @@ func Test_newSecure_modifyResponse(t *testing.T) { emptyHandler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { w.WriteHeader(http.StatusOK) }) for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/ipallowlist/ip_allowlist_test.go b/pkg/middlewares/ipallowlist/ip_allowlist_test.go index df2e49835..130c6f7cc 100644 --- a/pkg/middlewares/ipallowlist/ip_allowlist_test.go +++ b/pkg/middlewares/ipallowlist/ip_allowlist_test.go @@ -33,7 +33,6 @@ func TestNewIPAllowLister(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -76,7 +75,6 @@ func TestIPAllowLister_ServeHTTP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/ipwhitelist/ip_whitelist_test.go b/pkg/middlewares/ipwhitelist/ip_whitelist_test.go index 2fabbfae7..13b6e9e5e 100644 --- a/pkg/middlewares/ipwhitelist/ip_whitelist_test.go +++ b/pkg/middlewares/ipwhitelist/ip_whitelist_test.go @@ -33,7 +33,6 @@ func TestNewIPWhiteLister(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -76,7 +75,6 @@ func TestIPWhiteLister_ServeHTTP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/metrics/metrics_test.go b/pkg/middlewares/metrics/metrics_test.go index 9390e49f5..c6b9f7eae 100644 --- a/pkg/middlewares/metrics/metrics_test.go +++ b/pkg/middlewares/metrics/metrics_test.go @@ -86,7 +86,6 @@ func TestCloseNotifier(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -120,7 +119,6 @@ func Test_getMethod(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.method, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/passtlsclientcert/pass_tls_client_cert_test.go b/pkg/middlewares/passtlsclientcert/pass_tls_client_cert_test.go index 96c1bdffa..911fd3cb7 100644 --- a/pkg/middlewares/passtlsclientcert/pass_tls_client_cert_test.go +++ b/pkg/middlewares/passtlsclientcert/pass_tls_client_cert_test.go @@ -310,7 +310,6 @@ func TestPassTLSClientCert_PEM(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -533,7 +532,6 @@ func TestPassTLSClientCert_certInfo(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -604,7 +602,6 @@ WqeUSNGYV//RunTeuRDAf5OxehERb1srzBXhRZ3cZdzXbgR/`, } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -645,7 +642,6 @@ func Test_getSANs(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/pipelining/pipelining_test.go b/pkg/middlewares/pipelining/pipelining_test.go index 80a03d2d6..9ad64f103 100644 --- a/pkg/middlewares/pipelining/pipelining_test.go +++ b/pkg/middlewares/pipelining/pipelining_test.go @@ -54,7 +54,6 @@ func TestNew(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/ratelimiter/rate_limiter_test.go b/pkg/middlewares/ratelimiter/rate_limiter_test.go index 51a077819..e3ff92ec5 100644 --- a/pkg/middlewares/ratelimiter/rate_limiter_test.go +++ b/pkg/middlewares/ratelimiter/rate_limiter_test.go @@ -89,7 +89,6 @@ func TestNewRateLimiter(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -248,7 +247,6 @@ func TestRateLimit(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { if test.loadDuration >= time.Minute && testing.Short() { t.Skip("skipping test in short mode.") diff --git a/pkg/middlewares/redirect/redirect_regex_test.go b/pkg/middlewares/redirect/redirect_regex_test.go index d6346cf75..0c04d9ef8 100644 --- a/pkg/middlewares/redirect/redirect_regex_test.go +++ b/pkg/middlewares/redirect/redirect_regex_test.go @@ -154,7 +154,6 @@ func TestRedirectRegexHandler(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/redirect/redirect_scheme_test.go b/pkg/middlewares/redirect/redirect_scheme_test.go index 8de531a2c..7ff97bba4 100644 --- a/pkg/middlewares/redirect/redirect_scheme_test.go +++ b/pkg/middlewares/redirect/redirect_scheme_test.go @@ -283,8 +283,6 @@ func TestRedirectSchemeHandler(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/requestdecorator/hostresolver_test.go b/pkg/middlewares/requestdecorator/hostresolver_test.go index ed199c387..f1c8c39a2 100644 --- a/pkg/middlewares/requestdecorator/hostresolver_test.go +++ b/pkg/middlewares/requestdecorator/hostresolver_test.go @@ -35,7 +35,6 @@ func TestCNAMEFlatten(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/requestdecorator/request_decorator_test.go b/pkg/middlewares/requestdecorator/request_decorator_test.go index 03c8f4018..c289fcaec 100644 --- a/pkg/middlewares/requestdecorator/request_decorator_test.go +++ b/pkg/middlewares/requestdecorator/request_decorator_test.go @@ -38,7 +38,6 @@ func TestRequestHost(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -80,7 +79,6 @@ func TestRequestFlattening(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -173,7 +171,6 @@ func Test_parseHost(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/retry/retry_test.go b/pkg/middlewares/retry/retry_test.go index 5645e007c..be2606ce4 100644 --- a/pkg/middlewares/retry/retry_test.go +++ b/pkg/middlewares/retry/retry_test.go @@ -101,7 +101,6 @@ func TestRetry(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -272,7 +271,6 @@ func TestRetryWebsocket(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/snicheck/snicheck_test.go b/pkg/middlewares/snicheck/snicheck_test.go index a9d57bdf6..d7411e555 100644 --- a/pkg/middlewares/snicheck/snicheck_test.go +++ b/pkg/middlewares/snicheck/snicheck_test.go @@ -37,7 +37,6 @@ func TestSNICheck_ServeHTTP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/stripprefix/strip_prefix_test.go b/pkg/middlewares/stripprefix/strip_prefix_test.go index 5c9d7171e..b592224e9 100644 --- a/pkg/middlewares/stripprefix/strip_prefix_test.go +++ b/pkg/middlewares/stripprefix/strip_prefix_test.go @@ -178,7 +178,6 @@ func TestStripPrefix(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/stripprefixregex/strip_prefix_regex_test.go b/pkg/middlewares/stripprefixregex/strip_prefix_regex_test.go index 763985ba9..18e831d35 100644 --- a/pkg/middlewares/stripprefixregex/strip_prefix_regex_test.go +++ b/pkg/middlewares/stripprefixregex/strip_prefix_regex_test.go @@ -108,7 +108,6 @@ func TestStripPrefixRegex(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.path, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/tcp/ipallowlist/ip_allowlist_test.go b/pkg/middlewares/tcp/ipallowlist/ip_allowlist_test.go index 4f3b59e15..d55a1fffa 100644 --- a/pkg/middlewares/tcp/ipallowlist/ip_allowlist_test.go +++ b/pkg/middlewares/tcp/ipallowlist/ip_allowlist_test.go @@ -39,7 +39,6 @@ func TestNewIPAllowLister(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -81,7 +80,6 @@ func TestIPAllowLister_ServeHTTP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/middlewares/tcp/ipwhitelist/ip_whitelist_test.go b/pkg/middlewares/tcp/ipwhitelist/ip_whitelist_test.go index 67780dfb6..e419f6bfe 100644 --- a/pkg/middlewares/tcp/ipwhitelist/ip_whitelist_test.go +++ b/pkg/middlewares/tcp/ipwhitelist/ip_whitelist_test.go @@ -39,7 +39,6 @@ func TestNewIPWhiteLister(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -81,7 +80,6 @@ func TestIPWhiteLister_ServeHTTP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/muxer/http/mux_test.go b/pkg/muxer/http/mux_test.go index 0f6649dc7..768c7b898 100644 --- a/pkg/muxer/http/mux_test.go +++ b/pkg/muxer/http/mux_test.go @@ -653,8 +653,6 @@ func Test_addRoute(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -808,14 +806,12 @@ func Test_addRoutePriority(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() muxer, err := NewMuxer() require.NoError(t, err) for _, route := range test.cases { - route := route handler := http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) { w.Header().Set("X-From", route.xFrom) }) @@ -900,7 +896,6 @@ func TestHostRegexp(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -964,7 +959,6 @@ func TestParseDomains(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.expression, func(t *testing.T) { t.Parallel() @@ -1027,7 +1021,6 @@ func TestAbsoluteFormURL(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/muxer/tcp/mux_test.go b/pkg/muxer/tcp/mux_test.go index 827d295b1..45820acdb 100644 --- a/pkg/muxer/tcp/mux_test.go +++ b/pkg/muxer/tcp/mux_test.go @@ -502,8 +502,6 @@ func Test_addTCPRoute(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -614,7 +612,6 @@ func TestParseHostSNI(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.expression, func(t *testing.T) { t.Parallel() @@ -659,8 +656,6 @@ func Test_HostSNICatchAll(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -761,8 +756,6 @@ func Test_HostSNI(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -876,8 +869,6 @@ func Test_HostSNIRegexp(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -969,8 +960,6 @@ func Test_ClientIP(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -1038,8 +1027,6 @@ func Test_ALPN(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -1107,8 +1094,6 @@ func Test_Priority(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -1117,7 +1102,6 @@ func Test_Priority(t *testing.T) { matchedRule := "" for rule, priority := range test.rules { - rule := rule err := muxer.AddRoute(rule, priority, tcp.HandlerFunc(func(conn tcp.WriteCloser) { matchedRule = rule })) diff --git a/pkg/provider/acme/local_store_test.go b/pkg/provider/acme/local_store_test.go index 44d4ab0fc..4614078c0 100644 --- a/pkg/provider/acme/local_store_test.go +++ b/pkg/provider/acme/local_store_test.go @@ -44,7 +44,6 @@ func TestLocalStore_GetAccount(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { s := NewLocalStore(test.filename) diff --git a/pkg/provider/acme/provider_test.go b/pkg/provider/acme/provider_test.go index 2a55de2c9..8a72be6de 100644 --- a/pkg/provider/acme/provider_test.go +++ b/pkg/provider/acme/provider_test.go @@ -166,7 +166,6 @@ func TestGetUncheckedCertificates(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -241,7 +240,6 @@ func TestProvider_sanitizeDomains(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -423,7 +421,6 @@ func TestDeleteUnnecessaryDomains(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -497,7 +494,6 @@ func TestIsAccountMatchingCaServer(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -573,7 +569,6 @@ func TestInitAccount(t *testing.T) { }, } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -632,7 +627,6 @@ func Test_getCertificateRenewDurations(t *testing.T) { }, } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/aggregator/aggregator.go b/pkg/provider/aggregator/aggregator.go index 3ddb1c3f2..888d51fa0 100644 --- a/pkg/provider/aggregator/aggregator.go +++ b/pkg/provider/aggregator/aggregator.go @@ -183,7 +183,6 @@ func (p ProviderAggregator) Provide(configurationChan chan<- dynamic.Message, po } for _, prd := range p.providers { - prd := prd safe.Go(func() { p.launchProvider(configurationChan, pool, prd) }) diff --git a/pkg/provider/constraints/constraints_labels_test.go b/pkg/provider/constraints/constraints_labels_test.go index 43fb9e0c6..d17d40915 100644 --- a/pkg/provider/constraints/constraints_labels_test.go +++ b/pkg/provider/constraints/constraints_labels_test.go @@ -188,7 +188,6 @@ func TestMatchLabels(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.expr, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/constraints/constraints_tags_test.go b/pkg/provider/constraints/constraints_tags_test.go index a563b0b30..531c715c3 100644 --- a/pkg/provider/constraints/constraints_tags_test.go +++ b/pkg/provider/constraints/constraints_tags_test.go @@ -95,7 +95,6 @@ func TestMatchTags(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.expr, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/consulcatalog/config_test.go b/pkg/provider/consulcatalog/config_test.go index 9039f4697..5bfba067e 100644 --- a/pkg/provider/consulcatalog/config_test.go +++ b/pkg/provider/consulcatalog/config_test.go @@ -276,7 +276,6 @@ func TestDefaultRule(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3161,8 +3160,6 @@ func Test_buildConfiguration(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3232,8 +3229,6 @@ func TestNamespaces(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/consulcatalog/convert_types_test.go b/pkg/provider/consulcatalog/convert_types_test.go index 8dece14c2..e8ac70367 100644 --- a/pkg/provider/consulcatalog/convert_types_test.go +++ b/pkg/provider/consulcatalog/convert_types_test.go @@ -52,7 +52,6 @@ func Test_tagsToNeutralLabels(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/docker/config_test.go b/pkg/provider/docker/config_test.go index 0b835c6c4..3a9fe896f 100644 --- a/pkg/provider/docker/config_test.go +++ b/pkg/provider/docker/config_test.go @@ -369,7 +369,6 @@ func TestDefaultRule(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3484,8 +3483,6 @@ func Test_buildConfiguration(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3668,7 +3665,6 @@ func TestDockerGetIPPort(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3745,7 +3741,6 @@ func TestDockerGetPort(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3847,7 +3842,6 @@ func TestDockerGetIPAddress(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3915,7 +3909,6 @@ func TestSwarmGetIPAddress(t *testing.T) { } for serviceID, test := range testCases { - test := test t.Run(strconv.Itoa(serviceID), func(t *testing.T) { t.Parallel() @@ -3950,7 +3943,6 @@ func TestSwarmGetPort(t *testing.T) { } for serviceID, test := range testCases { - test := test t.Run(strconv.Itoa(serviceID), func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/docker/docker.go b/pkg/provider/docker/docker.go index 0516620ce..5775b2785 100644 --- a/pkg/provider/docker/docker.go +++ b/pkg/provider/docker/docker.go @@ -456,8 +456,7 @@ func (p *Provider) listServices(ctx context.Context, dockerClient client.APIClie networkMap := make(map[string]*dockertypes.NetworkResource) for _, network := range networkList { - networkToAdd := network - networkMap[network.ID] = &networkToAdd + networkMap[network.ID] = &network } var dockerDataList []dockerData diff --git a/pkg/provider/docker/swarm_test.go b/pkg/provider/docker/swarm_test.go index 541af22f2..70c3aacee 100644 --- a/pkg/provider/docker/swarm_test.go +++ b/pkg/provider/docker/swarm_test.go @@ -79,7 +79,6 @@ func TestListTasks(t *testing.T) { } for caseID, test := range testCases { - test := test t.Run(strconv.Itoa(caseID), func(t *testing.T) { t.Parallel() @@ -271,7 +270,6 @@ func TestListServices(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -392,7 +390,6 @@ func TestSwarmTaskParsing(t *testing.T) { } for caseID, test := range testCases { - test := test t.Run(strconv.Itoa(caseID), func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/ecs/config_test.go b/pkg/provider/ecs/config_test.go index f00506094..3d8f912b8 100644 --- a/pkg/provider/ecs/config_test.go +++ b/pkg/provider/ecs/config_test.go @@ -340,7 +340,6 @@ func TestDefaultRule(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3071,8 +3070,6 @@ func Test_buildConfiguration(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/ecs/ecs_test.go b/pkg/provider/ecs/ecs_test.go index ede115a41..b05bd2165 100644 --- a/pkg/provider/ecs/ecs_test.go +++ b/pkg/provider/ecs/ecs_test.go @@ -68,7 +68,6 @@ func TestChunkIDs(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kubernetes/crd/kubernetes_test.go b/pkg/provider/kubernetes/crd/kubernetes_test.go index 389b9a659..6724e5c63 100644 --- a/pkg/provider/kubernetes/crd/kubernetes_test.go +++ b/pkg/provider/kubernetes/crd/kubernetes_test.go @@ -1422,8 +1422,6 @@ func TestLoadIngressRouteTCPs(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4169,7 +4167,6 @@ func TestLoadIngressRoutes(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4650,8 +4647,6 @@ func TestLoadIngressRouteUDPs(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4727,8 +4722,6 @@ func TestParseServiceProtocol(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4959,7 +4952,6 @@ func TestGetServicePort(t *testing.T) { }, } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -5931,8 +5923,6 @@ func TestCrossNamespace(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -6226,8 +6216,6 @@ func TestExternalNameService(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -6434,8 +6422,6 @@ func TestNativeLB(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kubernetes/gateway/client_test.go b/pkg/provider/kubernetes/gateway/client_test.go index 794f7d6a8..069aa7628 100644 --- a/pkg/provider/kubernetes/gateway/client_test.go +++ b/pkg/provider/kubernetes/gateway/client_test.go @@ -233,8 +233,6 @@ func TestStatusEquals(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kubernetes/gateway/kubernetes_test.go b/pkg/provider/kubernetes/gateway/kubernetes_test.go index 6fdfd82f0..ea0e7c504 100644 --- a/pkg/provider/kubernetes/gateway/kubernetes_test.go +++ b/pkg/provider/kubernetes/gateway/kubernetes_test.go @@ -1471,7 +1471,6 @@ func TestLoadHTTPRoutes(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -2206,7 +2205,6 @@ func TestLoadTCPRoutes(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -3299,7 +3297,6 @@ func TestLoadTLSRoutes(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4251,7 +4248,6 @@ func TestLoadMixedRoutes(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4361,7 +4357,6 @@ func Test_hostRule(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() rule, err := hostRule(test.hostnames) @@ -4592,7 +4587,6 @@ func Test_extractRule(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4657,7 +4651,6 @@ func Test_hostSNIRule(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -4978,7 +4971,6 @@ func Test_shouldAttach(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -5082,7 +5074,6 @@ func Test_matchingHostnames(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -5194,7 +5185,6 @@ func Test_getAllowedRoutes(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -5240,7 +5230,6 @@ func Test_makeListenerKey(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kubernetes/ingress/annotations_test.go b/pkg/provider/kubernetes/ingress/annotations_test.go index 45ef1cf78..7dbf309f9 100644 --- a/pkg/provider/kubernetes/ingress/annotations_test.go +++ b/pkg/provider/kubernetes/ingress/annotations_test.go @@ -80,7 +80,6 @@ func Test_parseRouterConfig(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -155,7 +154,6 @@ func Test_parseServiceConfig(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -239,7 +237,6 @@ func Test_convertAnnotations(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kubernetes/ingress/client_test.go b/pkg/provider/kubernetes/ingress/client_test.go index 9ab39fdc6..725ba3491 100644 --- a/pkg/provider/kubernetes/ingress/client_test.go +++ b/pkg/provider/kubernetes/ingress/client_test.go @@ -47,7 +47,6 @@ func TestTranslateNotFoundError(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -125,7 +124,6 @@ func TestIsLoadBalancerIngressEquals(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kubernetes/ingress/kubernetes_test.go b/pkg/provider/kubernetes/ingress/kubernetes_test.go index 329ca0735..abd08ab01 100644 --- a/pkg/provider/kubernetes/ingress/kubernetes_test.go +++ b/pkg/provider/kubernetes/ingress/kubernetes_test.go @@ -1602,8 +1602,6 @@ func TestLoadConfigurationFromIngresses(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -1748,8 +1746,6 @@ func TestLoadConfigurationFromIngressesWithExternalNameServices(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -1828,8 +1824,6 @@ func TestLoadConfigurationFromIngressesWithNativeLB(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -2031,7 +2025,6 @@ func TestGetCertificates(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kubernetes/k8s/event_handler_test.go b/pkg/provider/kubernetes/k8s/event_handler_test.go index e8b7a31a2..323b4f550 100644 --- a/pkg/provider/kubernetes/k8s/event_handler_test.go +++ b/pkg/provider/kubernetes/k8s/event_handler_test.go @@ -507,7 +507,6 @@ func Test_detectChanges(t *testing.T) { }, } for _, test := range tests { - test := test t.Run(test.name, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/kv/consul/consul_test.go b/pkg/provider/kv/consul/consul_test.go index 286c0cd9c..765463cf3 100644 --- a/pkg/provider/kv/consul/consul_test.go +++ b/pkg/provider/kv/consul/consul_test.go @@ -35,8 +35,6 @@ func TestNamespaces(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/marathon/builder_test.go b/pkg/provider/marathon/builder_test.go index e18110787..4cf7a3468 100644 --- a/pkg/provider/marathon/builder_test.go +++ b/pkg/provider/marathon/builder_test.go @@ -121,8 +121,7 @@ func readinessCheckResult(taskID string, ready bool) func(*marathon.Application) func withTasks(tasks ...marathon.Task) func(*marathon.Application) { return func(application *marathon.Application) { for _, task := range tasks { - tu := task - application.Tasks = append(application.Tasks, &tu) + application.Tasks = append(application.Tasks, &task) } } } diff --git a/pkg/provider/marathon/config_test.go b/pkg/provider/marathon/config_test.go index 6e78830c5..8400b877a 100644 --- a/pkg/provider/marathon/config_test.go +++ b/pkg/provider/marathon/config_test.go @@ -2035,7 +2035,6 @@ func TestBuildConfiguration(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -2089,7 +2088,6 @@ func TestApplicationFilterEnabled(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -2492,7 +2490,6 @@ func TestGetServer(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/marathon/label_test.go b/pkg/provider/marathon/label_test.go index 472b91d23..1ce8901e4 100644 --- a/pkg/provider/marathon/label_test.go +++ b/pkg/provider/marathon/label_test.go @@ -123,7 +123,6 @@ func TestGetConfiguration(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/marathon/readiness_test.go b/pkg/provider/marathon/readiness_test.go index 8d61c01c4..86e063815 100644 --- a/pkg/provider/marathon/readiness_test.go +++ b/pkg/provider/marathon/readiness_test.go @@ -115,7 +115,6 @@ func TestEnabledReadinessChecker(t *testing.T) { } for _, test := range tests { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() rc := testReadinessChecker() diff --git a/pkg/provider/nomad/config_test.go b/pkg/provider/nomad/config_test.go index 9c964586b..84a4791bc 100644 --- a/pkg/provider/nomad/config_test.go +++ b/pkg/provider/nomad/config_test.go @@ -2765,8 +2765,6 @@ func TestNamespaces(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/nomad/nomad_test.go b/pkg/provider/nomad/nomad_test.go index f3fcd8894..745a85a23 100644 --- a/pkg/provider/nomad/nomad_test.go +++ b/pkg/provider/nomad/nomad_test.go @@ -118,7 +118,6 @@ func TestProvider_SetDefaults_Endpoint(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { for k, v := range test.envs { t.Setenv(k, v) diff --git a/pkg/provider/nomad/tag_test.go b/pkg/provider/nomad/tag_test.go index e15f5486c..69be3addb 100644 --- a/pkg/provider/nomad/tag_test.go +++ b/pkg/provider/nomad/tag_test.go @@ -97,7 +97,6 @@ func Test_tagsToLabels(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/rancher/config_test.go b/pkg/provider/rancher/config_test.go index dbed070c9..633e85415 100644 --- a/pkg/provider/rancher/config_test.go +++ b/pkg/provider/rancher/config_test.go @@ -1262,8 +1262,6 @@ func Test_buildConfiguration(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/provider/traefik/internal_test.go b/pkg/provider/traefik/internal_test.go index 1e7d5b282..133273044 100644 --- a/pkg/provider/traefik/internal_test.go +++ b/pkg/provider/traefik/internal_test.go @@ -257,7 +257,6 @@ func Test_createConfiguration(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/redactor/redactor_doOnJSON_test.go b/pkg/redactor/redactor_doOnJSON_test.go index ddae9c579..03e4082a1 100644 --- a/pkg/redactor/redactor_doOnJSON_test.go +++ b/pkg/redactor/redactor_doOnJSON_test.go @@ -59,7 +59,6 @@ func Test_doOnJSON_simple(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.name, func(t *testing.T) { t.Parallel() output := doOnJSON(test.input) diff --git a/pkg/redactor/redactor_doOnStruct_test.go b/pkg/redactor/redactor_doOnStruct_test.go index 96fe5edc9..af1f1faad 100644 --- a/pkg/redactor/redactor_doOnStruct_test.go +++ b/pkg/redactor/redactor_doOnStruct_test.go @@ -369,7 +369,6 @@ func Test_doOnStruct(t *testing.T) { } for _, test := range testCase { - test := test t.Run(test.name, func(t *testing.T) { t.Parallel() val := reflect.ValueOf(test.base).Elem() diff --git a/pkg/rules/parser.go b/pkg/rules/parser.go index 061895d3c..2555c01b9 100644 --- a/pkg/rules/parser.go +++ b/pkg/rules/parser.go @@ -31,7 +31,6 @@ func NewParser(matchers []string) (predicate.Parser, error) { parserFuncs := make(map[string]interface{}) for _, matcherName := range matchers { - matcherName := matcherName fn := func(value ...string) TreeBuilder { return func() *Tree { return &Tree{ diff --git a/pkg/rules/parser_test.go b/pkg/rules/parser_test.go index 503ab4298..b8aed5d5a 100644 --- a/pkg/rules/parser_test.go +++ b/pkg/rules/parser_test.go @@ -250,8 +250,6 @@ func TestRuleMatch(t *testing.T) { require.NoError(t, err) for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/safe/routine_test.go b/pkg/safe/routine_test.go index 1880b7c4e..12875972c 100644 --- a/pkg/safe/routine_test.go +++ b/pkg/safe/routine_test.go @@ -63,7 +63,6 @@ func TestPoolWithCtx(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { // These subtests cannot be run in parallel, since the testRoutine // is shared across the subtests. diff --git a/pkg/server/aggregator_test.go b/pkg/server/aggregator_test.go index cc4557896..48ba181a2 100644 --- a/pkg/server/aggregator_test.go +++ b/pkg/server/aggregator_test.go @@ -113,7 +113,6 @@ func Test_mergeConfiguration(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -162,7 +161,6 @@ func Test_mergeConfiguration_tlsCertificates(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -346,8 +344,6 @@ func Test_mergeConfiguration_tlsOptions(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -439,7 +435,6 @@ func Test_mergeConfiguration_tlsStore(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -662,7 +657,6 @@ func Test_applyModel(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/cookie/cookie_test.go b/pkg/server/cookie/cookie_test.go index 125e9dce1..452f0f221 100644 --- a/pkg/server/cookie/cookie_test.go +++ b/pkg/server/cookie/cookie_test.go @@ -34,7 +34,6 @@ func TestGetName(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -64,7 +63,6 @@ func Test_sanitizeName(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/middleware/middlewares_test.go b/pkg/server/middleware/middlewares_test.go index 5d8db1aa8..58b053053 100644 --- a/pkg/server/middleware/middlewares_test.go +++ b/pkg/server/middleware/middlewares_test.go @@ -256,7 +256,6 @@ func TestBuilder_BuildChainWithContext(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -364,7 +363,6 @@ func TestBuilder_buildConstructor(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/provider/provider_test.go b/pkg/server/provider/provider_test.go index dc229e532..7686d1bb5 100644 --- a/pkg/server/provider/provider_test.go +++ b/pkg/server/provider/provider_test.go @@ -47,7 +47,6 @@ func TestAddInContext(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -103,7 +102,6 @@ func TestGetQualifiedName(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/router/router.go b/pkg/server/router/router.go index e8f214ffb..547066ddc 100644 --- a/pkg/server/router/router.go +++ b/pkg/server/router/router.go @@ -67,7 +67,6 @@ func (m *Manager) BuildHandlers(rootCtx context.Context, entryPoints []string, t entryPointHandlers := make(map[string]http.Handler) for entryPointName, routers := range m.getHTTPRouters(rootCtx, entryPoints, tls) { - entryPointName := entryPointName ctx := log.With(rootCtx, log.Str(log.EntryPointName, entryPointName)) handler, err := m.buildEntryPointHandler(ctx, routers) diff --git a/pkg/server/router/router_test.go b/pkg/server/router/router_test.go index 65d074039..3cce3c4c4 100644 --- a/pkg/server/router/router_test.go +++ b/pkg/server/router/router_test.go @@ -301,7 +301,6 @@ func TestRouterManager_Get(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -765,7 +764,6 @@ func TestRuntimeConfiguration(t *testing.T) { }, } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/router/tcp/manager.go b/pkg/server/router/tcp/manager.go index b149f5f0b..5b6e18385 100644 --- a/pkg/server/router/tcp/manager.go +++ b/pkg/server/router/tcp/manager.go @@ -73,8 +73,6 @@ func (m *Manager) BuildHandlers(rootCtx context.Context, entryPoints []string) m entryPointHandlers := make(map[string]*Router) for _, entryPointName := range entryPoints { - entryPointName := entryPointName - routers := entryPointsRouters[entryPointName] ctx := log.With(rootCtx, log.Str(log.EntryPointName, entryPointName)) diff --git a/pkg/server/router/tcp/manager_test.go b/pkg/server/router/tcp/manager_test.go index dd31ce3b3..0b6066b8a 100644 --- a/pkg/server/router/tcp/manager_test.go +++ b/pkg/server/router/tcp/manager_test.go @@ -298,8 +298,6 @@ func TestRuntimeConfiguration(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/router/tcp/router_test.go b/pkg/server/router/tcp/router_test.go index 89d8c11ff..70f23b12c 100644 --- a/pkg/server/router/tcp/router_test.go +++ b/pkg/server/router/tcp/router_test.go @@ -539,8 +539,6 @@ func Test_Routing(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/router/udp/router.go b/pkg/server/router/udp/router.go index 75bea883a..dedea33e9 100644 --- a/pkg/server/router/udp/router.go +++ b/pkg/server/router/udp/router.go @@ -42,8 +42,6 @@ func (m *Manager) BuildHandlers(rootCtx context.Context, entryPoints []string) m entryPointHandlers := make(map[string]udp.Handler) for _, entryPointName := range entryPoints { - entryPointName := entryPointName - routers := entryPointsRouters[entryPointName] ctx := log.With(rootCtx, log.Str(log.EntryPointName, entryPointName)) diff --git a/pkg/server/router/udp/router_test.go b/pkg/server/router/udp/router_test.go index d29d7d2d6..a71662b48 100644 --- a/pkg/server/router/udp/router_test.go +++ b/pkg/server/router/udp/router_test.go @@ -106,8 +106,6 @@ func TestRuntimeConfiguration(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/routerfactory_test.go b/pkg/server/routerfactory_test.go index fdf4efc69..f70b62933 100644 --- a/pkg/server/routerfactory_test.go +++ b/pkg/server/routerfactory_test.go @@ -168,8 +168,6 @@ func TestServerResponseEmptyBackend(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/service/roundtripper_test.go b/pkg/server/service/roundtripper_test.go index d9212ab8d..8171de56b 100644 --- a/pkg/server/service/roundtripper_test.go +++ b/pkg/server/service/roundtripper_test.go @@ -259,7 +259,6 @@ func TestDisableHTTP2(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -333,7 +332,6 @@ func TestKerberosRoundTripper(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/service/service_test.go b/pkg/server/service/service_test.go index 0ff7e86ae..b761b410e 100644 --- a/pkg/server/service/service_test.go +++ b/pkg/server/service/service_test.go @@ -66,7 +66,6 @@ func TestGetLoadBalancer(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -336,7 +335,6 @@ func TestGetLoadBalancerServiceHandler(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { handler, err := sm.getLoadBalancerServiceHandler(context.Background(), test.serviceName, test.service) @@ -522,7 +520,6 @@ func TestManager_Build(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/service/tcp/service_test.go b/pkg/server/service/tcp/service_test.go index fc6c4702c..6587469cc 100644 --- a/pkg/server/service/tcp/service_test.go +++ b/pkg/server/service/tcp/service_test.go @@ -174,7 +174,6 @@ func TestManager_BuildTCP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/server/service/udp/service_test.go b/pkg/server/service/udp/service_test.go index 0f06c1feb..a3ee3b15e 100644 --- a/pkg/server/service/udp/service_test.go +++ b/pkg/server/service/udp/service_test.go @@ -174,7 +174,6 @@ func TestManager_BuildUDP(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/tcp/proxy_test.go b/pkg/tcp/proxy_test.go index fb8cad1dd..feddb80fa 100644 --- a/pkg/tcp/proxy_test.go +++ b/pkg/tcp/proxy_test.go @@ -104,8 +104,6 @@ func TestProxyProtocol(t *testing.T) { } for _, test := range testCases { - test := test - t.Run(test.desc, func(t *testing.T) { backendListener, err := net.Listen("tcp", ":0") require.NoError(t, err) @@ -194,7 +192,6 @@ func TestLookupAddress(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/tcp/wrr_load_balancer_test.go b/pkg/tcp/wrr_load_balancer_test.go index 5ee18caad..fc3438ac6 100644 --- a/pkg/tcp/wrr_load_balancer_test.go +++ b/pkg/tcp/wrr_load_balancer_test.go @@ -121,13 +121,11 @@ func TestLoadBalancing(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() balancer := NewWRRLoadBalancer() for server, weight := range test.serversWeight { - server := server balancer.AddWeightServer(HandlerFunc(func(conn WriteCloser) { _, err := conn.Write([]byte(server)) require.NoError(t, err) diff --git a/pkg/tls/certificate_store_test.go b/pkg/tls/certificate_store_test.go index a80516142..aaf55b647 100644 --- a/pkg/tls/certificate_store_test.go +++ b/pkg/tls/certificate_store_test.go @@ -56,7 +56,6 @@ func TestGetBestCertificate(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() dynamicMap := map[string]*tls.Certificate{} diff --git a/pkg/tls/tlsmanager_test.go b/pkg/tls/tlsmanager_test.go index 644a80d03..c91e244a1 100644 --- a/pkg/tls/tlsmanager_test.go +++ b/pkg/tls/tlsmanager_test.go @@ -161,7 +161,6 @@ func TestManager_Get(t *testing.T) { tlsManager.UpdateConfigs(context.Background(), nil, tlsConfigs, dynamicConfigs) for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -301,7 +300,6 @@ func TestClientAuth(t *testing.T) { tlsManager.UpdateConfigs(context.Background(), nil, tlsConfigs, nil) for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/tracing/operation_name_test.go b/pkg/tracing/operation_name_test.go index daae56806..df29f0dbf 100644 --- a/pkg/tracing/operation_name_test.go +++ b/pkg/tracing/operation_name_test.go @@ -54,7 +54,6 @@ func Test_generateOperationName(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -83,7 +82,6 @@ func TestComputeHash(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -122,7 +120,6 @@ func TestTruncateString(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/types/domain_test.go b/pkg/types/domain_test.go index dc97c7971..f9f7d42cb 100644 --- a/pkg/types/domain_test.go +++ b/pkg/types/domain_test.go @@ -37,7 +37,6 @@ func TestDomain_ToStrArray(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -77,7 +76,6 @@ func TestDomain_Set(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() @@ -171,7 +169,6 @@ func TestMatchDomain(t *testing.T) { } for _, test := range testCases { - test := test t.Run(test.desc, func(t *testing.T) { t.Parallel() diff --git a/pkg/types/tls_test.go b/pkg/types/tls_test.go index d3e93f408..b123aba53 100644 --- a/pkg/types/tls_test.go +++ b/pkg/types/tls_test.go @@ -104,8 +104,6 @@ func TestClientTLS_CreateTLSConfig(t *testing.T) { } for _, test := range tests { - test := test - t.Run(test.desc, func(t *testing.T) { tlsConfig, err := test.clientTLS.CreateTLSConfig(context.Background()) if test.wantErr {