https://codereview.appspot.com/8171044/diff/4002/src/pkg/testing/testing.go File src/pkg/testing/testing.go (right): https://codereview.appspot.com/8171044/diff/4002/src/pkg/testing/testing.go#newcode14 src/pkg/testing/testing.go:14: // Tests may also be declared in external test ...
11 years, 1 month ago
(2013-03-29 21:04:33 UTC)
#2
https://codereview.appspot.com/8171044/diff/14001/src/pkg/testing/testing.go File src/pkg/testing/testing.go (right): https://codereview.appspot.com/8171044/diff/14001/src/pkg/testing/testing.go#newcode12 src/pkg/testing/testing.go:12: // in which case they can access all identifiers ...
11 years, 1 month ago
(2013-03-29 22:15:18 UTC)
#4
Here is a not very good attempt at matching the pity style of this package's ...
11 years, 1 month ago
(2013-03-30 05:52:30 UTC)
#6
Here is a not very good attempt at matching the pity style of this package's
comments.
https://codereview.appspot.com/8171044/diff/22001/src/pkg/testing/testing.go
File src/pkg/testing/testing.go (right):
https://codereview.appspot.com/8171044/diff/22001/src/pkg/testing/testing.go#...
src/pkg/testing/testing.go:15: // by ``go build'' but recognized by ``go test''
and compiled with the main package
Suggestion
// Additional tests may also be placed into ```_test.go``` files in the same
// directory using a package declaration including the suffix ```_test```.
// These are called external tests, and can be useful for breaking dependency
// cycles, or for black box testing.
//
// External tests do not occupy the same package namespace as internal tests
// so must ```import``` the package under test directory, and have no access too
// unexported symbols.
Issue 8171044: code review 8171044: cmd/go: document usage of external test packages
Created 11 years, 1 month ago by kisielk
Modified 10 years, 9 months ago
Reviewers: khr
Base URL:
Comments: 6