FAQ
Please take another look.


https://codereview.appspot.com/6499102/diff/5003/src/pkg/go/build/build_test.go
File src/pkg/go/build/build_test.go (right):

https://codereview.appspot.com/6499102/diff/5003/src/pkg/go/build/build_test.go#newcode67
src/pkg/go/build/build_test.go:67: t.Fatal("Import should not accept
empty strings.")
On 2012/09/13 14:05:51, rsc wrote:
Use Go syntax to explain what happened.
t.Fatalf(`Import("") returned nil error`)
Done.

https://codereview.appspot.com/6499102/diff/5003/src/pkg/go/build/build_test.go#newcode70
src/pkg/go/build/build_test.go:70: t.Fatal("Import should return non-nil
package containing partial information.")
On 2012/09/13 14:05:51, rsc wrote:
t.Fatal(`Import("") returned nil Package`)
Done.

https://codereview.appspot.com/6499102/diff/5003/src/pkg/go/build/build_test.go#newcode73
src/pkg/go/build/build_test.go:73: t.Fatalf(`Wrong import path, want "",
got %s.`, p.ImportPath)
On 2012/09/13 14:05:51, rsc wrote:
Please reverse the message to print in the same order you wrote the if
statement.
t.Fatalf("ImportPath=%q, want %q", p.ImportPath, "")
Done.

https://codereview.appspot.com/6499102/

Search Discussions

Discussion Posts

Previous

Follow ups

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 11 of 15 | next ›
Discussion Overview
groupgolang-dev @
categoriesgo
postedSep 12, '12 at 8:46p
activeSep 13, '12 at 2:34p
posts15
users3
websitegolang.org

People

Translate

site design / logo © 2022 Grokbase