testing: Benchmark prints nothing on failure

[rogpeppe]

Benchmark functions can call Log, Error and Fatal, but if the benchmark is run with the Benchmark function, all messages are discarded.

https://play.golang.org/p/fd5Ed-il70V

Perhaps the messages should be printed to standard error?

[fodil-a]

Hi, I can confirm that the output is discarded, here is the writer used in the context of the benchmark

go/src/testing/benchmark.go

Line 772 in fbc6a97

w: discard{},

Calls from Log, Error, or Fatal will print nothing, because the Write call of the discard writer does nothing.

go/src/testing/benchmark.go

Lines 783 to 785 in fbc6a97

	type discard struct{}
	func (discard) Write(b []byte) (n int, err error) { return len(b), nil }

[lahsivjar]

I stumbled upon this today, any updates on this? It feels like a pretty valid case and I was hoping it could do one of the following:

1. testing.Benchmark takes io.Writer as an input.
2. Allow users to create a custom testing.B and call Benchmark on top of it.

[fodil-a]

If anyone from the team agrees the the approach of creating a new func that take a io.Writer and uses it here

go/src/testing/benchmark.go

Line 772 in fbc6a97

w: discard{},

I can do the code, should be pretty easy to add and test 😊