integration/commands_test.go
8cceafc8
 package docker
 
 import (
 	"bufio"
 	"fmt"
 	"io"
5190f7f3
 	"io/ioutil"
b3974abe
 	"os"
 	"path"
d9fe0647
 	"regexp"
8cceafc8
 	"strings"
 	"testing"
 	"time"
2c8b63cb
 
 	"github.com/dotcloud/docker/api/client"
 	"github.com/dotcloud/docker/daemon"
 	"github.com/dotcloud/docker/engine"
 	"github.com/dotcloud/docker/image"
 	"github.com/dotcloud/docker/pkg/term"
 	"github.com/dotcloud/docker/utils"
8cceafc8
 )
 
 func closeWrap(args ...io.Closer) error {
 	e := false
 	ret := fmt.Errorf("Error closing elements")
 	for _, c := range args {
 		if err := c.Close(); err != nil {
 			e = true
 			ret = fmt.Errorf("%s\n%s", ret, err)
 		}
 	}
 	if e {
 		return ret
 	}
 	return nil
 }
 
359b7df5
 func setRaw(t *testing.T, c *daemon.Container) *term.State {
697be6aa
 	pty, err := c.GetPtyMaster()
 	if err != nil {
 		t.Fatal(err)
 	}
 	state, err := term.MakeRaw(pty.Fd())
 	if err != nil {
 		t.Fatal(err)
 	}
 	return state
 }
 
359b7df5
 func unsetRaw(t *testing.T, c *daemon.Container, state *term.State) {
697be6aa
 	pty, err := c.GetPtyMaster()
 	if err != nil {
 		t.Fatal(err)
 	}
 	term.RestoreTerminal(pty.Fd(), state)
 }
 
359b7df5
 func waitContainerStart(t *testing.T, timeout time.Duration) *daemon.Container {
 	var container *daemon.Container
697be6aa
 
 	setTimeout(t, "Waiting for the container to be started timed out", timeout, func() {
 		for {
359b7df5
 			l := globalDaemon.List()
697be6aa
 			if len(l) == 1 && l[0].State.IsRunning() {
 				container = l[0]
 				break
 			}
 			time.Sleep(10 * time.Millisecond)
 		}
 	})
 
 	if container == nil {
 		t.Fatal("An error occured while waiting for the container to start")
 	}
 
 	return container
 }
 
cfeed391
 func setTimeout(t *testing.T, msg string, d time.Duration, f func()) {
8cceafc8
 	c := make(chan bool)
 
 	// Make sure we are not too long
 	go func() {
 		time.Sleep(d)
 		c <- true
 	}()
cfeed391
 	go func() {
 		f()
 		c <- false
 	}()
ed7a4236
 	if <-c && msg != "" {
cfeed391
 		t.Fatal(msg)
8cceafc8
 	}
 }
 
1f75a0bf
 func expectPipe(expected string, r io.Reader) error {
 	o, err := bufio.NewReader(r).ReadString('\n')
 	if err != nil {
 		return err
 	}
 	if strings.Trim(o, " \r\n") != expected {
 		return fmt.Errorf("Unexpected output. Expected [%s], received [%s]", expected, o)
 	}
 	return nil
 }
 
8cceafc8
 func assertPipe(input, output string, r io.Reader, w io.Writer, count int) error {
 	for i := 0; i < count; i++ {
 		if _, err := w.Write([]byte(input)); err != nil {
 			return err
 		}
1f75a0bf
 		if err := expectPipe(output, r); err != nil {
8cceafc8
 			return err
 		}
 	}
 	return nil
 }
 
