Fix StandardStreamTest and ignore exmaples in test suite

Author: SimonBaeumer

.travis.yml

@@ -9,7 +9,6 @@ stages:
   - test
 
 go:
-  - 1.12.x
   - 1.13.x
 
 sudo: required
@@ -28,11 +27,6 @@ jobs:
       script:
         - make test
 
-    - name: macOS integration
-      os: osx
-      script:
-        - make integration
-
     - name: windows Unit
       os: windows
       before_install:

Makefile

@@ -24,8 +24,12 @@ lint:
 
 test:
 	$(info INFO: Starting build $@)
-	go test ./...
+	go test `go list ./... | grep -v examples`
+
+test-windows:
+	$(info INFO: Starting build $@)
+	go test .
 
 test-coverage:
 	$(info INFO: Starting build $@)
-	go test -coverprofile c.out ./...
+	go test -coverprofile c.out `go list ./... | grep -v examples`

README.md

@@ -6,7 +6,7 @@
 
 # cmd package
 
-A simple package to execute shell commands on windows, darwin and windows.
+A simple package to execute shell commands on linux, darwin and windows.
 
 ## Usage
 
@@ -58,6 +58,4 @@ make test
     - Coverage reports
     - Go report
     - Codeclimate
- - Travis-Pipeline
- - Documentation
  - os.Stdout and os.Stderr output access after execution

command.go

@@ -22,12 +22,26 @@ type Command struct {
 	StdoutWriter io.Writer
 	WorkingDir   string
 	executed     bool
+	exitCode     int
+	// stderr and stdout retrieve the output after the command was executed
 	stderr       bytes.Buffer
 	stdout       bytes.Buffer
-	exitCode     int
 }
 
-//NewCommand creates a new command
+// NewCommand creates a new command
+// You can add option with variadic option argument
+//
+// Example:
+//      c := cmd.NewCommand("echo hello", function (c *Command) {
+//		    c.WorkingDir = "/tmp"
+//      })
+//      c.Execute()
+//
+// or you can use existing options functions
+//
+//      c := cmd.NewCommand("echo hello", cmd.WithStandardStreams)
+//      c.Execute()
+//
 func NewCommand(cmd string, options ...func(*Command)) *Command {
 	c := &Command{
 		Command:  cmd,

command_linux_test.go

@@ -2,6 +2,8 @@ package cmd
 
 import (
 	"github.com/stretchr/testify/assert"
+	"io/ioutil"
+	"os"
 	"strings"
 	"testing"
 )
@@ -48,8 +50,19 @@ func TestCommand_WithWorkingDir(t *testing.T) {
 }
 
 func TestCommand_WithStandardStreams(t *testing.T) {
+	tmpFile, _ := ioutil.TempFile("/tmp", "stdout_")
+	originalStdout := os.Stdout
+	os.Stdout = tmpFile
+
+	// Reset os.Stdout to its original value
+	defer func() {
+		os.Stdout = originalStdout
+	}()
+
 	cmd := NewCommand("echo hey", WithStandardStreams)
 	cmd.Execute()
 
-	assert.Equal(t, "/tmp\n", cmd.Stdout())
+	r, err  := ioutil.ReadFile(tmpFile.Name())
+	assert.Nil(t, err)
+	assert.Equal(t, "hey\n", string(r))
 }