Improve TestAttachClosedOnContainerStop
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com> Upstream-commit: 9d8f01b4f5ce3fe20c3116a486006a3e01f59fc5 Component: engine
This commit is contained in:
@ -4,6 +4,7 @@ package main
|
||||
|
||||
import (
|
||||
"bufio"
|
||||
"io/ioutil"
|
||||
"os/exec"
|
||||
"strings"
|
||||
"time"
|
||||
@ -23,7 +24,7 @@ func (s *DockerSuite) TestAttachClosedOnContainerStop(c *check.C) {
|
||||
id := strings.TrimSpace(out)
|
||||
c.Assert(waitRun(id), check.IsNil)
|
||||
|
||||
_, tty, err := pty.Open()
|
||||
pty, tty, err := pty.Open()
|
||||
c.Assert(err, check.IsNil)
|
||||
|
||||
attachCmd := exec.Command(dockerBinary, "attach", id)
|
||||
@ -35,6 +36,7 @@ func (s *DockerSuite) TestAttachClosedOnContainerStop(c *check.C) {
|
||||
|
||||
errChan := make(chan error)
|
||||
go func() {
|
||||
time.Sleep(300 * time.Millisecond)
|
||||
defer close(errChan)
|
||||
// Container is waiting for us to signal it to stop
|
||||
dockerCmd(c, "stop", id)
|
||||
@ -48,7 +50,9 @@ func (s *DockerSuite) TestAttachClosedOnContainerStop(c *check.C) {
|
||||
|
||||
select {
|
||||
case err := <-errChan:
|
||||
c.Assert(err, check.IsNil)
|
||||
tty.Close()
|
||||
out, _ := ioutil.ReadAll(pty)
|
||||
c.Assert(err, check.IsNil, check.Commentf("out: %v", string(out)))
|
||||
case <-time.After(attachWait):
|
||||
c.Fatal("timed out without attach returning")
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user