remove *exec.Cmd as it is already covered by Runner interface + tidy …

…up test

errors_test.go

@@ -11,6 +11,7 @@ import (
 func TestErrors(t *testing.T) {
 	var errors runn.Errors
 	errors.Add(fmt.Errorf("error1"))
+	errors.Add(nil)
 	errors.Add(fmt.Errorf("error2"))
 	errors.Add(fmt.Errorf("error3"))
 

executor.go

@@ -1,11 +1,5 @@
 package runn
 
-import (
-	"bytes"
-	"fmt"
-	"os/exec"
-)
-
 // Executor do the code statement execution
 type Executor struct {
 	StopWhenError bool
@@ -31,20 +25,6 @@ func (e Executor) Execute(stmts ...interface{}) (err error) {
 				}
 				errs.Add(runErr)
 			}
-		case *exec.Cmd:
-			execCmd := stmt.(*exec.Cmd)
-			buf := bytes.Buffer{}
-			execCmd.Stdout = &buf
-			execCmd.Stderr = &buf
-
-			runErr := execCmd.Run()
-			if err != nil {
-				runErr = fmt.Errorf("%s: %s", err.Error(), buf.String())
-				if e.StopWhenError {
-					return runErr
-				}
-				errs.Add(runErr)
-			}
 		}
 	}
 	if len(errs) > 0 {

executor_test.go

@@ -12,7 +12,7 @@ import (
 
 type RunnerImplementationWithError struct{}
 
-func (i RunnerImplementationWithError) Run() error { return errors.New("some-error") }
+func (i RunnerImplementationWithError) Run() error { return errors.New("some-runner-error") }
 
 type RunnerImplementationNoError struct{}
 
@@ -27,12 +27,12 @@ func TestExecutor_All(t *testing.T) {
 		{
 			false,
 			[]interface{}{
-				RunnerImplementationWithError{},
 				errors.New("error1"),
+				RunnerImplementationWithError{},
 				errors.New("error2"),
 				exec.Command("wrong-command", "bad-argument"),
 			},
-			errors.New("some-error; error1; error2; exec: \"wrong-command\": executable file not found in $PATH"),
+			errors.New("error1; some-runner-error; error2; exec: \"wrong-command\": executable file not found in $PATH"),
 		},
 		{
 			true,
@@ -43,15 +43,27 @@ func TestExecutor_All(t *testing.T) {
 			errors.New("error1"),
 		},
 		{
-			false,
-			[]interface{}{},
-			nil,
+			true,
+			[]interface{}{
+				RunnerImplementationWithError{},
+				errors.New("unreachable-error"),
+			},
+			errors.New("some-runner-error"),
 		},
 		{
 			true,
+			[]interface{}{
+				exec.Command("wrong-command", "bad-argument"),
+				errors.New("unreachable-error"),
+			},
+			errors.New("exec: \"wrong-command\": executable file not found in $PATH"),
+		},
+		{
+			false,
 			[]interface{}{},
 			nil,
 		},
+
 		{
 			false,
 			[]interface{}{

runner.go

@@ -1,5 +1,6 @@
 package runn
 
+// Runner contain run function
 type Runner interface {
 	Run() error
 }