From b15a1e3cfb64aeeb90f74e0748524b38fde5ebf9 Mon Sep 17 00:00:00 2001 From: Michal Bohuslávek Date: Thu, 23 Aug 2018 19:51:50 +0200 Subject: text/template: Put bad function name in quotes in panic from (*Template).Funcs This turns panic: function name is not a valid identifier into panic: function name "" is not a valid identifier and also makes it consistent with the func signature check. This CL also makes the testBadFuncName func a test helper. Change-Id: Id967cb61ac28228de81e1cd76a39f5195a5ebd11 Reviewed-on: https://go-review.googlesource.com/130998 Reviewed-by: Brad Fitzpatrick Run-TryBot: Brad Fitzpatrick TryBot-Result: Gobot Gobot --- src/text/template/exec_test.go | 1 + 1 file changed, 1 insertion(+) (limited to 'src/text/template/exec_test.go') diff --git a/src/text/template/exec_test.go b/src/text/template/exec_test.go index 6f40d80635..648ad8ff03 100644 --- a/src/text/template/exec_test.go +++ b/src/text/template/exec_test.go @@ -1279,6 +1279,7 @@ func TestBadFuncNames(t *testing.T) { } func testBadFuncName(name string, t *testing.T) { + t.Helper() defer func() { recover() }() -- cgit v1.3-5-g9baa