Merge pull request #177 from ngageoint/issue176

Closes #176 added red text for stderr output

.gitignore

@@ -13,3 +13,6 @@ temp-*
 
 # test manifests updated during tests
 testdata/complete/seed.manifest.json
+
+# useless OSX Finder files
+*.DS_Store

commands/run.go

@@ -18,7 +18,9 @@ import (
 	"syscall"
 	"time"
 
+	"github.com/fatih/color"
 	"github.com/ngageoint/seed-cli/constants"
+	"github.com/ngageoint/seed-cli/streampainter"
 	common_const "github.com/ngageoint/seed-common/constants"
 	"github.com/ngageoint/seed-common/objects"
 	"github.com/ngageoint/seed-common/util"
@@ -157,7 +159,7 @@ func DockerRun(imageName, outputDir, metadataSchema string, inputs, json, settin
 	dockerRun := exec.Command("docker", dockerArgs...)
 	var errs bytes.Buffer
 	if !quiet {
-		dockerRun.Stderr = io.MultiWriter(&errs, os.Stderr)
+		dockerRun.Stderr = io.MultiWriter(&errs, streampainter.NewStreamPainter(color.FgRed))
 		dockerRun.Stdout = os.Stderr
 	}
 

streampainter/streampainter.go

@@ -0,0 +1,24 @@
+package streampainter
+
+import (
+	"os"
+
+	"github.com/fatih/color"
+)
+
+type StreamPainter struct {
+	paintColor color.Attribute
+}
+
+/*
+ * NewStreamPainter returns new streampainter
+ */
+func NewStreamPainter(textColor color.Attribute) *StreamPainter {
+	return &StreamPainter{textColor}
+}
+
+func (w *StreamPainter) Write(p []byte) (int, error) {
+	n := len(p)
+	color.New(w.paintColor).Fprintf(os.Stderr, string(p[:n]))
+	return n, nil
+}

streampainter/streampainter_test.go

@@ -0,0 +1,18 @@
+package streampainter
+
+import (
+	"testing"
+
+	"github.com/fatih/color"
+)
+
+func TestStreamPainter(t *testing.T) {
+	writer := NewStreamPainter(color.FgRed)
+	_, err := writer.Write([]byte("should be in red"))
+	if err != nil {
+		t.Errorf("Error should be 'nil', but was %v", err.Error())
+	}
+	if writer.paintColor != color.FgRed {
+		t.Errorf("Assigned color expected FgRed, but was %v", writer.paintColor)
+	}
+}