cmd/go: fix bad error message in coverage for package without non-test files

Was checking for nil map; must check for empty map instead.

Fixes #6065

Before:

go test -cover
# testmain
/var/folders/00/013l0000h01000cxqpysvccm0004fc/T/go-build233480051/_/Users/r/issue/_test/_testmain.go:11: imported and not used: "_/Users/r/issue"
FAIL	_/Users/r/issue [build failed]

Now:

go test -cover
testing: warning: no tests to run
PASS
coverage: 0.0% of statements
ok  	_/Users/r/issue	0.021s

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/12916043

diff --git a/src/cmd/go/test.go b/src/cmd/go/test.go
index c197007c43686d7de2899712b1e91d061f61a01a..d4a1c50f410c1c07e12bf16d10dab1b781a104af 100644 (file)
--- a/src/cmd/go/test.go
+++ b/src/cmd/go/test.go
@@ -1010,7 +1010,7 @@ type coverInfo struct {
 func writeTestmain(out string, pmain, p *Package) error {
        var cover []coverInfo
        for _, cp := range pmain.imports {
-               if cp.coverVars != nil {
+               if len(cp.coverVars) > 0 {
                        cover = append(cover, coverInfo{cp, cp.coverVars})
                }
        }