tests: Include stderr in TestGopherJSCanBeVendored output.

dmitshur · dmitshur · commit 8a9d21b0b1d3 · 2018-04-15T20:37:10.000-04:00
This way, we get more information when/if TestGopherJSCanBeVendored fails. Before: $ go test -v -run=TestGopherJSCanBeVendored === RUN TestGopherJSCanBeVendored --- FAIL: TestGopherJSCanBeVendored (5.20s) gorepo_test.go:38: exit status 1 FAIL exit status 1 FAIL github.com/gopherjs/gopherjs/tests 5.203s After: $ go test -v -run=TestGopherJSCanBeVendored === RUN TestGopherJSCanBeVendored vendoring github.com/gopherjs/gopherjs/js package is not supported, see #415 --- FAIL: TestGopherJSCanBeVendored (5.19s) gorepo_test.go:40: exit status 1 FAIL exit status 1 FAIL github.com/gopherjs/gopherjs/tests 5.190s
diff --git a/tests/gorepo_test.go b/tests/gorepo_test.go
@@ -33,7 +33,9 @@ func TestGopherJSCanBeVendored(t *testing.T) {
 		t.Skip("test meant to be run using normal Go compiler (needs os/exec)")
 	}
 
-	got, err := exec.Command("bash", "gopherjsvendored_test.sh").Output()
+	cmd := exec.Command("bash", "gopherjsvendored_test.sh")
+	cmd.Stderr = os.Stdout
+	got, err := cmd.Output()
 	if err != nil {
 		t.Fatal(err)
 	}

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,9 @@ func TestGopherJSCanBeVendored(t *testing.T) {`
`33`	`33`	`t.Skip("test meant to be run using normal Go compiler (needs os/exec)")`
`34`	`34`	`}`
`35`	`35`
`36`		`- got, err := exec.Command("bash", "gopherjsvendored_test.sh").Output()`
	`36`	`+ cmd := exec.Command("bash", "gopherjsvendored_test.sh")`
	`37`	`+ cmd.Stderr = os.Stdout`
	`38`	`+ got, err := cmd.Output()`
`37`	`39`	`if err != nil {`
`38`	`40`	`t.Fatal(err)`
`39`	`41`	`}`