diff --git a/cli/command/cli_test.go b/cli/command/cli_test.go index 2d12974d93..c8a724346f 100644 --- a/cli/command/cli_test.go +++ b/cli/command/cli_test.go @@ -187,19 +187,18 @@ func TestInitializeFromClient(t *testing.T) { }, } - for _, testcase := range testcases { - testcase := testcase - t.Run(testcase.doc, func(t *testing.T) { + for _, tc := range testcases { + t.Run(tc.doc, func(t *testing.T) { apiclient := &fakeClient{ - pingFunc: testcase.pingFunc, + pingFunc: tc.pingFunc, version: defaultVersion, } cli := &DockerCli{client: apiclient} err := cli.Initialize(flags.NewClientOptions()) assert.NilError(t, err) - assert.DeepEqual(t, cli.ServerInfo(), testcase.expectedServer) - assert.Equal(t, apiclient.negotiated, testcase.negotiated) + assert.DeepEqual(t, cli.ServerInfo(), tc.expectedServer) + assert.Equal(t, apiclient.negotiated, tc.negotiated) }) } } @@ -277,10 +276,9 @@ func TestExperimentalCLI(t *testing.T) { }, } - for _, testcase := range testcases { - testcase := testcase - t.Run(testcase.doc, func(t *testing.T) { - dir := fs.NewDir(t, testcase.doc, fs.WithFile("config.json", testcase.configfile)) + for _, tc := range testcases { + t.Run(tc.doc, func(t *testing.T) { + dir := fs.NewDir(t, tc.doc, fs.WithFile("config.json", tc.configfile)) defer dir.Remove() apiclient := &fakeClient{ version: defaultVersion, diff --git a/cli/command/completion/functions_test.go b/cli/command/completion/functions_test.go index aacd9769e1..9357a26acc 100644 --- a/cli/command/completion/functions_test.go +++ b/cli/command/completion/functions_test.go @@ -150,7 +150,6 @@ func TestCompleteContainerNames(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { if tc.showIDs { t.Setenv("DOCKER_COMPLETION_SHOW_CONTAINER_IDS", "yes") @@ -227,7 +226,6 @@ func TestCompleteImageNames(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { comp := ImageNames(fakeCLI{&fakeClient{ imageListFunc: func(options image.ListOptions) ([]image.Summary, error) { @@ -273,7 +271,6 @@ func TestCompleteNetworkNames(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { comp := NetworkNames(fakeCLI{&fakeClient{ networkListFunc: func(ctx context.Context, options network.ListOptions) ([]network.Summary, error) { @@ -331,7 +328,6 @@ func TestCompleteVolumeNames(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { comp := VolumeNames(fakeCLI{&fakeClient{ volumeListFunc: func(filter filters.Args) (volume.ListResponse, error) { diff --git a/cli/command/config/create_test.go b/cli/command/config/create_test.go index 848f9c6e07..855fab3fdf 100644 --- a/cli/command/config/create_test.go +++ b/cli/command/config/create_test.go @@ -43,7 +43,6 @@ func TestConfigCreateErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.expectedError, func(t *testing.T) { cmd := newConfigCreateCommand( test.NewFakeCli(&fakeClient{ diff --git a/cli/command/config/formatter_test.go b/cli/command/config/formatter_test.go index 73ab880577..8736a876a4 100644 --- a/cli/command/config/formatter_test.go +++ b/cli/command/config/formatter_test.go @@ -61,7 +61,6 @@ id_rsa }, } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/container/attach_test.go b/cli/command/container/attach_test.go index 2bbef24a8d..5b0276f38c 100644 --- a/cli/command/container/attach_test.go +++ b/cli/command/container/attach_test.go @@ -73,7 +73,6 @@ func TestNewAttachCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewAttachCommand(test.NewFakeCli(&fakeClient{inspectFunc: tc.containerInspectFunc})) cmd.SetOut(io.Discard) diff --git a/cli/command/container/cp_test.go b/cli/command/container/cp_test.go index d6a87bc366..113562e425 100644 --- a/cli/command/container/cp_test.go +++ b/cli/command/container/cp_test.go @@ -178,7 +178,6 @@ func TestSplitCpArg(t *testing.T) { }, } for _, tc := range testcases { - tc := tc t.Run(tc.doc, func(t *testing.T) { if tc.os == "windows" && runtime.GOOS != "windows" { t.Skip("skipping windows test on non-windows platform") diff --git a/cli/command/container/create_test.go b/cli/command/container/create_test.go index 0051a2bb78..2627d65f19 100644 --- a/cli/command/container/create_test.go +++ b/cli/command/container/create_test.go @@ -113,7 +113,6 @@ func TestCreateContainerImagePullPolicy(t *testing.T) { }, } for _, tc := range cases { - tc := tc t.Run(tc.PullPolicy, func(t *testing.T) { pullCounter := 0 @@ -176,7 +175,6 @@ func TestCreateContainerImagePullPolicyInvalid(t *testing.T) { }, } for _, tc := range cases { - tc := tc t.Run(tc.PullPolicy, func(t *testing.T) { dockerCli := test.NewFakeCli(&fakeClient{}) err := runCreate( @@ -207,7 +205,6 @@ func TestCreateContainerValidateFlags(t *testing.T) { expectedErr: `invalid argument "STDINFO" for "-a, --attach" flag: valid streams are STDIN, STDOUT and STDERR`, }, } { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewCreateCommand(test.NewFakeCli(&fakeClient{})) cmd.SetOut(io.Discard) @@ -251,7 +248,6 @@ func TestNewCreateCommandWithContentTrustErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc fakeCLI := test.NewFakeCli(&fakeClient{ createContainerFunc: func(config *container.Config, hostConfig *container.HostConfig, @@ -312,7 +308,6 @@ func TestNewCreateCommandWithWarnings(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { fakeCLI := test.NewFakeCli(&fakeClient{ createContainerFunc: func(config *container.Config, diff --git a/cli/command/container/formatter_diff_test.go b/cli/command/container/formatter_diff_test.go index 0b7c2f12f6..e06117e84a 100644 --- a/cli/command/container/formatter_diff_test.go +++ b/cli/command/container/formatter_diff_test.go @@ -47,7 +47,6 @@ D: /usr/app/old_app.js } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { out := bytes.NewBufferString("") tc.context.Output = out diff --git a/cli/command/container/formatter_stats_test.go b/cli/command/container/formatter_stats_test.go index ad395e6f74..b3011902ec 100644 --- a/cli/command/container/formatter_stats_test.go +++ b/cli/command/container/formatter_stats_test.go @@ -178,7 +178,6 @@ container2 -- -- } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out @@ -223,7 +222,6 @@ func TestContainerStatsContextWriteWithNoStats(t *testing.T) { } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { err := statsFormatWrite(tc.context, []StatsEntry{}, "linux", false) assert.NilError(t, err) @@ -265,7 +263,6 @@ func TestContainerStatsContextWriteWithNoStatsWindows(t *testing.T) { } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { err := statsFormatWrite(tc.context, []StatsEntry{}, "windows", false) assert.NilError(t, err) diff --git a/cli/command/container/list_test.go b/cli/command/container/list_test.go index d6f7d4f11f..fac33332ba 100644 --- a/cli/command/container/list_test.go +++ b/cli/command/container/list_test.go @@ -277,7 +277,6 @@ func TestContainerListFormatSizeSetsOption(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ containerListFunc: func(options container.ListOptions) ([]container.Summary, error) { diff --git a/cli/command/container/opts_test.go b/cli/command/container/opts_test.go index e0ff746bc5..46bc96aaea 100644 --- a/cli/command/container/opts_test.go +++ b/cli/command/container/opts_test.go @@ -126,7 +126,6 @@ func TestParseRunAttach(t *testing.T) { }, } for _, tc := range tests { - tc := tc t.Run(tc.input, func(t *testing.T) { config, _, _ := mustParse(t, tc.input) assert.Equal(t, config.AttachStdin, tc.expected.AttachStdin) @@ -802,7 +801,6 @@ func TestParseRestartPolicy(t *testing.T) { }, } for _, tc := range tests { - tc := tc t.Run(tc.input, func(t *testing.T) { _, hostConfig, _, err := parseRun([]string{"--restart=" + tc.input, "img", "cmd"}) if tc.expectedErr != "" { diff --git a/cli/command/container/port_test.go b/cli/command/container/port_test.go index 8502cf761d..152e47ab4b 100644 --- a/cli/command/container/port_test.go +++ b/cli/command/container/port_test.go @@ -43,7 +43,6 @@ func TestNewPortCommandOutput(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ inspectFunc: func(string) (container.InspectResponse, error) { diff --git a/cli/command/container/restart_test.go b/cli/command/container/restart_test.go index d1b6ed43ea..f09323ec2a 100644 --- a/cli/command/container/restart_test.go +++ b/cli/command/container/restart_test.go @@ -58,7 +58,6 @@ func TestRestart(t *testing.T) { expectedErr: "conflicting options: cannot specify both --timeout and --time", }, } { - tc := tc t.Run(tc.name, func(t *testing.T) { var restarted []string mutex := new(sync.Mutex) diff --git a/cli/command/container/rm_test.go b/cli/command/container/rm_test.go index 493a2a68ba..505fc7c48b 100644 --- a/cli/command/container/rm_test.go +++ b/cli/command/container/rm_test.go @@ -23,7 +23,6 @@ func TestRemoveForce(t *testing.T) { {name: "without force", args: []string{"nosuchcontainer", "mycontainer"}, expectedErr: "no such container"}, {name: "with force", args: []string{"--force", "nosuchcontainer", "mycontainer"}, expectedErr: ""}, } { - tc := tc t.Run(tc.name, func(t *testing.T) { var removed []string mutex := new(sync.Mutex) diff --git a/cli/command/container/run_test.go b/cli/command/container/run_test.go index fad200ebae..7c2e36db64 100644 --- a/cli/command/container/run_test.go +++ b/cli/command/container/run_test.go @@ -35,7 +35,6 @@ func TestRunValidateFlags(t *testing.T) { expectedErr: "conflicting options: cannot specify both --attach and --detach", }, } { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewRunCommand(test.NewFakeCli(&fakeClient{})) cmd.SetOut(io.Discard) @@ -245,7 +244,6 @@ func TestRunCommandWithContentTrustErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { fakeCLI := test.NewFakeCli(&fakeClient{ createContainerFunc: func(config *container.Config, @@ -286,7 +284,6 @@ func TestRunContainerImagePullPolicyInvalid(t *testing.T) { }, } for _, tc := range cases { - tc := tc t.Run(tc.PullPolicy, func(t *testing.T) { dockerCli := test.NewFakeCli(&fakeClient{}) err := runRun( diff --git a/cli/command/container/stop_test.go b/cli/command/container/stop_test.go index 1fad3cd05e..50013d4775 100644 --- a/cli/command/container/stop_test.go +++ b/cli/command/container/stop_test.go @@ -58,7 +58,6 @@ func TestStop(t *testing.T) { expectedErr: "conflicting options: cannot specify both --timeout and --time", }, } { - tc := tc t.Run(tc.name, func(t *testing.T) { var stopped []string mutex := new(sync.Mutex) diff --git a/cli/command/context/create_test.go b/cli/command/context/create_test.go index 5d3f23fc31..8f9a2bc8c2 100644 --- a/cli/command/context/create_test.go +++ b/cli/command/context/create_test.go @@ -94,7 +94,6 @@ func TestCreate(t *testing.T) { }, } for _, tc := range tests { - tc := tc t.Run(tc.options.Name, func(t *testing.T) { err := RunCreate(cli, &tc.options) if tc.expecterErr == "" { @@ -164,25 +163,24 @@ func TestCreateFromContext(t *testing.T) { cli.SetCurrentContext("dummy") - for _, c := range cases { - c := c - t.Run(c.name, func(t *testing.T) { + for _, tc := range cases { + t.Run(tc.name, func(t *testing.T) { cli.ResetOutputBuffers() err := RunCreate(cli, &CreateOptions{ From: "original", - Name: c.name, - Description: c.description, - Docker: c.docker, + Name: tc.name, + Description: tc.description, + Docker: tc.docker, }) assert.NilError(t, err) - assertContextCreateLogging(t, cli, c.name) - newContext, err := cli.ContextStore().GetMetadata(c.name) + assertContextCreateLogging(t, cli, tc.name) + newContext, err := cli.ContextStore().GetMetadata(tc.name) assert.NilError(t, err) newContextTyped, err := command.GetDockerContext(newContext) assert.NilError(t, err) dockerEndpoint, err := docker.EndpointFromContext(newContext) assert.NilError(t, err) - assert.Equal(t, newContextTyped.Description, c.expectedDescription) + assert.Equal(t, newContextTyped.Description, tc.expectedDescription) assert.Equal(t, dockerEndpoint.Host, "tcp://42.42.42.42:2375") }) } @@ -219,23 +217,22 @@ func TestCreateFromCurrent(t *testing.T) { cli.SetCurrentContext("original") - for _, c := range cases { - c := c - t.Run(c.name, func(t *testing.T) { + for _, tc := range cases { + t.Run(tc.name, func(t *testing.T) { cli.ResetOutputBuffers() err := RunCreate(cli, &CreateOptions{ - Name: c.name, - Description: c.description, + Name: tc.name, + Description: tc.description, }) assert.NilError(t, err) - assertContextCreateLogging(t, cli, c.name) - newContext, err := cli.ContextStore().GetMetadata(c.name) + assertContextCreateLogging(t, cli, tc.name) + newContext, err := cli.ContextStore().GetMetadata(tc.name) assert.NilError(t, err) newContextTyped, err := command.GetDockerContext(newContext) assert.NilError(t, err) dockerEndpoint, err := docker.EndpointFromContext(newContext) assert.NilError(t, err) - assert.Equal(t, newContextTyped.Description, c.expectedDescription) + assert.Equal(t, newContextTyped.Description, tc.expectedDescription) assert.Equal(t, dockerEndpoint.Host, "tcp://42.42.42.42:2375") }) } diff --git a/cli/command/formatter/container_test.go b/cli/command/formatter/container_test.go index 3c8552f666..65ff2eb11d 100644 --- a/cli/command/formatter/container_test.go +++ b/cli/command/formatter/container_test.go @@ -346,7 +346,6 @@ size: 0B } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out @@ -411,7 +410,6 @@ func TestContainerContextWriteWithNoContainers(t *testing.T) { } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { err := ContainerWrite(tc.context, containers) assert.NilError(t, err) diff --git a/cli/command/formatter/disk_usage_test.go b/cli/command/formatter/disk_usage_test.go index 6205873958..bb1e6dc677 100644 --- a/cli/command/formatter/disk_usage_test.go +++ b/cli/command/formatter/disk_usage_test.go @@ -106,7 +106,6 @@ Build Cache 0 0 0B 0B } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/formatter/image_test.go b/cli/command/formatter/image_test.go index fcdf727c52..e180f4dba0 100644 --- a/cli/command/formatter/image_test.go +++ b/cli/command/formatter/image_test.go @@ -304,7 +304,6 @@ image_id: imageID3 } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out @@ -365,7 +364,6 @@ func TestImageContextWriteWithNoImage(t *testing.T) { } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { err := ImageWrite(tc.context, images) assert.NilError(t, err) diff --git a/cli/command/formatter/volume_test.go b/cli/command/formatter/volume_test.go index 3a8869eadc..f27ad17437 100644 --- a/cli/command/formatter/volume_test.go +++ b/cli/command/formatter/volume_test.go @@ -131,7 +131,6 @@ foobar_bar } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/image/formatter_history_test.go b/cli/command/image/formatter_history_test.go index b85aac43aa..cff2f7f612 100644 --- a/cli/command/image/formatter_history_test.go +++ b/cli/command/image/formatter_history_test.go @@ -255,7 +255,6 @@ imageID6 17 years ago /bin/bash echo 183MB } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { err := HistoryWrite(tc.context, true, histories) assert.NilError(t, err) diff --git a/cli/command/image/history_test.go b/cli/command/image/history_test.go index 0a69df8d15..4a406864d5 100644 --- a/cli/command/image/history_test.go +++ b/cli/command/image/history_test.go @@ -42,7 +42,6 @@ func TestNewHistoryCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewHistoryCommand(test.NewFakeCli(&fakeClient{imageHistoryFunc: tc.imageHistoryFunc})) cmd.SetOut(io.Discard) @@ -109,7 +108,6 @@ func TestNewHistoryCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { // Set to UTC timezone as timestamps in output are // printed in the current timezone diff --git a/cli/command/image/import_test.go b/cli/command/image/import_test.go index 196171ed31..f61a80b9d3 100644 --- a/cli/command/image/import_test.go +++ b/cli/command/image/import_test.go @@ -98,7 +98,6 @@ func TestNewImportCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewImportCommand(test.NewFakeCli(&fakeClient{imageImportFunc: tc.imageImportFunc})) cmd.SetOut(io.Discard) diff --git a/cli/command/image/inspect_test.go b/cli/command/image/inspect_test.go index 9e2163db57..6958060ca5 100644 --- a/cli/command/image/inspect_test.go +++ b/cli/command/image/inspect_test.go @@ -25,7 +25,6 @@ func TestNewInspectCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newInspectCommand(test.NewFakeCli(&fakeClient{})) cmd.SetOut(io.Discard) @@ -79,7 +78,6 @@ func TestNewInspectCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { imageInspectInvocationCount = 0 cli := test.NewFakeCli(&fakeClient{imageInspectFunc: tc.imageInspectFunc}) diff --git a/cli/command/image/list_test.go b/cli/command/image/list_test.go index 8ccec6ab56..3129fe12ac 100644 --- a/cli/command/image/list_test.go +++ b/cli/command/image/list_test.go @@ -35,7 +35,6 @@ func TestNewImagesCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewImagesCommand(test.NewFakeCli(&fakeClient{imageListFunc: tc.imageListFunc})) cmd.SetOut(io.Discard) @@ -83,7 +82,6 @@ func TestNewImagesCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{imageListFunc: tc.imageListFunc}) cli.SetConfigFile(&configfile.ConfigFile{ImagesFormat: tc.imageFormat}) diff --git a/cli/command/image/load_test.go b/cli/command/image/load_test.go index 5b0e4532e8..8544e58745 100644 --- a/cli/command/image/load_test.go +++ b/cli/command/image/load_test.go @@ -52,7 +52,6 @@ func TestNewLoadCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{imageLoadFunc: tc.imageLoadFunc}) cli.In().SetIsTerminal(tc.isTerminalIn) @@ -116,7 +115,6 @@ func TestNewLoadCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{imageLoadFunc: tc.imageLoadFunc}) cmd := NewLoadCommand(cli) diff --git a/cli/command/image/prune_test.go b/cli/command/image/prune_test.go index 710c1cf748..8e8d3540a6 100644 --- a/cli/command/image/prune_test.go +++ b/cli/command/image/prune_test.go @@ -39,7 +39,6 @@ func TestNewPruneCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewPruneCommand(test.NewFakeCli(&fakeClient{ imagesPruneFunc: tc.imagesPruneFunc, @@ -98,7 +97,6 @@ func TestNewPruneCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{imagesPruneFunc: tc.imagesPruneFunc}) // when prompted, answer "Y" to confirm the prune. diff --git a/cli/command/image/pull_test.go b/cli/command/image/pull_test.go index a28aaf6166..a853949f32 100644 --- a/cli/command/image/pull_test.go +++ b/cli/command/image/pull_test.go @@ -38,7 +38,6 @@ func TestNewPullCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{}) cmd := NewPullCommand(cli) @@ -73,7 +72,6 @@ func TestNewPullCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ imagePullFunc: func(ref string, options image.PullOptions) (io.ReadCloser, error) { @@ -119,7 +117,6 @@ func TestNewPullCommandWithContentTrustErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ imagePullFunc: func(ref string, options image.PullOptions) (io.ReadCloser, error) { diff --git a/cli/command/image/push_test.go b/cli/command/image/push_test.go index 3f9d2fe59f..132b0e9a5d 100644 --- a/cli/command/image/push_test.go +++ b/cli/command/image/push_test.go @@ -38,7 +38,6 @@ func TestNewPushCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{imagePushFunc: tc.imagePushFunc}) cmd := NewPushCommand(cli) @@ -68,7 +67,6 @@ func TestNewPushCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ imagePushFunc: func(ref string, options image.PushOptions) (io.ReadCloser, error) { diff --git a/cli/command/image/remove_test.go b/cli/command/image/remove_test.go index 0340a65b94..61a7a346f5 100644 --- a/cli/command/image/remove_test.go +++ b/cli/command/image/remove_test.go @@ -62,7 +62,6 @@ func TestNewRemoveCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewRemoveCommand(test.NewFakeCli(&fakeClient{ imageRemoveFunc: tc.imageRemoveFunc, @@ -121,7 +120,6 @@ func TestNewRemoveCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{imageRemoveFunc: tc.imageRemoveFunc}) cmd := NewRemoveCommand(cli) diff --git a/cli/command/image/save_test.go b/cli/command/image/save_test.go index 65d931566a..e9b1396600 100644 --- a/cli/command/image/save_test.go +++ b/cli/command/image/save_test.go @@ -59,7 +59,6 @@ func TestNewSaveCommandErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{imageSaveFunc: tc.imageSaveFunc}) cli.Out().SetIsTerminal(tc.isTerminal) @@ -113,7 +112,6 @@ func TestNewSaveCommandSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(strings.Join(tc.args, " "), func(t *testing.T) { cmd := NewSaveCommand(test.NewFakeCli(&fakeClient{ imageSaveFunc: tc.imageSaveFunc, diff --git a/cli/command/manifest/create_test.go b/cli/command/manifest/create_test.go index 0b1296e805..c87d5d09ab 100644 --- a/cli/command/manifest/create_test.go +++ b/cli/command/manifest/create_test.go @@ -31,7 +31,6 @@ func TestManifestCreateErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.expectedError, func(t *testing.T) { cli := test.NewFakeCli(nil) cmd := newCreateListCommand(cli) diff --git a/cli/command/network/create_test.go b/cli/command/network/create_test.go index 94ff0ffc97..651767d65e 100644 --- a/cli/command/network/create_test.go +++ b/cli/command/network/create_test.go @@ -218,7 +218,6 @@ func TestNetworkCreateIPv6(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ networkCreateFunc: func(ctx context.Context, name string, createBody network.CreateOptions) (network.CreateResponse, error) { diff --git a/cli/command/network/formatter_test.go b/cli/command/network/formatter_test.go index 8a87a6760b..018f789b17 100644 --- a/cli/command/network/formatter_test.go +++ b/cli/command/network/formatter_test.go @@ -161,7 +161,6 @@ foobar_bar 2017-01-01 00:00:00 +0000 UTC } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/network/list_test.go b/cli/command/network/list_test.go index e92c0d09fa..cc08a00096 100644 --- a/cli/command/network/list_test.go +++ b/cli/command/network/list_test.go @@ -83,7 +83,6 @@ func TestNetworkList(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.doc, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{networkListFunc: tc.networkListFunc}) cmd := newListCommand(cli) diff --git a/cli/command/network/remove_test.go b/cli/command/network/remove_test.go index e3217a1874..01052b0bf5 100644 --- a/cli/command/network/remove_test.go +++ b/cli/command/network/remove_test.go @@ -63,7 +63,6 @@ func TestNetworkRemoveForce(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { fakeCli := test.NewFakeCli(&fakeClient{ networkRemoveFunc: func(ctx context.Context, networkID string) error { diff --git a/cli/command/node/formatter_test.go b/cli/command/node/formatter_test.go index e86d15a760..4eb172a934 100644 --- a/cli/command/node/formatter_test.go +++ b/cli/command/node/formatter_test.go @@ -202,7 +202,6 @@ foobar_boo Unknown } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/node/inspect_test.go b/cli/command/node/inspect_test.go index 9cb742fbb9..84a9f6bc76 100644 --- a/cli/command/node/inspect_test.go +++ b/cli/command/node/inspect_test.go @@ -106,7 +106,6 @@ func TestNodeInspectPretty(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ nodeInspectFunc: tc.nodeInspectFunc, diff --git a/cli/command/node/ps_test.go b/cli/command/node/ps_test.go index 84b1612170..60daac3b8c 100644 --- a/cli/command/node/ps_test.go +++ b/cli/command/node/ps_test.go @@ -134,7 +134,6 @@ func TestNodePs(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ infoFunc: tc.infoFunc, diff --git a/cli/command/plugin/formatter_test.go b/cli/command/plugin/formatter_test.go index 9d8b5c362c..642ee2afc4 100644 --- a/cli/command/plugin/formatter_test.go +++ b/cli/command/plugin/formatter_test.go @@ -131,7 +131,6 @@ foobar_bar } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/plugin/inspect_test.go b/cli/command/plugin/inspect_test.go index 2dcb895889..414ff479a0 100644 --- a/cli/command/plugin/inspect_test.go +++ b/cli/command/plugin/inspect_test.go @@ -66,7 +66,6 @@ func TestInspectErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.description, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{pluginInspectFunc: tc.inspectFunc}) cmd := newInspectCommand(cli) @@ -138,7 +137,6 @@ func TestInspect(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.description, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{pluginInspectFunc: tc.inspectFunc}) cmd := newInspectCommand(cli) diff --git a/cli/command/plugin/install_test.go b/cli/command/plugin/install_test.go index 4eb55c1a91..b59ed98fcb 100644 --- a/cli/command/plugin/install_test.go +++ b/cli/command/plugin/install_test.go @@ -54,7 +54,6 @@ func TestInstallErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.description, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{pluginInstallFunc: tc.installFunc}) cmd := newInstallCommand(cli) @@ -94,7 +93,6 @@ func TestInstallContentTrustErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.description, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ pluginInstallFunc: func(name string, options types.PluginInstallOptions) (io.ReadCloser, error) { @@ -138,7 +136,6 @@ func TestInstall(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.description, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{pluginInstallFunc: tc.installFunc}) cmd := newInstallCommand(cli) diff --git a/cli/command/plugin/list_test.go b/cli/command/plugin/list_test.go index 07138856fa..91665da654 100644 --- a/cli/command/plugin/list_test.go +++ b/cli/command/plugin/list_test.go @@ -46,7 +46,6 @@ func TestListErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.description, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{pluginListFunc: tc.listFunc}) cmd := newListCommand(cli) @@ -166,7 +165,6 @@ func TestList(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.description, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{pluginListFunc: tc.listFunc}) cmd := newListCommand(cli) diff --git a/cli/command/registry/formatter_search_test.go b/cli/command/registry/formatter_search_test.go index fd50a87f23..071bc70fed 100644 --- a/cli/command/registry/formatter_search_test.go +++ b/cli/command/registry/formatter_search_test.go @@ -203,7 +203,6 @@ result2 5 } for _, tc := range cases { - tc := tc t.Run(tc.doc, func(t *testing.T) { var out bytes.Buffer err := SearchWrite(formatter.Context{Format: tc.format, Output: &out}, results) diff --git a/cli/command/secret/formatter_test.go b/cli/command/secret/formatter_test.go index 3aec4c801c..8ede765a57 100644 --- a/cli/command/secret/formatter_test.go +++ b/cli/command/secret/formatter_test.go @@ -61,7 +61,6 @@ id_rsa }, } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/secret/inspect_test.go b/cli/command/secret/inspect_test.go index ca509c1249..fffe8a237f 100644 --- a/cli/command/secret/inspect_test.go +++ b/cli/command/secret/inspect_test.go @@ -93,7 +93,6 @@ func TestSecretInspectWithoutFormat(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ secretInspectFunc: tc.secretInspectFunc, @@ -132,7 +131,6 @@ func TestSecretInspectWithFormat(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ secretInspectFunc: tc.secretInspectFunc, diff --git a/cli/command/service/formatter_test.go b/cli/command/service/formatter_test.go index adb09476ad..cc74bfe6fa 100644 --- a/cli/command/service/formatter_test.go +++ b/cli/command/service/formatter_test.go @@ -223,7 +223,6 @@ zarp2 } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/service/list_test.go b/cli/command/service/list_test.go index 28952e5c0a..65da3c5589 100644 --- a/cli/command/service/list_test.go +++ b/cli/command/service/list_test.go @@ -168,7 +168,6 @@ func TestServiceListServiceStatus(t *testing.T) { } for _, tc := range matrix { - tc := tc t.Run(tc.doc, func(t *testing.T) { if tc.cluster == nil { tc.cluster = generateCluster(t, tc.opts) diff --git a/cli/command/service/opts_test.go b/cli/command/service/opts_test.go index 0d387b3676..799d32cebd 100644 --- a/cli/command/service/opts_test.go +++ b/cli/command/service/opts_test.go @@ -50,7 +50,6 @@ func TestCredentialSpecOpt(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.name, func(t *testing.T) { var cs credentialSpecOpt diff --git a/cli/command/service/rollback_test.go b/cli/command/service/rollback_test.go index 28e7f27c68..968cbdfa36 100644 --- a/cli/command/service/rollback_test.go +++ b/cli/command/service/rollback_test.go @@ -91,7 +91,6 @@ func TestRollbackWithErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newRollbackCommand( test.NewFakeCli(&fakeClient{ diff --git a/cli/command/service/update_test.go b/cli/command/service/update_test.go index 847b6efcb8..136f655122 100644 --- a/cli/command/service/update_test.go +++ b/cli/command/service/update_test.go @@ -1690,7 +1690,6 @@ func TestUpdateUlimits(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.name, func(t *testing.T) { svc := swarm.ServiceSpec{ TaskTemplate: swarm.TaskSpec{ diff --git a/cli/command/stack/formatter/formatter_test.go b/cli/command/stack/formatter/formatter_test.go index ec9ef83ada..fdb70f9d42 100644 --- a/cli/command/stack/formatter/formatter_test.go +++ b/cli/command/stack/formatter/formatter_test.go @@ -51,7 +51,6 @@ bar {Name: "bar", Services: 1}, } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/stack/list_test.go b/cli/command/stack/list_test.go index be76aa7e3d..dd4bd4940a 100644 --- a/cli/command/stack/list_test.go +++ b/cli/command/stack/list_test.go @@ -48,7 +48,6 @@ func TestListErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.expectedError, func(t *testing.T) { cmd := newListCommand(test.NewFakeCli(&fakeClient{ serviceListFunc: tc.serviceListFunc, @@ -104,7 +103,6 @@ func TestStackList(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.doc, func(t *testing.T) { var services []swarm.Service for _, name := range tc.serviceNames { diff --git a/cli/command/stack/ps_test.go b/cli/command/stack/ps_test.go index 009d51ed83..321425139c 100644 --- a/cli/command/stack/ps_test.go +++ b/cli/command/stack/ps_test.go @@ -40,7 +40,6 @@ func TestStackPsErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.expectedError, func(t *testing.T) { cmd := newPsCommand(test.NewFakeCli(&fakeClient{ taskListFunc: tc.taskListFunc, @@ -160,7 +159,6 @@ func TestStackPs(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.doc, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ taskListFunc: tc.taskListFunc, diff --git a/cli/command/stack/services_test.go b/cli/command/stack/services_test.go index 8d24a4e49e..29f9558654 100644 --- a/cli/command/stack/services_test.go +++ b/cli/command/stack/services_test.go @@ -67,7 +67,6 @@ func TestStackServicesErrors(t *testing.T) { } for _, tc := range testCases { - tc := tc t.Run(tc.expectedError, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ serviceListFunc: tc.serviceListFunc, diff --git a/cli/command/stack/swarm/deploy_test.go b/cli/command/stack/swarm/deploy_test.go index 112c48fa77..2452ad6f64 100644 --- a/cli/command/stack/swarm/deploy_test.go +++ b/cli/command/stack/swarm/deploy_test.go @@ -88,7 +88,6 @@ func TestServiceUpdateResolveImageChanged(t *testing.T) { ctx := context.Background() for _, tc := range testcases { - tc := tc t.Run(tc.image, func(t *testing.T) { spec := map[string]swarm.ServiceSpec{ "myservice": { diff --git a/cli/command/swarm/init_test.go b/cli/command/swarm/init_test.go index fbed83bde8..8a3d981b66 100644 --- a/cli/command/swarm/init_test.go +++ b/cli/command/swarm/init_test.go @@ -63,7 +63,6 @@ func TestSwarmInitErrorOnAPIFailure(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newInitCommand( test.NewFakeCli(&fakeClient{ diff --git a/cli/command/swarm/join_test.go b/cli/command/swarm/join_test.go index 6d60d693d2..53c0c08627 100644 --- a/cli/command/swarm/join_test.go +++ b/cli/command/swarm/join_test.go @@ -48,7 +48,6 @@ func TestSwarmJoinErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newJoinCommand( test.NewFakeCli(&fakeClient{ @@ -93,7 +92,6 @@ func TestSwarmJoin(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ infoFunc: tc.infoFunc, diff --git a/cli/command/swarm/join_token_test.go b/cli/command/swarm/join_token_test.go index c97cc0012d..0942b2b7c7 100644 --- a/cli/command/swarm/join_token_test.go +++ b/cli/command/swarm/join_token_test.go @@ -87,7 +87,6 @@ func TestSwarmJoinTokenErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ swarmInspectFunc: tc.swarmInspectFunc, @@ -198,7 +197,6 @@ func TestSwarmJoinToken(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ swarmInspectFunc: tc.swarmInspectFunc, diff --git a/cli/command/swarm/leave_test.go b/cli/command/swarm/leave_test.go index 58192e5ab4..b3a868b25c 100644 --- a/cli/command/swarm/leave_test.go +++ b/cli/command/swarm/leave_test.go @@ -32,7 +32,6 @@ func TestSwarmLeaveErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newLeaveCommand( test.NewFakeCli(&fakeClient{ diff --git a/cli/command/swarm/unlock_key_test.go b/cli/command/swarm/unlock_key_test.go index 15d7f2e840..8cb403dc7f 100644 --- a/cli/command/swarm/unlock_key_test.go +++ b/cli/command/swarm/unlock_key_test.go @@ -80,7 +80,6 @@ func TestSwarmUnlockKeyErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newUnlockKeyCommand( test.NewFakeCli(&fakeClient{ @@ -158,7 +157,6 @@ func TestSwarmUnlockKey(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ swarmInspectFunc: tc.swarmInspectFunc, diff --git a/cli/command/swarm/unlock_test.go b/cli/command/swarm/unlock_test.go index 04a996acb5..07c3cf1d6e 100644 --- a/cli/command/swarm/unlock_test.go +++ b/cli/command/swarm/unlock_test.go @@ -64,7 +64,6 @@ func TestSwarmUnlockErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newUnlockCommand( test.NewFakeCli(&fakeClient{ diff --git a/cli/command/swarm/update_test.go b/cli/command/swarm/update_test.go index debab12d21..3f147f3f90 100644 --- a/cli/command/swarm/update_test.go +++ b/cli/command/swarm/update_test.go @@ -65,7 +65,6 @@ func TestSwarmUpdateErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newUpdateCommand( test.NewFakeCli(&fakeClient{ @@ -169,7 +168,6 @@ func TestSwarmUpdate(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ swarmInspectFunc: tc.swarmInspectFunc, diff --git a/cli/command/system/events_test.go b/cli/command/system/events_test.go index 79769c26fa..a08423eed7 100644 --- a/cli/command/system/events_test.go +++ b/cli/command/system/events_test.go @@ -53,7 +53,6 @@ func TestEventsFormat(t *testing.T) { } for _, tc := range tests { - tc := tc t.Run(tc.name, func(t *testing.T) { // Set to UTC timezone as timestamps in output are // printed in the current timezone diff --git a/cli/command/system/info_test.go b/cli/command/system/info_test.go index 038bd493f0..e89c385d53 100644 --- a/cli/command/system/info_test.go +++ b/cli/command/system/info_test.go @@ -374,7 +374,6 @@ func TestPrettyPrintInfo(t *testing.T) { expectedError: "errors pretty printing info", }, } { - tc := tc t.Run(tc.doc, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{}) err := prettyPrintInfo(cli, tc.dockerInfo) @@ -452,7 +451,6 @@ func TestFormatInfo(t *testing.T) { expectedError: `template: :1:2: executing "" at <.badString>: can't evaluate field badString in type system.dockerInfo`, }, } { - tc := tc t.Run(tc.doc, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{}) info := dockerInfo{ @@ -518,7 +516,6 @@ func TestNeedsServerInfo(t *testing.T) { inf := dockerInfo{ClientInfo: &clientInfo{}} for _, tc := range tests { - tc := tc t.Run(tc.doc, func(t *testing.T) { assert.Equal(t, needsServerInfo(tc.template, inf), tc.expected) }) diff --git a/cli/command/task/formatter_test.go b/cli/command/task/formatter_test.go index 4a7ed7fb2f..c857c64bb4 100644 --- a/cli/command/task/formatter_test.go +++ b/cli/command/task/formatter_test.go @@ -71,7 +71,6 @@ foobar_bar foo2 } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/telemetry_utils_test.go b/cli/command/telemetry_utils_test.go index e30840a513..b5e3671f45 100644 --- a/cli/command/telemetry_utils_test.go +++ b/cli/command/telemetry_utils_test.go @@ -56,7 +56,6 @@ func TestGetFullCommandName(t *testing.T) { expected: "root child grandchild", }, } { - tc := tc t.Run(tc.testName, func(t *testing.T) { t.Parallel() actual := getFullCommandName(tc.cmd) @@ -91,7 +90,6 @@ func TestGetCommandName(t *testing.T) { expected: "child grandchild", }, } { - tc := tc t.Run(tc.testName, func(t *testing.T) { t.Parallel() actual := getCommandName(tc.cmd) @@ -130,7 +128,6 @@ func TestStdioAttributes(t *testing.T) { }, }, } { - tc := tc t.Run(tc.test, func(t *testing.T) { t.Parallel() cli := &DockerCli{ @@ -179,7 +176,6 @@ func TestAttributesFromError(t *testing.T) { }, }, } { - tc := tc t.Run(tc.testName, func(t *testing.T) { t.Parallel() actual := attributesFromError(tc.err) diff --git a/cli/command/trust/formatter_test.go b/cli/command/trust/formatter_test.go index 2b3af092ca..db1534526f 100644 --- a/cli/command/trust/formatter_test.go +++ b/cli/command/trust/formatter_test.go @@ -127,7 +127,6 @@ tag3 bbbbbbbb } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out @@ -231,7 +230,6 @@ eve foobarbazquxquux, key31, key32 {Name: "eve", Keys: []string{"key31", "key32", "foobarbazquxquux"}}, } for _, tc := range cases { - tc := tc t.Run(string(tc.context.Format), func(t *testing.T) { var out bytes.Buffer tc.context.Output = &out diff --git a/cli/command/trust/key_load_test.go b/cli/command/trust/key_load_test.go index 4d9e737e08..b2233f282a 100644 --- a/cli/command/trust/key_load_test.go +++ b/cli/command/trust/key_load_test.go @@ -116,7 +116,6 @@ var testKeys = map[string][]byte{ func TestLoadKeyFromPath(t *testing.T) { skip.If(t, runtime.GOOS == "windows") for keyID, keyBytes := range testKeys { - keyID, keyBytes := keyID, keyBytes t.Run(fmt.Sprintf("load-key-id-%s-from-path", keyID), func(t *testing.T) { privKeyFilepath := filepath.Join(t.TempDir(), "privkey.pem") assert.NilError(t, os.WriteFile(privKeyFilepath, keyBytes, notary.PrivNoExecPerms)) @@ -163,7 +162,6 @@ func TestLoadKeyFromPath(t *testing.T) { func TestLoadKeyTooPermissive(t *testing.T) { skip.If(t, runtime.GOOS == "windows") for keyID, keyBytes := range testKeys { - keyID, keyBytes := keyID, keyBytes t.Run(fmt.Sprintf("load-key-id-%s-too-permissive", keyID), func(t *testing.T) { privKeyDir := t.TempDir() privKeyFilepath := filepath.Join(privKeyDir, "privkey477.pem") diff --git a/cli/command/trust/signer_remove_test.go b/cli/command/trust/signer_remove_test.go index 8e2cc146d9..402ddda991 100644 --- a/cli/command/trust/signer_remove_test.go +++ b/cli/command/trust/signer_remove_test.go @@ -30,7 +30,6 @@ func TestTrustSignerRemoveErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := newSignerRemoveCommand( test.NewFakeCli(&fakeClient{})) @@ -66,7 +65,6 @@ func TestTrustSignerRemoveErrors(t *testing.T) { }, } for _, tc := range testCasesWithOutput { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{}) cli.SetNotaryClient(notaryfake.GetOfflineNotaryRepository) diff --git a/cli/command/volume/prune_test.go b/cli/command/volume/prune_test.go index 5cb1aca34a..8c49e22f51 100644 --- a/cli/command/volume/prune_test.go +++ b/cli/command/volume/prune_test.go @@ -52,7 +52,6 @@ func TestVolumePruneErrors(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cmd := NewPruneCommand( test.NewFakeCli(&fakeClient{ @@ -104,7 +103,6 @@ func TestVolumePruneSuccess(t *testing.T) { }, } for _, tc := range testCases { - tc := tc t.Run(tc.name, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{volumePruneFunc: tc.volumePruneFunc}) cmd := NewPruneCommand(cli) @@ -166,7 +164,6 @@ func TestVolumePrunePromptNo(t *testing.T) { skip.If(t, runtime.GOOS == "windows", "TODO: fix test on windows") for _, input := range []string{"n", "N", "no", "anything", "really"} { - input := input t.Run(input, func(t *testing.T) { cli := test.NewFakeCli(&fakeClient{ volumePruneFunc: simplePruneFunc,