1. docker
  2. Commit 7ba9a18ade7393d942d995d34a92d757ef4af0b7
Browse code

cleanup: remove startCommand function, only used once, and unecessary abstraction.

Docker-DCO-1.1-Signed-off-by: Jessica Frazelle <jess@docker.com> (github: jfrazelle)

Jessica Frazelle authored on 2014/12/04 10:42:25
Showing 2 changed files
integration-cli/docker_cli_attach_test.go
History View file @ 7ba9a18
... ... 
@@ -50,7 +50,7 @@ func TestAttachMultipleAndRestart(t *testing.T) {
50 50 
 				t.Fatal(err)
51 51 
 			}
52 52
53 
-			if _, err := startCommand(c); err != nil {
53 
+			if err := c.Start(); err != nil {
54 54 
 				t.Fatal(err)
55 55 
 			}
56 56
integration-cli/utils.go
History View file @ 7ba9a18
... ... 
@@ -95,13 +95,6 @@ func runCommand(cmd *exec.Cmd) (exitCode int, err error) {
95 95 
 	return
96 96 
 }
97 97
98 
-func startCommand(cmd *exec.Cmd) (exitCode int, err error) {
99 
-	exitCode = 0
100 
-	err = cmd.Start()
101 
-	exitCode = processExitCode(err)
102 
-	return
103 
-}
104 
-
105 98 
 func logDone(message string) {
106 99 
 	fmt.Printf("[PASSED]: %s\n", message)
107 100 
 }