Bladeren bron

Fix loop-closure bugs in tests

...which were flagged by golangci-lint v1.51.

Signed-off-by: Cory Snider <csnider@mirantis.com>
Cory Snider 2 jaren geleden
bovenliggende
commit
0c68b655f6
2 gewijzigde bestanden met toevoegingen van 2 en 0 verwijderingen
  1. 1 0
      distribution/xfer/download_test.go
  2. 1 0
      integration/build/build_test.go

+ 1 - 0
distribution/xfer/download_test.go

@@ -389,6 +389,7 @@ func TestMaxDownloadAttempts(t *testing.T) {
 		},
 		},
 	}
 	}
 	for _, tc := range tests {
 	for _, tc := range tests {
+		tc := tc
 		t.Run(tc.name, func(t *testing.T) {
 		t.Run(tc.name, func(t *testing.T) {
 			t.Parallel()
 			t.Parallel()
 			layerStore := &mockLayerStore{make(map[layer.ChainID]*mockLayer)}
 			layerStore := &mockLayerStore{make(map[layer.ChainID]*mockLayer)}

+ 1 - 0
integration/build/build_test.go

@@ -90,6 +90,7 @@ func TestBuildWithRemoveAndForceRemove(t *testing.T) {
 	client := testEnv.APIClient()
 	client := testEnv.APIClient()
 	ctx := context.Background()
 	ctx := context.Background()
 	for _, c := range cases {
 	for _, c := range cases {
+		c := c
 		t.Run(c.name, func(t *testing.T) {
 		t.Run(c.name, func(t *testing.T) {
 			t.Parallel()
 			t.Parallel()
 			dockerfile := []byte(c.dockerfile)
 			dockerfile := []byte(c.dockerfile)