Parcourir la source

Merge pull request #43132 from lzhfromustc/patch_1_7

test: fix a simple goroutine leak in distribution/xfer
Sebastiaan van Stijn il y a 3 ans
Parent
commit
eddbd6ff1e
1 fichiers modifiés avec 1 ajouts et 0 suppressions
  1. 1 0
      distribution/xfer/download_test.go

+ 1 - 0
distribution/xfer/download_test.go

@@ -350,6 +350,7 @@ func TestCancelledDownload(t *testing.T) {
 	descriptors := downloadDescriptors(nil)
 	descriptors := downloadDescriptors(nil)
 	_, _, err := ldm.Download(ctx, *image.NewRootFS(), runtime.GOOS, descriptors, progress.ChanOutput(progressChan))
 	_, _, err := ldm.Download(ctx, *image.NewRootFS(), runtime.GOOS, descriptors, progress.ChanOutput(progressChan))
 	if err != context.Canceled {
 	if err != context.Canceled {
+		close(progressChan)
 		t.Fatal("expected download to be cancelled")
 		t.Fatal("expected download to be cancelled")
 	}
 	}