293157b8
 // TestRunWorkdirExistsAndIsFile checks that if 'docker run -w' with existing file can be detected
 func TestRunWorkdirExistsAndIsFile(t *testing.T) {
 
185b040e
 	cli := client.NewDockerCli(nil, nil, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
293157b8
 	defer cleanup(globalEngine, t)
 
 	c := make(chan struct{})
 	go func() {
 		defer close(c)
 		if err := cli.CmdRun("-w", "/bin/cat", unitTestImageID, "pwd"); err == nil {
 			t.Fatal("should have failed to run when using /bin/cat as working dir.")
 		}
 	}()
 
 	setTimeout(t, "CmdRun timed out", 5*time.Second, func() {
 		<-c
 	})
 }
 
bc823acc
 func TestRunExit(t *testing.T) {
 	stdin, stdinPipe := io.Pipe()
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
bc823acc
 
 	c1 := make(chan struct{})
 	go func() {
 		cli.CmdRun("-i", unitTestImageID, "/bin/cat")
 		close(c1)
 	}()
 
 	setTimeout(t, "Read/Write assertion timed out", 2*time.Second, func() {
ad43d88a
 		if err := assertPipe("hello\n", "hello", stdout, stdinPipe, 150); err != nil {
bc823acc
 			t.Fatal(err)
 		}
 	})
 
359b7df5
 	container := globalDaemon.List()[0]
bc823acc
 
 	// Closing /bin/cat stdin, expect it to exit
 	if err := stdin.Close(); err != nil {
 		t.Fatal(err)
 	}
 
 	// as the process exited, CmdRun must finish and unblock. Wait for it
 	setTimeout(t, "Waiting for CmdRun timed out", 10*time.Second, func() {
 		<-c1
 
 		go func() {
 			cli.CmdWait(container.ID)
 		}()
 
 		if _, err := bufio.NewReader(stdout).ReadString('\n'); err != nil {
 			t.Fatal(err)
 		}
 	})
 
 	// Make sure that the client has been disconnected
 	setTimeout(t, "The client should have been disconnected once the remote process exited.", 2*time.Second, func() {
 		// Expecting pipe i/o error, just check that read does not block
 		stdin.Read([]byte{})
 	})
 
 	// Cleanup pipes
 	if err := closeWrap(stdin, stdinPipe, stdout, stdoutPipe); err != nil {
 		t.Fatal(err)
 	}
 }
 
 // Expected behaviour: the process dies when the client disconnects
 func TestRunDisconnect(t *testing.T) {
 
 	stdin, stdinPipe := io.Pipe()
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
bc823acc
 
 	c1 := make(chan struct{})
 	go func() {
 		// We're simulating a disconnect so the return value doesn't matter. What matters is the
 		// fact that CmdRun returns.
 		cli.CmdRun("-i", unitTestImageID, "/bin/cat")
 		close(c1)
 	}()
 
 	setTimeout(t, "Read/Write assertion timed out", 2*time.Second, func() {
ad43d88a
 		if err := assertPipe("hello\n", "hello", stdout, stdinPipe, 150); err != nil {
bc823acc
 			t.Fatal(err)
 		}
 	})
 
 	// Close pipes (simulate disconnect)
 	if err := closeWrap(stdin, stdinPipe, stdout, stdoutPipe); err != nil {
 		t.Fatal(err)
 	}
 
 	// as the pipes are close, we expect the process to die,
 	// therefore CmdRun to unblock. Wait for CmdRun
 	setTimeout(t, "Waiting for CmdRun timed out", 2*time.Second, func() {
 		<-c1
 	})
 
 	// Client disconnect after run -i should cause stdin to be closed, which should
 	// cause /bin/cat to exit.
 	setTimeout(t, "Waiting for /bin/cat to exit timed out", 2*time.Second, func() {
359b7df5
 		container := globalDaemon.List()[0]
57d86a56
 		container.State.WaitStop(-1 * time.Second)
33e70864
 		if container.State.IsRunning() {
bc823acc
 			t.Fatalf("/bin/cat is still running after closing stdin")
 		}
 	})
 }
 
86c00be1
 // Expected behaviour: the process stay alive when the client disconnects
 // but the client detaches.
bc823acc
 func TestRunDisconnectTty(t *testing.T) {
 
 	stdin, stdinPipe := io.Pipe()
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
bc823acc
 
 	c1 := make(chan struct{})
 	go func() {
86c00be1
 		defer close(c1)
bc823acc
 		// We're simulating a disconnect so the return value doesn't matter. What matters is the
 		// fact that CmdRun returns.
 		if err := cli.CmdRun("-i", "-t", unitTestImageID, "/bin/cat"); err != nil {
8f39f0b5
 			utils.Debugf("Error CmdRun: %s", err)
bc823acc
 		}
 	}()
 
86c00be1
 	container := waitContainerStart(t, 10*time.Second)
bc823acc
 
86c00be1
 	state := setRaw(t, container)
 	defer unsetRaw(t, container, state)
bc823acc
 
86c00be1
 	// Client disconnect after run -i should keep stdin out in TTY mode
fe302fbf
 	setTimeout(t, "Read/Write assertion timed out", 2*time.Second, func() {
ad43d88a
 		if err := assertPipe("hello\n", "hello", stdout, stdinPipe, 150); err != nil {
bc823acc
 			t.Fatal(err)
 		}
 	})
 
 	// Close pipes (simulate disconnect)
 	if err := closeWrap(stdin, stdinPipe, stdout, stdoutPipe); err != nil {
 		t.Fatal(err)
 	}
 
86c00be1
 	// wait for CmdRun to return
 	setTimeout(t, "Waiting for CmdRun timed out", 5*time.Second, func() {
 		<-c1
 	})
 
bc823acc
 	// In tty mode, we expect the process to stay alive even after client's stdin closes.
 
 	// Give some time to monitor to do his thing
57d86a56
 	container.State.WaitStop(500 * time.Millisecond)
33e70864
 	if !container.State.IsRunning() {
bc823acc
 		t.Fatalf("/bin/cat should  still be running after closing stdin (tty mode)")
 	}
 }
 
2bd089da
 // TestRunDetach checks attaching and detaching with the escape sequence.
 func TestRunDetach(t *testing.T) {
 
 	stdin, stdinPipe := io.Pipe()
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
2bd089da
 
 	ch := make(chan struct{})
 	go func() {
 		defer close(ch)
 		cli.CmdRun("-i", "-t", unitTestImageID, "cat")
 	}()
 
86c00be1
 	container := waitContainerStart(t, 10*time.Second)
 
 	state := setRaw(t, container)
 	defer unsetRaw(t, container, state)
 
2bd089da
 	setTimeout(t, "First read/write assertion timed out", 2*time.Second, func() {
ad43d88a
 		if err := assertPipe("hello\n", "hello", stdout, stdinPipe, 150); err != nil {
2bd089da
 			t.Fatal(err)
 		}
 	})
 
 	setTimeout(t, "Escape sequence timeout", 5*time.Second, func() {
86c00be1
 		stdinPipe.Write([]byte{16})
 		time.Sleep(100 * time.Millisecond)
 		stdinPipe.Write([]byte{17})
2bd089da
 	})
 
 	// wait for CmdRun to return
135c1fce
 	setTimeout(t, "Waiting for CmdRun timed out", 15*time.Second, func() {
2bd089da
 		<-ch
 	})
86c00be1
 	closeWrap(stdin, stdinPipe, stdout, stdoutPipe)
2bd089da
 
e97364ec
 	time.Sleep(500 * time.Millisecond)
33e70864
 	if !container.State.IsRunning() {
e97364ec
 		t.Fatal("The detached container should be still running")
 	}
 
 	setTimeout(t, "Waiting for container to die timed out", 20*time.Second, func() {
2bd089da
 		container.Kill()
 	})
 }
 
b3974abe
 // TestAttachDetach checks that attach in tty mode can be detached using the long container ID
2bd089da
 func TestAttachDetach(t *testing.T) {
 	stdin, stdinPipe := io.Pipe()
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
2bd089da
 
3e014aa6
 	ch := make(chan struct{})
 	go func() {
 		defer close(ch)
2bd089da
 		if err := cli.CmdRun("-i", "-t", "-d", unitTestImageID, "cat"); err != nil {
 			t.Fatal(err)
 		}
3e014aa6
 	}()
2bd089da
 
697be6aa
 	container := waitContainerStart(t, 10*time.Second)
e535f544
 
3e014aa6
 	setTimeout(t, "Reading container's id timed out", 10*time.Second, func() {
 		buf := make([]byte, 1024)
 		n, err := stdout.Read(buf)
 		if err != nil {
 			t.Fatal(err)
 		}
 
b3974abe
 		if strings.Trim(string(buf[:n]), " \r\n") != container.ID {
 			t.Fatalf("Wrong ID received. Expect %s, received %s", container.ID, buf[:n])
3e014aa6
 		}
 	})
 	setTimeout(t, "Starting container timed out", 10*time.Second, func() {
 		<-ch
 	})
2bd089da
 
697be6aa
 	state := setRaw(t, container)
 	defer unsetRaw(t, container, state)
67e9e0e1
 
2bd089da
 	stdin, stdinPipe = io.Pipe()
 	stdout, stdoutPipe = io.Pipe()
185b040e
 	cli = client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
2bd089da
 
3e014aa6
 	ch = make(chan struct{})
2bd089da
 	go func() {
 		defer close(ch)
b3974abe
 		if err := cli.CmdAttach(container.ID); err != nil {
 			if err != io.ErrClosedPipe {
 				t.Fatal(err)
 			}
 		}
 	}()
 
 	setTimeout(t, "First read/write assertion timed out", 2*time.Second, func() {
ad43d88a
 		if err := assertPipe("hello\n", "hello", stdout, stdinPipe, 150); err != nil {
b3974abe
 			if err != io.ErrClosedPipe {
 				t.Fatal(err)
 			}
 		}
 	})
 
 	setTimeout(t, "Escape sequence timeout", 5*time.Second, func() {
63d6cbe3
 		stdinPipe.Write([]byte{16})
 		time.Sleep(100 * time.Millisecond)
 		stdinPipe.Write([]byte{17})
b3974abe
 	})
 
 	// wait for CmdRun to return
 	setTimeout(t, "Waiting for CmdAttach timed out", 15*time.Second, func() {
 		<-ch
 	})
 
63d6cbe3
 	closeWrap(stdin, stdinPipe, stdout, stdoutPipe)
 
b3974abe
 	time.Sleep(500 * time.Millisecond)
33e70864
 	if !container.State.IsRunning() {
b3974abe
 		t.Fatal("The detached container should be still running")
 	}
 
 	setTimeout(t, "Waiting for container to die timedout", 5*time.Second, func() {
 		container.Kill()
 	})
 }
 
 // TestAttachDetachTruncatedID checks that attach in tty mode can be detached
 func TestAttachDetachTruncatedID(t *testing.T) {
 	stdin, stdinPipe := io.Pipe()
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
b3974abe
 
86c00be1
 	// Discard the CmdRun output
b3974abe
 	go stdout.Read(make([]byte, 1024))
 	setTimeout(t, "Starting container timed out", 2*time.Second, func() {
 		if err := cli.CmdRun("-i", "-t", "-d", unitTestImageID, "cat"); err != nil {
 			t.Fatal(err)
 		}
 	})
 
2e6a9586
 	container := waitContainerStart(t, 10*time.Second)
 
 	state := setRaw(t, container)
 	defer unsetRaw(t, container, state)
b3974abe
 
 	stdin, stdinPipe = io.Pipe()
 	stdout, stdoutPipe = io.Pipe()
185b040e
 	cli = client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
b3974abe
 
 	ch := make(chan struct{})
 	go func() {
 		defer close(ch)
 		if err := cli.CmdAttach(utils.TruncateID(container.ID)); err != nil {
3e014aa6
 			if err != io.ErrClosedPipe {
 				t.Fatal(err)
 			}
2bd089da
 		}
 	}()
 
 	setTimeout(t, "First read/write assertion timed out", 2*time.Second, func() {
ad43d88a
 		if err := assertPipe("hello\n", "hello", stdout, stdinPipe, 150); err != nil {
3e014aa6
 			if err != io.ErrClosedPipe {
 				t.Fatal(err)
 			}
2bd089da
 		}
 	})
 
 	setTimeout(t, "Escape sequence timeout", 5*time.Second, func() {
2e6a9586
 		stdinPipe.Write([]byte{16})
 		time.Sleep(100 * time.Millisecond)
 		stdinPipe.Write([]byte{17})
2bd089da
 	})
 
 	// wait for CmdRun to return
135c1fce
 	setTimeout(t, "Waiting for CmdAttach timed out", 15*time.Second, func() {
2bd089da
 		<-ch
 	})
2e6a9586
 	closeWrap(stdin, stdinPipe, stdout, stdoutPipe)
2bd089da
 
e97364ec
 	time.Sleep(500 * time.Millisecond)
33e70864
 	if !container.State.IsRunning() {
e97364ec
 		t.Fatal("The detached container should be still running")
 	}
 
2bd089da
 	setTimeout(t, "Waiting for container to die timedout", 5*time.Second, func() {
 		container.Kill()
 	})
 }
 
bc823acc
 // Expected behaviour, the process stays alive when the client disconnects
 func TestAttachDisconnect(t *testing.T) {
 	stdin, stdinPipe := io.Pipe()
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(stdin, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
bc823acc
 
 	go func() {
 		// Start a process in daemon mode
 		if err := cli.CmdRun("-d", "-i", unitTestImageID, "/bin/cat"); err != nil {
8f39f0b5
 			utils.Debugf("Error CmdRun: %s", err)
bc823acc
 		}
 	}()
 
 	setTimeout(t, "Waiting for CmdRun timed out", 10*time.Second, func() {
 		if _, err := bufio.NewReader(stdout).ReadString('\n'); err != nil {
 			t.Fatal(err)
 		}
 	})
 
 	setTimeout(t, "Waiting for the container to be started timed out", 10*time.Second, func() {
 		for {
359b7df5
 			l := globalDaemon.List()
33e70864
 			if len(l) == 1 && l[0].State.IsRunning() {
bc823acc
 				break
 			}
 			time.Sleep(10 * time.Millisecond)
 		}
 	})
 
359b7df5
 	container := globalDaemon.List()[0]
bc823acc
 
 	// Attach to it
 	c1 := make(chan struct{})
 	go func() {
 		// We're simulating a disconnect so the return value doesn't matter. What matters is the
 		// fact that CmdAttach returns.
 		cli.CmdAttach(container.ID)
 		close(c1)
 	}()
 
 	setTimeout(t, "First read/write assertion timed out", 2*time.Second, func() {
ad43d88a
 		if err := assertPipe("hello\n", "hello", stdout, stdinPipe, 150); err != nil {
bc823acc
 			t.Fatal(err)
 		}
 	})
 	// Close pipes (client disconnects)
 	if err := closeWrap(stdin, stdinPipe, stdout, stdoutPipe); err != nil {
 		t.Fatal(err)
 	}
 
 	// Wait for attach to finish, the client disconnected, therefore, Attach finished his job
 	setTimeout(t, "Waiting for CmdAttach timed out", 2*time.Second, func() {
 		<-c1
 	})
 
 	// We closed stdin, expect /bin/cat to still be running
 	// Wait a little bit to make sure container.monitor() did his thing
57d86a56
 	_, err := container.State.WaitStop(500 * time.Millisecond)
33e70864
 	if err == nil || !container.State.IsRunning() {
bc823acc
 		t.Fatalf("/bin/cat is not running after closing stdin")
 	}
 
9b2a5964
 	// Try to avoid the timeout in destroy. Best effort, don't check error
bc823acc
 	cStdin, _ := container.StdinPipe()
 	cStdin.Close()
57d86a56
 	container.State.WaitStop(-1 * time.Second)
bc823acc
 }
22e7e107
 
 // Expected behaviour: container gets deleted automatically after exit
 func TestRunAutoRemove(t *testing.T) {
655b16c7
 	t.Skip("Fixme. Skipping test for now, race condition")
22e7e107
 	stdout, stdoutPipe := io.Pipe()
185b040e
 	cli := client.NewDockerCli(nil, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
22e7e107
 
 	c := make(chan struct{})
 	go func() {
 		defer close(c)
44fe8cbb
 		if err := cli.CmdRun("--rm", unitTestImageID, "hostname"); err != nil {
22e7e107
 			t.Fatal(err)
 		}
 	}()
 
 	var temporaryContainerID string
 	setTimeout(t, "Reading command output time out", 2*time.Second, func() {
 		cmdOutput, err := bufio.NewReader(stdout).ReadString('\n')
 		if err != nil {
 			t.Fatal(err)
 		}
 		temporaryContainerID = cmdOutput
 		if err := closeWrap(stdout, stdoutPipe); err != nil {
 			t.Fatal(err)
 		}
 	})
 
3e014aa6
 	setTimeout(t, "CmdRun timed out", 10*time.Second, func() {
22e7e107
 		<-c
 	})
 
 	time.Sleep(500 * time.Millisecond)
 
359b7df5
 	if len(globalDaemon.List()) > 0 {
22e7e107
 		t.Fatalf("failed to remove container automatically: container %s still exists", temporaryContainerID)
 	}
 }
177a2f59
 
35430e89
 // Expected behaviour: error out when attempting to bind mount non-existing source paths
 func TestRunErrorBindNonExistingSource(t *testing.T) {
 
185b040e
 	cli := client.NewDockerCli(nil, nil, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
35430e89
 
 	c := make(chan struct{})
 	go func() {
 		defer close(c)
2ec11466
 		// This check is made at runtime, can't be "unit tested"
35430e89
 		if err := cli.CmdRun("-v", "/i/dont/exist:/tmp", unitTestImageID, "echo 'should fail'"); err == nil {
 			t.Fatal("should have failed to run when using /i/dont/exist as a source for the bind mount")
 		}
 	}()
 
 	setTimeout(t, "CmdRun timed out", 5*time.Second, func() {
 		<-c
 	})
 }
d9fe0647
 
 func TestImagesViz(t *testing.T) {
7606412d
 	t.Skip("Image viz is deprecated")
d9fe0647
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(nil, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
d9fe0647
 
c001a5af
 	image := buildTestImages(t, globalEngine)
f1aeac36
 
d9fe0647
 	c := make(chan struct{})
 	go func() {
 		defer close(c)
35054601
 		if err := cli.CmdImages("--viz"); err != nil {
d9fe0647
 			t.Fatal(err)
 		}
 		stdoutPipe.Close()
 	}()
 
 	setTimeout(t, "Reading command output time out", 2*time.Second, func() {
 		cmdOutputBytes, err := ioutil.ReadAll(bufio.NewReader(stdout))
 		if err != nil {
 			t.Fatal(err)
 		}
 		cmdOutput := string(cmdOutputBytes)
 
 		regexpStrings := []string{
 			"digraph docker {",
 			fmt.Sprintf("base -> \"%s\" \\[style=invis]", unitTestImageIDShort),
 			fmt.Sprintf("label=\"%s\\\\n%s:latest\"", unitTestImageIDShort, unitTestImageName),
f1aeac36
 			fmt.Sprintf("label=\"%s\\\\n%s:%s\"", utils.TruncateID(image.ID), "test", "latest"),
d9fe0647
 			"base \\[style=invisible]",
 		}
 
 		compiledRegexps := []*regexp.Regexp{}
 		for _, regexpString := range regexpStrings {
 			regexp, err := regexp.Compile(regexpString)
 			if err != nil {
 				fmt.Println("Error in regex string: ", err)
 				return
 			}
 			compiledRegexps = append(compiledRegexps, regexp)
 		}
 
 		for _, regexp := range compiledRegexps {
 			if !regexp.MatchString(cmdOutput) {
35054601
 				t.Fatalf("images --viz content '%s' did not match regexp '%s'", cmdOutput, regexp)
d9fe0647
 			}
 		}
 	})
 }
f1aeac36
 
 func TestImagesTree(t *testing.T) {
7606412d
 	t.Skip("Image tree is deprecated")
f1aeac36
 	stdout, stdoutPipe := io.Pipe()
 
185b040e
 	cli := client.NewDockerCli(nil, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
f1aeac36
 
c001a5af
 	image := buildTestImages(t, globalEngine)
f1aeac36
 
 	c := make(chan struct{})
 	go func() {
 		defer close(c)
35054601
 		if err := cli.CmdImages("--tree"); err != nil {
f1aeac36
 			t.Fatal(err)
 		}
 		stdoutPipe.Close()
 	}()
 
 	setTimeout(t, "Reading command output time out", 2*time.Second, func() {
 		cmdOutputBytes, err := ioutil.ReadAll(bufio.NewReader(stdout))
 		if err != nil {
 			t.Fatal(err)
 		}
 		cmdOutput := string(cmdOutputBytes)
 		regexpStrings := []string{
c618a906
 			fmt.Sprintf("└─%s Virtual Size: \\d+.\\d+ MB Tags: %s:latest", unitTestImageIDShort, unitTestImageName),
08623dc2
 			"(?m)   └─[0-9a-f]+.*",
 			"(?m)    └─[0-9a-f]+.*",
 			"(?m)      └─[0-9a-f]+.*",
c618a906
 			fmt.Sprintf("(?m)^        └─%s Virtual Size: \\d+.\\d+ MB Tags: test:latest", utils.TruncateID(image.ID)),
f1aeac36
 		}
 
 		compiledRegexps := []*regexp.Regexp{}
 		for _, regexpString := range regexpStrings {
 			regexp, err := regexp.Compile(regexpString)
 			if err != nil {
 				fmt.Println("Error in regex string: ", err)
 				return
 			}
 			compiledRegexps = append(compiledRegexps, regexp)
 		}
 
 		for _, regexp := range compiledRegexps {
 			if !regexp.MatchString(cmdOutput) {
35054601
 				t.Fatalf("images --tree content '%s' did not match regexp '%s'", cmdOutput, regexp)
f1aeac36
 			}
 		}
 	})
 }
 
82a54398
 func buildTestImages(t *testing.T, eng *engine.Engine) *image.Image {
f1aeac36
 
 	var testBuilder = testContextTemplate{
 		`
 from   {IMAGE}
 run    sh -c 'echo root:testpass > /tmp/passwd'
 run    mkdir -p /var/run/sshd
 run    [ "$(cat /tmp/passwd)" = "root:testpass" ]
 run    [ "$(ls -d /var/run/sshd)" = "/var/run/sshd" ]
 `,
 		nil,
 		nil,
 	}
b04c6466
 	image, err := buildImage(testBuilder, t, eng, true)
 	if err != nil {
 		t.Fatal(err)
 	}
f1aeac36
 
e43ff2f6
 	if err := eng.Job("tag", image.ID, "test").Run(); err != nil {
f1aeac36
 		t.Fatal(err)
 	}
 
 	return image
 }
b3974abe
 
 // #2098 - Docker cidFiles only contain short version of the containerId
35054601
 //sudo docker run --cidfile /tmp/docker_test.cid ubuntu echo "test"
 // TestRunCidFile tests that run --cidfile returns the longid
a56d1b93
 func TestRunCidFileCheckIDLength(t *testing.T) {
b3974abe
 	stdout, stdoutPipe := io.Pipe()
 
 	tmpDir, err := ioutil.TempDir("", "TestRunCidFile")
 	if err != nil {
 		t.Fatal(err)
 	}
 	tmpCidFile := path.Join(tmpDir, "cid")
 
185b040e
 	cli := client.NewDockerCli(nil, stdoutPipe, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c001a5af
 	defer cleanup(globalEngine, t)
b3974abe
 
 	c := make(chan struct{})
 	go func() {
 		defer close(c)
35054601
 		if err := cli.CmdRun("--cidfile", tmpCidFile, unitTestImageID, "ls"); err != nil {
b3974abe
 			t.Fatal(err)
 		}
 	}()
 
 	defer os.RemoveAll(tmpDir)
 	setTimeout(t, "Reading command output time out", 2*time.Second, func() {
 		cmdOutput, err := bufio.NewReader(stdout).ReadString('\n')
 		if err != nil {
 			t.Fatal(err)
 		}
 		if len(cmdOutput) < 1 {
 			t.Fatalf("'ls' should return something , not '%s'", cmdOutput)
 		}
 		//read the tmpCidFile
 		buffer, err := ioutil.ReadFile(tmpCidFile)
 		if err != nil {
 			t.Fatal(err)
 		}
 		id := string(buffer)
 
 		if len(id) != len("2bf44ea18873287bd9ace8a4cb536a7cbe134bed67e805fdf2f58a57f69b320c") {
35054601
 			t.Fatalf("--cidfile should be a long id, not '%s'", id)
b3974abe
 		}
 		//test that its a valid cid? (though the container is gone..)
 		//remove the file and dir.
 	})
 
 	setTimeout(t, "CmdRun timed out", 5*time.Second, func() {
 		<-c
 	})
 
 }
c995c9bb
 
a56d1b93
 // Ensure that CIDFile gets deleted if it's empty
 // Perform this test by making `docker run` fail
 func TestRunCidFileCleanupIfEmpty(t *testing.T) {
 	tmpDir, err := ioutil.TempDir("", "TestRunCidFile")
 	if err != nil {
 		t.Fatal(err)
 	}
 	tmpCidFile := path.Join(tmpDir, "cid")
 
185b040e
 	cli := client.NewDockerCli(nil, ioutil.Discard, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
a56d1b93
 	defer cleanup(globalEngine, t)
 
 	c := make(chan struct{})
 	go func() {
 		defer close(c)
 		if err := cli.CmdRun("--cidfile", tmpCidFile, unitTestImageID); err == nil {
 			t.Fatal("running without a command should haveve failed")
 		}
 		if _, err := os.Stat(tmpCidFile); err == nil {
 			t.Fatalf("empty CIDFile '%s' should've been deleted", tmpCidFile)
 		}
 	}()
 	defer os.RemoveAll(tmpDir)
 
 	setTimeout(t, "CmdRun timed out", 5*time.Second, func() {
 		<-c
 	})
 }
 
c995c9bb
 func TestContainerOrphaning(t *testing.T) {
 
 	// setup a temporary directory
 	tmpDir, err := ioutil.TempDir("", "project")
 	if err != nil {
 		t.Fatal(err)
 	}
 	defer os.RemoveAll(tmpDir)
 
 	// setup a CLI and server
185b040e
 	cli := client.NewDockerCli(nil, ioutil.Discard, ioutil.Discard, testDaemonProto, testDaemonAddr, nil)
c995c9bb
 	defer cleanup(globalEngine, t)
 	srv := mkServerFromEngine(globalEngine, t)
 
 	// closure to build something
 	buildSomething := func(template string, image string) string {
 		dockerfile := path.Join(tmpDir, "Dockerfile")
 		replacer := strings.NewReplacer("{IMAGE}", unitTestImageID)
 		contents := replacer.Replace(template)
 		ioutil.WriteFile(dockerfile, []byte(contents), 0x777)
 		if err := cli.CmdBuild("-t", image, tmpDir); err != nil {
 			t.Fatal(err)
 		}
603e00a3
 		job := globalEngine.Job("image_get", image)
 		info, _ := job.Stdout.AddEnv()
 		if err := job.Run(); err != nil {
c995c9bb
 			t.Fatal(err)
 		}
68fb7f4b
 		return info.Get("Id")
c995c9bb
 	}
 
 	// build an image
 	imageName := "orphan-test"
 	template1 := `
 	from {IMAGE}
 	cmd ["/bin/echo", "holla"]
 	`
 	img1 := buildSomething(template1, imageName)
 
 	// create a container using the fist image
 	if err := cli.CmdRun(imageName); err != nil {
 		t.Fatal(err)
 	}
 
 	// build a new image that splits lineage
 	template2 := `
 	from {IMAGE}
 	cmd ["/bin/echo", "holla"]
 	expose 22
 	`
 	buildSomething(template2, imageName)
 
 	// remove the second image by name
795ed6b1
 	resp := engine.NewTable("", 0)
326f6a4b
 	if err := srv.DeleteImage(imageName, resp, true, false, false); err == nil {
795ed6b1
 		t.Fatal("Expected error, got none")
 	}
c995c9bb
 
 	// see if we deleted the first image (and orphaned the container)
564e6bc7
 	for _, i := range resp.Data {
 		if img1 == i.Get("Deleted") {
c995c9bb
 			t.Fatal("Orphaned image with container")
 		}
 	}
 
 }