1. docker
  2. Commit aea8a0724a563102ebb1f785de7dac869ebfc59b
Browse code

integration/network: remove redundant capturing of loop vars (copyloopvar)

integration/network/inspect_test.go:77:3: The copy of the 'for' variable "tc" can be deleted (Go 1.22+) (copyloopvar)
tc := tc
^
integration/network/network_test.go:81:3: The copy of the 'for' variable "ep" can be deleted (Go 1.22+) (copyloopvar)
ep := ep
^
integration/network/network_test.go:143:3: The copy of the 'for' variable "ep" can be deleted (Go 1.22+) (copyloopvar)
ep := ep
^
integration/network/network_test.go:221:3: The copy of the 'for' variable "tc" can be deleted (Go 1.22+) (copyloopvar)
tc := tc
^

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>

Sebastiaan van Stijn authored on 2024/11/12 21:03:05
Showing 2 changed files
integration/network/inspect_test.go
History View file @ aea8a07
... ... 
@@ -74,7 +74,6 @@ func TestInspectNetwork(t *testing.T) {
74 74 
 		},
75 75 
 	}
76 76 
 	for _, tc := range tests {
77 
-		tc := tc
78 77 
 		t.Run(tc.name, func(t *testing.T) {
79 78 
 			ctx := testutil.StartSpan(ctx, t)
80 79 
 			nw, err := c.NetworkInspect(ctx, tc.network, tc.opts)
integration/network/network_test.go
History View file @ aea8a07
... ... 
@@ -78,7 +78,6 @@ func TestNetworkInvalidJSON(t *testing.T) {
78 78 
 	}
79 79
80 80 
 	for _, ep := range endpoints {
81 
-		ep := ep
82 81 
 		t.Run(ep[1:], func(t *testing.T) {
83 82 
 			t.Parallel()
84 83 
 			ctx := testutil.StartSpan(ctx, t)
... ... 
@@ -140,7 +139,6 @@ func TestNetworkList(t *testing.T) {
140 140 
 	}
141 141
142 142 
 	for _, ep := range endpoints {
143 
-		ep := ep
144 143 
 		t.Run(ep, func(t *testing.T) {
145 144 
 			ctx := testutil.StartSpan(ctx, t)
146 145 
 			t.Parallel()
... ... 
@@ -218,7 +216,6 @@ func TestDefaultNetworkOpts(t *testing.T) {
218 218 
 	}
219 219
220 220 
 	for _, tc := range tests {
221 
-		tc := tc
222 221 
 		t.Run(tc.name, func(t *testing.T) {
223 222 
 			ctx := testutil.StartSpan(ctx, t)
224 223 
 			d := daemon.New(t)