Merge pull request #6580 from thaJeztah/no_json

cli/command/image: remove uses of JSON field

Author: austinvazquez

cli/command/image/load.go

@@ -70,7 +70,7 @@ func runLoad(ctx context.Context, dockerCli command.Cli, opts loadOptions) error
 		if err != nil {
 			return err
 		}
-		defer file.Close()
+		defer func() { _ = file.Close() }()
 		input = file
 	}
 
@@ -95,12 +95,7 @@ func runLoad(ctx context.Context, dockerCli command.Cli, opts loadOptions) error
 	if err != nil {
 		return err
 	}
-	defer res.Close()
+	defer func() { _ = res.Close() }()
 
-	if res.JSON {
-		return jsonstream.Display(ctx, res, dockerCli.Out())
-	}
-
-	_, err = io.Copy(dockerCli.Out(), res)
-	return err
+	return jsonstream.Display(ctx, res, dockerCli.Out())
 }

cli/command/image/load_test.go

@@ -74,8 +74,8 @@ func TestNewLoadCommandInvalidInput(t *testing.T) {
 	assert.ErrorContains(t, err, expectedError)
 }
 
-func mockImageLoadResult(content string, json bool) client.ImageLoadResult {
-	out := client.ImageLoadResult{JSON: json}
+func mockImageLoadResult(content string) client.ImageLoadResult {
+	out := client.ImageLoadResult{}
 
 	// Set unexported field "body"
 	v := reflect.ValueOf(&out).Elem()
@@ -94,31 +94,21 @@ func TestNewLoadCommandSuccess(t *testing.T) {
 		{
 			name: "simple",
 			args: []string{},
-			imageLoadFunc: func(input io.Reader, options ...client.ImageLoadOption) (client.ImageLoadResult, error) {
-				// FIXME(thaJeztah): how to mock this?
-				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader("Success"))}, nil
-				return mockImageLoadResult(`Success`, false), nil
-			},
-		},
-		{
-			name: "json",
-			args: []string{},
 			imageLoadFunc: func(input io.Reader, options ...client.ImageLoadOption) (client.ImageLoadResult, error) {
 				// FIXME(thaJeztah): how to mock this?
 				// return client.ImageLoadResult{
-				// 	Body: io.NopCloser(strings.NewReader(`{"ID": "1"}`)),
-				// 	JSON: true,
+				// 	Body: io.NopCloser(strings.NewReader(`{"ID":"simple","Status":"success"}`)),
 				// }, nil
-				return mockImageLoadResult(`{"ID":"1"}`, true), nil
+				return mockImageLoadResult(`{"ID":"simple","Status":"success"}`), nil
 			},
 		},
 		{
 			name: "input-file",
 			args: []string{"--input", "testdata/load-command-success.input.txt"},
 			imageLoadFunc: func(input io.Reader, options ...client.ImageLoadOption) (client.ImageLoadResult, error) {
 				// FIXME(thaJeztah): how to mock this?
-				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader("Success"))}, nil
-				return mockImageLoadResult(`Success`, false), nil
+				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader(`{"ID":"input-file","Status":"success"}`))}, nil
+				return mockImageLoadResult(`{"ID":"input-file","Status":"success"}`), nil
 			},
 		},
 		{
@@ -129,8 +119,8 @@ func TestNewLoadCommandSuccess(t *testing.T) {
 				assert.Check(t, len(options) > 0) // can be 1 or two depending on whether a terminal is attached :/
 				// assert.Check(t, is.Contains(options, client.ImageHistoryWithPlatform(ocispec.Platform{OS: "linux", Architecture: "amd64"})))
 				// FIXME(thaJeztah): how to mock this?
-				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader("Success"))}, nil
-				return mockImageLoadResult(`Success`, false), nil
+				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader(`{"ID":"single-platform","Status":"success"}`))}, nil
+				return mockImageLoadResult(`{"ID":"single-platform","Status":"success"}`), nil
 			},
 		},
 		{
@@ -139,8 +129,8 @@ func TestNewLoadCommandSuccess(t *testing.T) {
 			imageLoadFunc: func(input io.Reader, options ...client.ImageLoadOption) (client.ImageLoadResult, error) {
 				assert.Check(t, len(options) > 0) // can be 1 or two depending on whether a terminal is attached :/
 				// FIXME(thaJeztah): how to mock this?
-				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader("Success"))}, nil
-				return mockImageLoadResult(`Success`, false), nil
+				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader(`{"ID":"with-comma-separated-platforms","Status":"success"}`))}, nil
+				return mockImageLoadResult(`{"ID":"with-comma-separated-platforms","Status":"success"}`), nil
 			},
 		},
 		{
@@ -149,8 +139,8 @@ func TestNewLoadCommandSuccess(t *testing.T) {
 			imageLoadFunc: func(input io.Reader, options ...client.ImageLoadOption) (client.ImageLoadResult, error) {
 				assert.Check(t, len(options) > 0) // can be 1 or two depending on whether a terminal is attached :/
 				// FIXME(thaJeztah): how to mock this?
-				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader("Success"))}, nil
-				return mockImageLoadResult(`Success`, false), nil
+				// return client.ImageLoadResult{Body: io.NopCloser(strings.NewReader(`{"ID":"with-multiple-platform-options","Status":"success"}`))}, nil
+				return mockImageLoadResult(`{"ID":"with-multiple-platform-options","Status":"success"}`), nil
 			},
 		},
 	}

cli/command/image/testdata/load-command-success.input-file.golden

@@ -1 +1 @@
-Success
+input-file: success

cli/command/image/testdata/load-command-success.json.golden

@@ -1 +1 @@
-Success
+simple: success

cli/command/image/testdata/load-command-success.simple.golden

@@ -1 +1 @@
-Success
+with-comma-separated-platforms: success

cli/command/image/testdata/load-command-success.with-comma-separated-platforms.golden

@@ -1 +1 @@
-Success
+with-multiple-platform-options: success

cli/command/image/testdata/load-command-success.with-multiple-platform-options.golden

@@ -1 +1 @@
-Success
+single-platform: success