progress: auto-select progress based on $PS1

bib/internal/progress/progress.go

@@ -69,9 +69,14 @@ type ProgressBar interface {
 // New creates a new progressbar based on the requested type
 func New(typ string) (ProgressBar, error) {
 	switch typ {
-	// XXX: autoseelct based on PS1 value (i.e. use term in
-	// interactive shells only?)
-	case "", "plain":
+	case "":
+		// autoselect based on if we are on an interactive
+		// shell, use plain progress for scripts
+		if _, ok := os.LookupEnv("PS1"); ok {
+			return NewTerminalProgressBar()
+		}
+		return NewPlainProgressBar()
+	case "plain":
 		return NewPlainProgressBar()
 	case "term":
 		return NewTerminalProgressBar()

bib/internal/progress/progress_test.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"fmt"
 	"reflect"
+	"strings"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -111,3 +112,23 @@ func TestTermProgress(t *testing.T) {
 	// check shutdown
 	assert.Contains(t, buf.String(), progress.CURSOR_SHOW)
 }
+
+func TestProgressNewAutoselect(t *testing.T) {
+	for _, tc := range []struct {
+		env      string
+		expected interface{}
+	}{
+		{"", &progress.PlainProgressBar{}},
+		{"PS1=foo", &progress.TerminalProgressBar{}},
+	} {
+
+		if l := strings.SplitN(tc.env, "=", 2); len(l) == 2 {
+			t.Setenv(l[0], l[1])
+		}
+
+		pb, err := progress.New("")
+		assert.NoError(t, err)
+		assert.Equal(t, reflect.TypeOf(pb), reflect.TypeOf(tc.expected), fmt.Sprintf("[%v] %T not the expected %T", tc.env, pb, tc.expected))
+	}
+
+}