bugfix use pointer mutex

Author: kokizzu

goproc.go

@@ -464,9 +464,9 @@ func Run1(cmd *Cmd) (string, string, error, int) {
 	onStdout := cmd.OnStdout
 	onStderr := cmd.OnStderr
 	stdoutBuff := bytes.Buffer{}
-	stdoutLock := sync.Mutex{}
+	stdoutLock := &sync.Mutex{}
 	stderrBuff := bytes.Buffer{}
-	stderrMutex := sync.Mutex{}
+	stderrMutex := &sync.Mutex{}
 	cmd.OnStdout = func(cmd *Cmd, s string) error {
 		stdoutLock.Lock()
 		stdoutBuff.WriteString(s)

pullpush.sh

@@ -20,6 +20,9 @@ go mod tidy
 ag gokil **/*.go && ( echo 'echo should not import previous gokil library..' ; kill 0 )
 echo "imports checked.."
 
+# test with race
+go test -race .
+
 # add and commit all files
 git add .
 git status

single_test.go

@@ -38,3 +38,13 @@ func TestRun1missing(t *testing.T) {
 	assert.Error(t, err)
 	assert.Equal(t, 0, exitCode)
 }
+
+func TestRun1long(t *testing.T) {
+	_, stderr, err, exitCode := Run1(&Cmd{
+		Program:    `ls`,
+		Parameters: []string{`-alr`},
+	})
+	assert.Equal(t, "", stderr)
+	assert.NoError(t, err)
+	assert.Equal(t, 0, exitCode)
+}