From 7ba9a18ade7393d942d995d34a92d757ef4af0b7 Mon Sep 17 00:00:00 2001 From: Jessica Frazelle Date: Wed, 3 Dec 2014 17:42:25 -0800 Subject: [PATCH] cleanup: remove startCommand function, only used once, and unecessary abstraction. Docker-DCO-1.1-Signed-off-by: Jessica Frazelle (github: jfrazelle) --- integration-cli/docker_cli_attach_test.go | 2 +- integration-cli/utils.go | 7 ------- 2 files changed, 1 insertion(+), 8 deletions(-) diff --git a/integration-cli/docker_cli_attach_test.go b/integration-cli/docker_cli_attach_test.go index 510f02ab18..d03a986e48 100644 --- a/integration-cli/docker_cli_attach_test.go +++ b/integration-cli/docker_cli_attach_test.go @@ -50,7 +50,7 @@ func TestAttachMultipleAndRestart(t *testing.T) { t.Fatal(err) } - if _, err := startCommand(c); err != nil { + if err := c.Start(); err != nil { t.Fatal(err) } diff --git a/integration-cli/utils.go b/integration-cli/utils.go index 05c27dc5ac..37ce221b9b 100644 --- a/integration-cli/utils.go +++ b/integration-cli/utils.go @@ -95,13 +95,6 @@ func runCommand(cmd *exec.Cmd) (exitCode int, err error) { return } -func startCommand(cmd *exec.Cmd) (exitCode int, err error) { - exitCode = 0 - err = cmd.Start() - exitCode = processExitCode(err) - return -} - func logDone(message string) { fmt.Printf("[PASSED]: %s\n", message) }