diff --git a/rule/unhandled-error.go b/rule/unhandled-error.go index 32a5fe48b..ce6fa3864 100644 --- a/rule/unhandled-error.go +++ b/rule/unhandled-error.go @@ -119,7 +119,7 @@ func (w *lintUnhandledErrors) addFailure(n *ast.CallExpr) { Category: "bad practice", Confidence: 1, Node: n, - Failure: fmt.Sprintf("Unhandled error in call to function %v", gofmt(n.Fun)), + Failure: fmt.Sprintf("Unhandled error in call to function %v", name), }) } diff --git a/testdata/unhandled-error-w-ignorelist.go b/testdata/unhandled-error-w-ignorelist.go index e1050c48c..5f21dc0d7 100644 --- a/testdata/unhandled-error-w-ignorelist.go +++ b/testdata/unhandled-error-w-ignorelist.go @@ -56,7 +56,7 @@ func testCase4() { b1.Write(nil) // ignore b2.Write(nil) // ignore - b2.Read([]byte("bytes")) // MATCH /Unhandled error in call to function b2.Read/ + b2.Read([]byte("bytes")) // MATCH /Unhandled error in call to function bytes.Buffer.Read/ } type unhandledErrorStruct1 struct { @@ -81,9 +81,9 @@ func testCase5() { // fixtures\.unhandledErrorStruct2\.reterr s1 := unhandledErrorStruct1{} _ = s1.reterr() - s1.reterr() // MATCH /Unhandled error in call to function s1.reterr/ + s1.reterr() // MATCH /Unhandled error in call to function fixtures.unhandledErrorStruct1.reterr/ s2 := unhandledErrorStruct2{} s2.reterr() // ignore - s2.reterr1() // MATCH /Unhandled error in call to function s2.reterr1/ + s2.reterr1() // MATCH /Unhandled error in call to function fixtures.unhandledErrorStruct2.reterr1/ }