commit 6aabc1ca790df3e0b556593b27bc25d26b151751
author Ian Lance Taylor <iant@golang.org> Fri Mar 22 07:22:32 2019 -0700
committer Ian Lance Taylor <iant@golang.org> Fri Mar 22 14:36:16 2019 +0000
tree 1d163bd0ca9c5943fa19373ee83e1e460915ac2e
parent e250d351ecad161c4cf2f50bce18a1b2df5dfd78

go/analysis/passes/tests: don't warn about missing method for each type

Only warn if the method is missing for all types.

Fixes golang/go#30971

Change-Id: I94169ad3266f68ca20378a8dc5538aed2541a773
Reviewed-on: https://go-review.googlesource.com/c/tools/+/168803
Run-TryBot: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gobot Gobot <gobot@golang.org>

go/analysis/passes/tests/testdata/src/b_x_test/b_test.go

diff --git a/go/analysis/passes/tests/testdata/src/b_x_test/b_test.go b/go/analysis/passes/tests/testdata/src/b_x_test/b_test.go
index 6779195..2fac000 100644
--- a/go/analysis/passes/tests/testdata/src/b_x_test/b_test.go
+++ b/go/analysis/passes/tests/testdata/src/b_x_test/b_test.go
@@ -1,10 +1,14 @@
 package b_x_test
 
-import "b"
+import (
+	"a"
+	"b"
+)
 
 func ExampleFoo_F() {
 	var x b.Foo
 	x.F()
+	a.Foo()
 }
 
 func ExampleFoo_G() { // want "ExampleFoo_G refers to unknown field or method: Foo.G"

go/analysis/passes/tests/tests.go

diff --git a/go/analysis/passes/tests/tests.go b/go/analysis/passes/tests/tests.go
index 6b1e9f8..9202b35 100644
--- a/go/analysis/passes/tests/tests.go
+++ b/go/analysis/passes/tests/tests.go
@@ -153,9 +153,9 @@
 			if obj, _, _ := types.LookupFieldOrMethod(obj.Type(), true, obj.Pkg(), mmbr); obj != nil {
 				found = true
 			}
-			if !found {
-				pass.Reportf(fn.Pos(), "%s refers to unknown field or method: %s.%s", fnName, ident, mmbr)
-			}
+		}
+		if !found {
+			pass.Reportf(fn.Pos(), "%s refers to unknown field or method: %s.%s", fnName, ident, mmbr)
 		}
 	}
 	if len(elems) == 3 && !isExampleSuffix(elems[2]) {