tests/lowlevel_test.go: Only compare stdout, ignore stderr.

dmitshur · dmitshur · commit fcc469a4d7af · 2016-04-30T16:28:06.000-07:00
The goal of the test is to compare stdout to make sure it equals to
what is expected.

Ignore stderr as there may be unrelated warnings there, for example the
warning that GopherJS prints when source map support module is not
installed.

Fixes the following invalid test failure:

	go test -run=TestTimeInternalizationExternalization github.com/gopherjs/gopherjs/tests
	--- FAIL: TestTimeInternalizationExternalization (0.30s)
		lowlevel_test.go:29: got != want:
			got:
			gopherjs: Source maps disabled. Use Node.js 4.x with source-map-support module for nice stack traces.
			{ S: 'externalization ok' }
			Sat, 29 Aug 2015 20:56:00 GMT

			want:
			{ S: 'externalization ok' }
			Sat, 29 Aug 2015 20:56:00 GMT
	FAIL
	FAIL	github.com/gopherjs/gopherjs/tests	0.313s
diff --git a/tests/lowlevel_test.go b/tests/lowlevel_test.go
@@ -15,7 +15,7 @@ import (
 //
 // See https://github.com/gopherjs/gopherjs/issues/279.
 func TestTimeInternalizationExternalization(t *testing.T) {
-	got, err := exec.Command("gopherjs", "run", filepath.Join("testdata", "time_inexternalization.go")).CombinedOutput()
+	got, err := exec.Command("gopherjs", "run", filepath.Join("testdata", "time_inexternalization.go")).Output()
 	if err != nil {
 		t.Fatalf("%v:\n%s", err, got)
 	}

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ import (`
`15`	`15`	`//`
`16`	`16`	`// See https://github.com/gopherjs/gopherjs/issues/279.`
`17`	`17`	`func TestTimeInternalizationExternalization(t *testing.T) {`
`18`		`- got, err := exec.Command("gopherjs", "run", filepath.Join("testdata", "time_inexternalization.go")).CombinedOutput()`
	`18`	`+ got, err := exec.Command("gopherjs", "run", filepath.Join("testdata", "time_inexternalization.go")).Output()`
`19`	`19`	`if err != nil {`
`20`	`20`	`t.Fatalf("%v:\n%s", err, got)`
`21`	`21`	`}`