chore: remove CaptureStandardOutput and reference as they aren't needed

Author: dylanhitt

command_linux_test.go

@@ -125,17 +125,6 @@ func TestWithCustomStdout(t *testing.T) {
 	assertEqualWithLineBreak(t, "stderr\nstdout", c.Combined())
 }
 
-func TestWithStandardStreams(t *testing.T) {
-	out, err := CaptureStandardOutput(func() interface{} {
-		c := NewCommand(">&2 echo stderr; sleep 0.01; echo stdout;", WithStandardStreams)
-		err := c.Execute()
-		return err
-	})
-
-	assertEqualWithLineBreak(t, "stderr\nstdout", out)
-	assert.Nil(t, err)
-}
-
 func TestWithEnvironmentVariables(t *testing.T) {
 	c := NewCommand("echo $env", WithEnvironmentVariables(map[string]string{"env": "value"}))
 	c.Execute()

testing_utils.go

@@ -1,48 +1,11 @@
 package cmd
 
 import (
-	"bytes"
-	"github.com/stretchr/testify/assert"
-	"io"
-	"log"
-	"os"
 	"runtime"
-	"sync"
 	"testing"
-)
 
-// CaptureStandardOutput allows to capture the output which will be written
-// to os.Stdout and os.Stderr.
-// It returns the captured output and the return value of the called function
-func CaptureStandardOutput(f func() interface{}) (string, interface{}) {
-	reader, writer, err := os.Pipe()
-	if err != nil {
-		panic(err)
-	}
-	stdout := os.Stdout
-	stderr := os.Stderr
-	defer func() {
-		os.Stdout = stdout
-		os.Stderr = stderr
-		log.SetOutput(os.Stderr)
-	}()
-	os.Stdout = writer
-	os.Stderr = writer
-	log.SetOutput(writer)
-	out := make(chan string)
-	wg := new(sync.WaitGroup)
-	wg.Add(1)
-	go func() {
-		var buf bytes.Buffer
-		wg.Done()
-		io.Copy(&buf, reader)
-		out <- buf.String()
-	}()
-	wg.Wait()
-	result := f()
-	writer.Close()
-	return <-out, result
-}
+	"github.com/stretchr/testify/assert"
+)
 
 func assertEqualWithLineBreak(t *testing.T, expected string, actual string) {
 	if runtime.GOOS == "windows" {