瀏覽代碼

Merge pull request #16641 from cpuguy83/16598_fix_TestExecStartFails

Make sure test container is running before exec
Doug Davis 9 年之前
父節點
當前提交
499b76d2ec
共有 1 個文件被更改,包括 5 次插入4 次删除
  1. 5 4
      integration-cli/docker_cli_exec_test.go

+ 5 - 4
integration-cli/docker_cli_exec_test.go

@@ -15,6 +15,7 @@ import (
 	"sync"
 	"sync"
 	"time"
 	"time"
 
 
+	"github.com/docker/docker/pkg/integration/checker"
 	"github.com/go-check/check"
 	"github.com/go-check/check"
 )
 )
 
 
@@ -616,9 +617,9 @@ func (s *DockerSuite) TestExecStartFails(c *check.C) {
 	testRequires(c, DaemonIsLinux)
 	testRequires(c, DaemonIsLinux)
 	name := "exec-15750"
 	name := "exec-15750"
 	dockerCmd(c, "run", "-d", "--name", name, "busybox", "top")
 	dockerCmd(c, "run", "-d", "--name", name, "busybox", "top")
+	c.Assert(waitRun(name), check.IsNil)
 
 
-	_, errmsg, status := dockerCmdWithStdoutStderr(nil, "exec", name, "no-such-cmd")
-	if status == 255 && !strings.Contains(errmsg, "executable file not found") {
-		c.Fatal("exec error message not received. The daemon might had crashed")
-	}
+	out, _, err := dockerCmdWithError("exec", name, "no-such-cmd")
+	c.Assert(err, check.NotNil, check.Commentf(out))
+	c.Assert(out, checker.Contains, "executable file not found")
 }
 }