소스 검색

Fix TestDiffEmptyArgClientError O:)

Signed-off-by: Vincent Demeester <vincent@sbr.pm>
Vincent Demeester 9 년 전
부모
커밋
7878705dcc
2개의 변경된 파일4개의 추가작업 그리고 1개의 파일을 삭제
  1. 3 0
      api/client/container/diff.go
  2. 1 1
      integration-cli/docker_cli_diff_test.go

+ 3 - 0
api/client/container/diff.go

@@ -34,6 +34,9 @@ func NewDiffCommand(dockerCli *client.DockerCli) *cobra.Command {
 }
 
 func runDiff(dockerCli *client.DockerCli, opts *diffOptions) error {
+	if opts.container == "" {
+		return fmt.Errorf("Container name cannot be empty")
+	}
 	ctx := context.Background()
 
 	changes, err := dockerCli.Client().ContainerDiff(ctx, opts.container)

+ 1 - 1
integration-cli/docker_cli_diff_test.go

@@ -83,5 +83,5 @@ func (s *DockerSuite) TestDiffEnsureDefaultDevs(c *check.C) {
 func (s *DockerSuite) TestDiffEmptyArgClientError(c *check.C) {
 	out, _, err := dockerCmdWithError("diff", "")
 	c.Assert(err, checker.NotNil)
-	c.Assert(strings.TrimSpace(out), checker.Contains, "\"docker diff\" requires exactly 1 argument(s).")
+	c.Assert(strings.TrimSpace(out), checker.Contains, "Container name cannot be empty")
 }