tpl/internal: Synch Go templates fork with Go 1.16dev
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Thu, 3 Dec 2020 12:50:17 +0000 (13:50 +0100)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Thu, 18 Feb 2021 13:11:48 +0000 (14:11 +0100)
25 files changed:
scripts/fork_go_templates/main.go
tpl/internal/go_templates/cfg/cfg.go
tpl/internal/go_templates/fmtsort/sort.go
tpl/internal/go_templates/fmtsort/sort_test.go
tpl/internal/go_templates/htmltemplate/clone_test.go
tpl/internal/go_templates/htmltemplate/content_test.go
tpl/internal/go_templates/htmltemplate/escape.go
tpl/internal/go_templates/htmltemplate/escape_test.go
tpl/internal/go_templates/htmltemplate/exec_test.go [new file with mode: 0644]
tpl/internal/go_templates/htmltemplate/html.go
tpl/internal/go_templates/htmltemplate/multi_test.go [new file with mode: 0644]
tpl/internal/go_templates/htmltemplate/template.go
tpl/internal/go_templates/htmltemplate/template_test.go
tpl/internal/go_templates/testenv/testenv.go
tpl/internal/go_templates/texttemplate/doc.go
tpl/internal/go_templates/texttemplate/exec.go
tpl/internal/go_templates/texttemplate/exec_test.go
tpl/internal/go_templates/texttemplate/helper.go
tpl/internal/go_templates/texttemplate/link_test.go [new file with mode: 0644]
tpl/internal/go_templates/texttemplate/multi_test.go
tpl/internal/go_templates/texttemplate/parse/lex.go
tpl/internal/go_templates/texttemplate/parse/lex_test.go
tpl/internal/go_templates/texttemplate/parse/node.go
tpl/internal/go_templates/texttemplate/parse/parse.go
tpl/internal/go_templates/texttemplate/parse/parse_test.go

index b66c8a111058e0bd096be58e580e57bd740b01ed..4f8bb93d74f7c74234811191003ad83ce5694f7d 100644 (file)
@@ -18,7 +18,7 @@ import (
 
 func main() {
        // TODO(bep) git checkout tag
-       // The current is built with Go version b68fa57c599720d33a2d735782969ce95eabf794 / go1.15dev
+       // The current is built with Go version da54dfb6a1f3bef827b9ec3780c98fde77a97d11 / go1.16dev
        fmt.Println("Forking ...")
        defer fmt.Println("Done ...")
 
index bdbe9df3e75d13a9e4376f5471143bc6a20d6522..553021374d5f1c0dff5317297c9268eaa8c0e0e5 100644 (file)
@@ -58,6 +58,7 @@ const KnownEnv = `
        GOSUMDB
        GOTMPDIR
        GOTOOLDIR
+       GOVCS
        GOWASM
        GO_EXTLINK_ENABLED
        PKG_CONFIG
index b01229bd06aab68aeb70212c4ff0ad22d054edfc..7127ba6ac3d970379f1aa29629418992c94c082d 100644 (file)
@@ -130,7 +130,7 @@ func compare(aVal, bVal reflect.Value) int {
                default:
                        return -1
                }
-       case reflect.Ptr:
+       case reflect.Ptr, reflect.UnsafePointer:
                a, b := aVal.Pointer(), bVal.Pointer()
                switch {
                case a < b:
index 364c5bf6d27f45d51de62abf83a33be67944427d..5205a6413ce2a4751686fea74eafed3e8fb60c65 100644 (file)
@@ -11,6 +11,7 @@ import (
        "reflect"
        "strings"
        "testing"
+       "unsafe"
 )
 
 var compareTests = [][]reflect.Value{
@@ -32,6 +33,7 @@ var compareTests = [][]reflect.Value{
        ct(reflect.TypeOf(complex128(0+1i)), -1-1i, -1+0i, -1+1i, 0-1i, 0+0i, 0+1i, 1-1i, 1+0i, 1+1i),
        ct(reflect.TypeOf(false), false, true),
        ct(reflect.TypeOf(&ints[0]), &ints[0], &ints[1], &ints[2]),
+       ct(reflect.TypeOf(unsafe.Pointer(&ints[0])), unsafe.Pointer(&ints[0]), unsafe.Pointer(&ints[1]), unsafe.Pointer(&ints[2])),
        ct(reflect.TypeOf(chans[0]), chans[0], chans[1], chans[2]),
        ct(reflect.TypeOf(toy{}), toy{0, 1}, toy{0, 2}, toy{1, -1}, toy{1, 1}),
        ct(reflect.TypeOf([2]int{}), [2]int{1, 1}, [2]int{1, 2}, [2]int{2, 0}),
@@ -118,6 +120,10 @@ var sortTests = []sortTest{
                pointerMap(),
                "PTR0:0 PTR1:1 PTR2:2",
        },
+       {
+               unsafePointerMap(),
+               "UNSAFEPTR0:0 UNSAFEPTR1:1 UNSAFEPTR2:2",
+       },
        {
                map[toy]string{{7, 2}: "72", {7, 1}: "71", {3, 4}: "34"},
                "{3 4}:34 {7 1}:71 {7 2}:72",
@@ -159,6 +165,14 @@ func sprintKey(key reflect.Value) string {
                        }
                }
                return "PTR???"
+       case "unsafe.Pointer":
+               ptr := key.Interface().(unsafe.Pointer)
+               for i := range ints {
+                       if ptr == unsafe.Pointer(&ints[i]) {
+                               return fmt.Sprintf("UNSAFEPTR%d", i)
+                       }
+               }
+               return "UNSAFEPTR???"
        case "chan int":
                c := key.Interface().(chan int)
                for i := range chans {
@@ -185,6 +199,14 @@ func pointerMap() map[*int]string {
        return m
 }
 
+func unsafePointerMap() map[unsafe.Pointer]string {
+       m := make(map[unsafe.Pointer]string)
+       for i := 2; i >= 0; i-- {
+               m[unsafe.Pointer(&ints[i])] = fmt.Sprint(i)
+       }
+       return m
+}
+
 func chanMap() map[chan int]string {
        m := make(map[chan int]string)
        for i := 2; i >= 0; i-- {
index 2035e710159ad51066f433e26096dbe9b956d87a..7c774e9233c0b0439bb75d484ab4679aaadec026 100644 (file)
@@ -10,7 +10,7 @@ import (
        "bytes"
        "errors"
        "fmt"
-       "io/ioutil"
+       "io"
        "strings"
        "sync"
        "testing"
@@ -18,7 +18,7 @@ import (
        "github.com/gohugoio/hugo/tpl/internal/go_templates/texttemplate/parse"
 )
 
-func TestAddParseTree(t *testing.T) {
+func TestAddParseTreeHTML(t *testing.T) {
        root := Must(New("root").Parse(`{{define "a"}} {{.}} {{template "b"}} {{.}} "></a>{{end}}`))
        tree, err := parse.Parse("t", `{{define "b"}}<a href="{{end}}`, "", "", nil, nil)
        if err != nil {
@@ -174,7 +174,7 @@ func TestCloneThenParse(t *testing.T) {
                t.Error("adding a template to a clone added it to the original")
        }
        // double check that the embedded template isn't available in the original
-       err := t0.ExecuteTemplate(ioutil.Discard, "a", nil)
+       err := t0.ExecuteTemplate(io.Discard, "a", nil)
        if err == nil {
                t.Error("expected 'no such template' error")
        }
@@ -188,13 +188,13 @@ func TestFuncMapWorksAfterClone(t *testing.T) {
 
        // get the expected error output (no clone)
        uncloned := Must(New("").Funcs(funcs).Parse("{{customFunc}}"))
-       wantErr := uncloned.Execute(ioutil.Discard, nil)
+       wantErr := uncloned.Execute(io.Discard, nil)
 
        // toClone must be the same as uncloned. It has to be recreated from scratch,
        // since cloning cannot occur after execution.
        toClone := Must(New("").Funcs(funcs).Parse("{{customFunc}}"))
        cloned := Must(toClone.Clone())
-       gotErr := cloned.Execute(ioutil.Discard, nil)
+       gotErr := cloned.Execute(io.Discard, nil)
 
        if wantErr.Error() != gotErr.Error() {
                t.Errorf("clone error message mismatch want %q got %q", wantErr, gotErr)
@@ -216,7 +216,7 @@ func TestTemplateCloneExecuteRace(t *testing.T) {
                go func() {
                        defer wg.Done()
                        for i := 0; i < 100; i++ {
-                               if err := tmpl.Execute(ioutil.Discard, "data"); err != nil {
+                               if err := tmpl.Execute(io.Discard, "data"); err != nil {
                                        panic(err)
                                }
                        }
@@ -240,7 +240,7 @@ func TestCloneGrowth(t *testing.T) {
        tmpl = Must(tmpl.Clone())
        Must(tmpl.Parse(`{{define "B"}}Text{{end}}`))
        for i := 0; i < 10; i++ {
-               tmpl.Execute(ioutil.Discard, nil)
+               tmpl.Execute(io.Discard, nil)
        }
        if len(tmpl.DefinedTemplates()) > 200 {
                t.Fatalf("too many templates: %v", len(tmpl.DefinedTemplates()))
@@ -260,7 +260,7 @@ func TestCloneRedefinedName(t *testing.T) {
        for i := 0; i < 2; i++ {
                t2 := Must(t1.Clone())
                t2 = Must(t2.New(fmt.Sprintf("%d", i)).Parse(page))
-               err := t2.Execute(ioutil.Discard, nil)
+               err := t2.Execute(io.Discard, nil)
                if err != nil {
                        t.Fatal(err)
                }
index b5de701d350a55167d81c533b3d70b50ceba869e..909a24bc07901e9924bf3c5dfd2c44743aeb50ad 100644 (file)
@@ -404,11 +404,11 @@ func TestTypedContent(t *testing.T) {
 }
 
 // Test that we print using the String method. Was issue 3073.
-type stringer struct {
+type myStringer struct {
        v int
 }
 
-func (s *stringer) String() string {
+func (s *myStringer) String() string {
        return fmt.Sprintf("string=%d", s.v)
 }
 
@@ -421,7 +421,7 @@ func (s *errorer) Error() string {
 }
 
 func TestStringer(t *testing.T) {
-       s := &stringer{3}
+       s := &myStringer{3}
        b := new(bytes.Buffer)
        tmpl := Must(New("x").Parse("{{.}}"))
        if err := tmpl.Execute(b, s); err != nil {
index 53b817595373d3a4d1a2863bd909bbc82aa724cc..5f4c92a17ef5ee7142d9f5302ad2cc6c479018a4 100644 (file)
@@ -125,6 +125,8 @@ func (e *escaper) escape(c context, n parse.Node) context {
        switch n := n.(type) {
        case *parse.ActionNode:
                return e.escapeAction(c, n)
+       case *parse.CommentNode:
+               return c
        case *parse.IfNode:
                return e.escapeBranch(c, &n.BranchNode, "if")
        case *parse.ListNode:
index 075db4e13714845ee2440d0760387a9b5575cb03..c569a9391067e6456053b0ae7fb08166860c5a8c 100644 (file)
@@ -1825,7 +1825,7 @@ func TestIndirectPrint(t *testing.T) {
 }
 
 // This is a test for issue 3272.
-func TestEmptyTemplate(t *testing.T) {
+func TestEmptyTemplateHTML(t *testing.T) {
        page := Must(New("page").ParseFiles(os.DevNull))
        if err := page.ExecuteTemplate(os.Stdout, "page", "nothing"); err == nil {
                t.Fatal("expected error")
diff --git a/tpl/internal/go_templates/htmltemplate/exec_test.go b/tpl/internal/go_templates/htmltemplate/exec_test.go
new file mode 100644 (file)
index 0000000..7af5829
--- /dev/null
@@ -0,0 +1,1711 @@
+// Copyright 2011 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+// Tests for template execution, copied from text/template.
+
+// +build go1.13,!windows
+
+package template
+
+import (
+       "bytes"
+       "errors"
+       "flag"
+       "fmt"
+       "io"
+       "reflect"
+       "strings"
+       "testing"
+
+       template "github.com/gohugoio/hugo/tpl/internal/go_templates/texttemplate"
+)
+
+var debug = flag.Bool("debug", false, "show the errors produced by the tests")
+
+// T has lots of interesting pieces to use to test execution.
+type T struct {
+       // Basics
+       True        bool
+       I           int
+       U16         uint16
+       X, S        string
+       FloatZero   float64
+       ComplexZero complex128
+       // Nested structs.
+       U *U
+       // Struct with String method.
+       V0     V
+       V1, V2 *V
+       // Struct with Error method.
+       W0     W
+       W1, W2 *W
+       // Slices
+       SI      []int
+       SICap   []int
+       SIEmpty []int
+       SB      []bool
+       // Arrays
+       AI [3]int
+       // Maps
+       MSI      map[string]int
+       MSIone   map[string]int // one element, for deterministic output
+       MSIEmpty map[string]int
+       MXI      map[interface{}]int
+       MII      map[int]int
+       MI32S    map[int32]string
+       MI64S    map[int64]string
+       MUI32S   map[uint32]string
+       MUI64S   map[uint64]string
+       MI8S     map[int8]string
+       MUI8S    map[uint8]string
+       SMSI     []map[string]int
+       // Empty interfaces; used to see if we can dig inside one.
+       Empty0 interface{} // nil
+       Empty1 interface{}
+       Empty2 interface{}
+       Empty3 interface{}
+       Empty4 interface{}
+       // Non-empty interfaces.
+       NonEmptyInterface         I
+       NonEmptyInterfacePtS      *I
+       NonEmptyInterfaceNil      I
+       NonEmptyInterfaceTypedNil I
+       // Stringer.
+       Str fmt.Stringer
+       Err error
+       // Pointers
+       PI  *int
+       PS  *string
+       PSI *[]int
+       NIL *int
+       // Function (not method)
+       BinaryFunc      func(string, string) string
+       VariadicFunc    func(...string) string
+       VariadicFuncInt func(int, ...string) string
+       NilOKFunc       func(*int) bool
+       ErrFunc         func() (string, error)
+       PanicFunc       func() string
+       // Template to test evaluation of templates.
+       Tmpl *Template
+       // Unexported field; cannot be accessed by template.
+       unexported int
+}
+
+type S []string
+
+func (S) Method0() string {
+       return "M0"
+}
+
+type U struct {
+       V string
+}
+
+type V struct {
+       j int
+}
+
+func (v *V) String() string {
+       if v == nil {
+               return "nilV"
+       }
+       return fmt.Sprintf("<%d>", v.j)
+}
+
+type W struct {
+       k int
+}
+
+func (w *W) Error() string {
+       if w == nil {
+               return "nilW"
+       }
+       return fmt.Sprintf("[%d]", w.k)
+}
+
+var siVal = I(S{"a", "b"})
+
+var tVal = &T{
+       True:   true,
+       I:      17,
+       U16:    16,
+       X:      "x",
+       S:      "xyz",
+       U:      &U{"v"},
+       V0:     V{6666},
+       V1:     &V{7777}, // leave V2 as nil
+       W0:     W{888},
+       W1:     &W{999}, // leave W2 as nil
+       SI:     []int{3, 4, 5},
+       SICap:  make([]int, 5, 10),
+       AI:     [3]int{3, 4, 5},
+       SB:     []bool{true, false},
+       MSI:    map[string]int{"one": 1, "two": 2, "three": 3},
+       MSIone: map[string]int{"one": 1},
+       MXI:    map[interface{}]int{"one": 1},
+       MII:    map[int]int{1: 1},
+       MI32S:  map[int32]string{1: "one", 2: "two"},
+       MI64S:  map[int64]string{2: "i642", 3: "i643"},
+       MUI32S: map[uint32]string{2: "u322", 3: "u323"},
+       MUI64S: map[uint64]string{2: "ui642", 3: "ui643"},
+       MI8S:   map[int8]string{2: "i82", 3: "i83"},
+       MUI8S:  map[uint8]string{2: "u82", 3: "u83"},
+       SMSI: []map[string]int{
+               {"one": 1, "two": 2},
+               {"eleven": 11, "twelve": 12},
+       },
+       Empty1:                    3,
+       Empty2:                    "empty2",
+       Empty3:                    []int{7, 8},
+       Empty4:                    &U{"UinEmpty"},
+       NonEmptyInterface:         &T{X: "x"},
+       NonEmptyInterfacePtS:      &siVal,
+       NonEmptyInterfaceTypedNil: (*T)(nil),
+       Str:                       bytes.NewBuffer([]byte("foozle")),
+       Err:                       errors.New("erroozle"),
+       PI:                        newInt(23),
+       PS:                        newString("a string"),
+       PSI:                       newIntSlice(21, 22, 23),
+       BinaryFunc:                func(a, b string) string { return fmt.Sprintf("[%s=%s]", a, b) },
+       VariadicFunc:              func(s ...string) string { return fmt.Sprint("<", strings.Join(s, "+"), ">") },
+       VariadicFuncInt:           func(a int, s ...string) string { return fmt.Sprint(a, "=<", strings.Join(s, "+"), ">") },
+       NilOKFunc:                 func(s *int) bool { return s == nil },
+       ErrFunc:                   func() (string, error) { return "bla", nil },
+       PanicFunc:                 func() string { panic("test panic") },
+       Tmpl:                      Must(New("x").Parse("test template")), // "x" is the value of .X
+}
+
+var tSliceOfNil = []*T{nil}
+
+// A non-empty interface.
+type I interface {
+       Method0() string
+}
+
+var iVal I = tVal
+
+// Helpers for creation.
+func newInt(n int) *int {
+       return &n
+}
+
+func newString(s string) *string {
+       return &s
+}
+
+func newIntSlice(n ...int) *[]int {
+       p := new([]int)
+       *p = make([]int, len(n))
+       copy(*p, n)
+       return p
+}
+
+// Simple methods with and without arguments.
+func (t *T) Method0() string {
+       return "M0"
+}
+
+func (t *T) Method1(a int) int {
+       return a
+}
+
+func (t *T) Method2(a uint16, b string) string {
+       return fmt.Sprintf("Method2: %d %s", a, b)
+}
+
+func (t *T) Method3(v interface{}) string {
+       return fmt.Sprintf("Method3: %v", v)
+}
+
+func (t *T) Copy() *T {
+       n := new(T)
+       *n = *t
+       return n
+}
+
+func (t *T) MAdd(a int, b []int) []int {
+       v := make([]int, len(b))
+       for i, x := range b {
+               v[i] = x + a
+       }
+       return v
+}
+
+var myError = errors.New("my error")
+
+// MyError returns a value and an error according to its argument.
+func (t *T) MyError(error bool) (bool, error) {
+       if error {
+               return true, myError
+       }
+       return false, nil
+}
+
+// A few methods to test chaining.
+func (t *T) GetU() *U {
+       return t.U
+}
+
+func (u *U) TrueFalse(b bool) string {
+       if b {
+               return "true"
+       }
+       return ""
+}
+
+func typeOf(arg interface{}) string {
+       return fmt.Sprintf("%T", arg)
+}
+
+type execTest struct {
+       name   string
+       input  string
+       output string
+       data   interface{}
+       ok     bool
+}
+
+// bigInt and bigUint are hex string representing numbers either side
+// of the max int boundary.
+// We do it this way so the test doesn't depend on ints being 32 bits.
+var (
+       bigInt  = fmt.Sprintf("0x%x", int(1<<uint(reflect.TypeOf(0).Bits()-1)-1))
+       bigUint = fmt.Sprintf("0x%x", uint(1<<uint(reflect.TypeOf(0).Bits()-1)))
+)
+
+var execTests = []execTest{
+       // Trivial cases.
+       {"empty", "", "", nil, true},
+       {"text", "some text", "some text", nil, true},
+       {"nil action", "{{nil}}", "", nil, false},
+
+       // Ideal constants.
+       {"ideal int", "{{typeOf 3}}", "int", 0, true},
+       {"ideal float", "{{typeOf 1.0}}", "float64", 0, true},
+       {"ideal exp float", "{{typeOf 1e1}}", "float64", 0, true},
+       {"ideal complex", "{{typeOf 1i}}", "complex128", 0, true},
+       {"ideal int", "{{typeOf " + bigInt + "}}", "int", 0, true},
+       {"ideal too big", "{{typeOf " + bigUint + "}}", "", 0, false},
+       {"ideal nil without type", "{{nil}}", "", 0, false},
+
+       // Fields of structs.
+       {".X", "-{{.X}}-", "-x-", tVal, true},
+       {".U.V", "-{{.U.V}}-", "-v-", tVal, true},
+       {".unexported", "{{.unexported}}", "", tVal, false},
+
+       // Fields on maps.
+       {"map .one", "{{.MSI.one}}", "1", tVal, true},
+       {"map .two", "{{.MSI.two}}", "2", tVal, true},
+       {"map .NO", "{{.MSI.NO}}", "", tVal, true}, // NOTE: <no value> in text/template
+       {"map .one interface", "{{.MXI.one}}", "1", tVal, true},
+       {"map .WRONG args", "{{.MSI.one 1}}", "", tVal, false},
+       {"map .WRONG type", "{{.MII.one}}", "", tVal, false},
+
+       // Dots of all kinds to test basic evaluation.
+       {"dot int", "<{{.}}>", "&lt;13>", 13, true},
+       {"dot uint", "<{{.}}>", "&lt;14>", uint(14), true},
+       {"dot float", "<{{.}}>", "&lt;15.1>", 15.1, true},
+       {"dot bool", "<{{.}}>", "&lt;true>", true, true},
+       {"dot complex", "<{{.}}>", "&lt;(16.2-17i)>", 16.2 - 17i, true},
+       {"dot string", "<{{.}}>", "&lt;hello>", "hello", true},
+       {"dot slice", "<{{.}}>", "&lt;[-1 -2 -3]>", []int{-1, -2, -3}, true},
+       {"dot map", "<{{.}}>", "&lt;map[two:22]>", map[string]int{"two": 22}, true},
+       {"dot struct", "<{{.}}>", "&lt;{7 seven}>", struct {
+               a int
+               b string
+       }{7, "seven"}, true},
+
+       // Variables.
+       {"$ int", "{{$}}", "123", 123, true},
+       {"$.I", "{{$.I}}", "17", tVal, true},
+       {"$.U.V", "{{$.U.V}}", "v", tVal, true},
+       {"declare in action", "{{$x := $.U.V}}{{$x}}", "v", tVal, true},
+       {"simple assignment", "{{$x := 2}}{{$x = 3}}{{$x}}", "3", tVal, true},
+       {"nested assignment",
+               "{{$x := 2}}{{if true}}{{$x = 3}}{{end}}{{$x}}",
+               "3", tVal, true},
+       {"nested assignment changes the last declaration",
+               "{{$x := 1}}{{if true}}{{$x := 2}}{{if true}}{{$x = 3}}{{end}}{{end}}{{$x}}",
+               "1", tVal, true},
+
+       // Type with String method.
+       {"V{6666}.String()", "-{{.V0}}-", "-{6666}-", tVal, true}, //  NOTE: -<6666>- in text/template
+       {"&V{7777}.String()", "-{{.V1}}-", "-&lt;7777&gt;-", tVal, true},
+       {"(*V)(nil).String()", "-{{.V2}}-", "-nilV-", tVal, true},
+
+       // Type with Error method.
+       {"W{888}.Error()", "-{{.W0}}-", "-{888}-", tVal, true}, // NOTE: -[888] in text/template
+       {"&W{999}.Error()", "-{{.W1}}-", "-[999]-", tVal, true},
+       {"(*W)(nil).Error()", "-{{.W2}}-", "-nilW-", tVal, true},
+
+       // Pointers.
+       {"*int", "{{.PI}}", "23", tVal, true},
+       {"*string", "{{.PS}}", "a string", tVal, true},
+       {"*[]int", "{{.PSI}}", "[21 22 23]", tVal, true},
+       {"*[]int[1]", "{{index .PSI 1}}", "22", tVal, true},
+       {"NIL", "{{.NIL}}", "&lt;nil&gt;", tVal, true},
+
+       // Empty interfaces holding values.
+       {"empty nil", "{{.Empty0}}", "", tVal, true}, // NOTE: <no value> in text/template
+       {"empty with int", "{{.Empty1}}", "3", tVal, true},
+       {"empty with string", "{{.Empty2}}", "empty2", tVal, true},
+       {"empty with slice", "{{.Empty3}}", "[7 8]", tVal, true},
+       {"empty with struct", "{{.Empty4}}", "{UinEmpty}", tVal, true},
+       {"empty with struct, field", "{{.Empty4.V}}", "UinEmpty", tVal, true},
+
+       // Edge cases with <no value> with an interface value
+       {"field on interface", "{{.foo}}", "", nil, true},                  // NOTE: <no value> in text/template
+       {"field on parenthesized interface", "{{(.).foo}}", "", nil, true}, // NOTE: <no value> in text/template
+
+       // Issue 31810: Parenthesized first element of pipeline with arguments.
+       // See also TestIssue31810.
+       {"unparenthesized non-function", "{{1 2}}", "", nil, false},
+       {"parenthesized non-function", "{{(1) 2}}", "", nil, false},
+       {"parenthesized non-function with no args", "{{(1)}}", "1", nil, true}, // This is fine.
+
+       // Method calls.
+       {".Method0", "-{{.Method0}}-", "-M0-", tVal, true},
+       {".Method1(1234)", "-{{.Method1 1234}}-", "-1234-", tVal, true},
+       {".Method1(.I)", "-{{.Method1 .I}}-", "-17-", tVal, true},
+       {".Method2(3, .X)", "-{{.Method2 3 .X}}-", "-Method2: 3 x-", tVal, true},
+       {".Method2(.U16, `str`)", "-{{.Method2 .U16 `str`}}-", "-Method2: 16 str-", tVal, true},
+       {".Method2(.U16, $x)", "{{if $x := .X}}-{{.Method2 .U16 $x}}{{end}}-", "-Method2: 16 x-", tVal, true},
+       {".Method3(nil constant)", "-{{.Method3 nil}}-", "-Method3: &lt;nil&gt;-", tVal, true},
+       {".Method3(nil value)", "-{{.Method3 .MXI.unset}}-", "-Method3: &lt;nil&gt;-", tVal, true},
+       {"method on var", "{{if $x := .}}-{{$x.Method2 .U16 $x.X}}{{end}}-", "-Method2: 16 x-", tVal, true},
+       {"method on chained var",
+               "{{range .MSIone}}{{if $.U.TrueFalse $.True}}{{$.U.TrueFalse $.True}}{{else}}WRONG{{end}}{{end}}",
+               "true", tVal, true},
+       {"chained method",
+               "{{range .MSIone}}{{if $.GetU.TrueFalse $.True}}{{$.U.TrueFalse $.True}}{{else}}WRONG{{end}}{{end}}",
+               "true", tVal, true},
+       {"chained method on variable",
+               "{{with $x := .}}{{with .SI}}{{$.GetU.TrueFalse $.True}}{{end}}{{end}}",
+               "true", tVal, true},
+       {".NilOKFunc not nil", "{{call .NilOKFunc .PI}}", "false", tVal, true},
+       {".NilOKFunc nil", "{{call .NilOKFunc nil}}", "true", tVal, true},
+       {"method on nil value from slice", "-{{range .}}{{.Method1 1234}}{{end}}-", "-1234-", tSliceOfNil, true},
+       {"method on typed nil interface value", "{{.NonEmptyInterfaceTypedNil.Method0}}", "M0", tVal, true},
+
+       // Function call builtin.
+       {".BinaryFunc", "{{call .BinaryFunc `1` `2`}}", "[1=2]", tVal, true},
+       {".VariadicFunc0", "{{call .VariadicFunc}}", "&lt;&gt;", tVal, true},
+       {".VariadicFunc2", "{{call .VariadicFunc `he` `llo`}}", "&lt;he&#43;llo&gt;", tVal, true},
+       {".VariadicFuncInt", "{{call .VariadicFuncInt 33 `he` `llo`}}", "33=&lt;he&#43;llo&gt;", tVal, true},
+       {"if .BinaryFunc call", "{{ if .BinaryFunc}}{{call .BinaryFunc `1` `2`}}{{end}}", "[1=2]", tVal, true},
+       {"if not .BinaryFunc call", "{{ if not .BinaryFunc}}{{call .BinaryFunc `1` `2`}}{{else}}No{{end}}", "No", tVal, true},
+       {"Interface Call", `{{stringer .S}}`, "foozle", map[string]interface{}{"S": bytes.NewBufferString("foozle")}, true},
+       {".ErrFunc", "{{call .ErrFunc}}", "bla", tVal, true},
+       {"call nil", "{{call nil}}", "", tVal, false},
+
+       // Erroneous function calls (check args).
+       {".BinaryFuncTooFew", "{{call .BinaryFunc `1`}}", "", tVal, false},
+       {".BinaryFuncTooMany", "{{call .BinaryFunc `1` `2` `3`}}", "", tVal, false},
+       {".BinaryFuncBad0", "{{call .BinaryFunc 1 3}}", "", tVal, false},
+       {".BinaryFuncBad1", "{{call .BinaryFunc `1` 3}}", "", tVal, false},
+       {".VariadicFuncBad0", "{{call .VariadicFunc 3}}", "", tVal, false},
+       {".VariadicFuncIntBad0", "{{call .VariadicFuncInt}}", "", tVal, false},
+       {".VariadicFuncIntBad`", "{{call .VariadicFuncInt `x`}}", "", tVal, false},
+       {".VariadicFuncNilBad", "{{call .VariadicFunc nil}}", "", tVal, false},
+
+       // Pipelines.
+       {"pipeline", "-{{.Method0 | .Method2 .U16}}-", "-Method2: 16 M0-", tVal, true},
+       {"pipeline func", "-{{call .VariadicFunc `llo` | call .VariadicFunc `he` }}-", "-&lt;he&#43;&lt;llo&gt;&gt;-", tVal, true},
+
+       // Nil values aren't missing arguments.
+       {"nil pipeline", "{{ .Empty0 | call .NilOKFunc }}", "true", tVal, true},
+       {"nil call arg", "{{ call .NilOKFunc .Empty0 }}", "true", tVal, true},
+       {"bad nil pipeline", "{{ .Empty0 | .VariadicFunc }}", "", tVal, false},
+
+       // Parenthesized expressions
+       {"parens in pipeline", "{{printf `%d %d %d` (1) (2 | add 3) (add 4 (add 5 6))}}", "1 5 15", tVal, true},
+
+       // Parenthesized expressions with field accesses
+       {"parens: $ in paren", "{{($).X}}", "x", tVal, true},
+       {"parens: $.GetU in paren", "{{($.GetU).V}}", "v", tVal, true},
+       {"parens: $ in paren in pipe", "{{($ | echo).X}}", "x", tVal, true},
+       {"parens: spaces and args", `{{(makemap "up" "down" "left" "right").left}}`, "right", tVal, true},
+
+       // If.
+       {"if true", "{{if true}}TRUE{{end}}", "TRUE", tVal, true},
+       {"if false", "{{if false}}TRUE{{else}}FALSE{{end}}", "FALSE", tVal, true},
+       {"if nil", "{{if nil}}TRUE{{end}}", "", tVal, false},
+       {"if on typed nil interface value", "{{if .NonEmptyInterfaceTypedNil}}TRUE{{ end }}", "", tVal, true},
+       {"if 1", "{{if 1}}NON-ZERO{{else}}ZERO{{end}}", "NON-ZERO", tVal, true},
+       {"if 0", "{{if 0}}NON-ZERO{{else}}ZERO{{end}}", "ZERO", tVal, true},
+       {"if 1.5", "{{if 1.5}}NON-ZERO{{else}}ZERO{{end}}", "NON-ZERO", tVal, true},
+       {"if 0.0", "{{if .FloatZero}}NON-ZERO{{else}}ZERO{{end}}", "ZERO", tVal, true},
+       {"if 1.5i", "{{if 1.5i}}NON-ZERO{{else}}ZERO{{end}}", "NON-ZERO", tVal, true},
+       {"if 0.0i", "{{if .ComplexZero}}NON-ZERO{{else}}ZERO{{end}}", "ZERO", tVal, true},
+       {"if emptystring", "{{if ``}}NON-EMPTY{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"if string", "{{if `notempty`}}NON-EMPTY{{else}}EMPTY{{end}}", "NON-EMPTY", tVal, true},
+       {"if emptyslice", "{{if .SIEmpty}}NON-EMPTY{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"if slice", "{{if .SI}}NON-EMPTY{{else}}EMPTY{{end}}", "NON-EMPTY", tVal, true},
+       {"if emptymap", "{{if .MSIEmpty}}NON-EMPTY{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"if map", "{{if .MSI}}NON-EMPTY{{else}}EMPTY{{end}}", "NON-EMPTY", tVal, true},
+       {"if map unset", "{{if .MXI.none}}NON-ZERO{{else}}ZERO{{end}}", "ZERO", tVal, true},
+       {"if map not unset", "{{if not .MXI.none}}ZERO{{else}}NON-ZERO{{end}}", "ZERO", tVal, true},
+       {"if $x with $y int", "{{if $x := true}}{{with $y := .I}}{{$x}},{{$y}}{{end}}{{end}}", "true,17", tVal, true},
+       {"if $x with $x int", "{{if $x := true}}{{with $x := .I}}{{$x}},{{end}}{{$x}}{{end}}", "17,true", tVal, true},
+       {"if else if", "{{if false}}FALSE{{else if true}}TRUE{{end}}", "TRUE", tVal, true},
+       {"if else chain", "{{if eq 1 3}}1{{else if eq 2 3}}2{{else if eq 3 3}}3{{end}}", "3", tVal, true},
+
+       // Print etc.
+       {"print", `{{print "hello, print"}}`, "hello, print", tVal, true},
+       {"print 123", `{{print 1 2 3}}`, "1 2 3", tVal, true},
+       {"print nil", `{{print nil}}`, "&lt;nil&gt;", tVal, true},
+       {"println", `{{println 1 2 3}}`, "1 2 3\n", tVal, true},
+       {"printf int", `{{printf "%04x" 127}}`, "007f", tVal, true},
+       {"printf float", `{{printf "%g" 3.5}}`, "3.5", tVal, true},
+       {"printf complex", `{{printf "%g" 1+7i}}`, "(1&#43;7i)", tVal, true},
+       {"printf string", `{{printf "%s" "hello"}}`, "hello", tVal, true},
+       {"printf function", `{{printf "%#q" zeroArgs}}`, "`zeroArgs`", tVal, true},
+       {"printf field", `{{printf "%s" .U.V}}`, "v", tVal, true},
+       {"printf method", `{{printf "%s" .Method0}}`, "M0", tVal, true},
+       {"printf dot", `{{with .I}}{{printf "%d" .}}{{end}}`, "17", tVal, true},
+       {"printf var", `{{with $x := .I}}{{printf "%d" $x}}{{end}}`, "17", tVal, true},
+       {"printf lots", `{{printf "%d %s %g %s" 127 "hello" 7-3i .Method0}}`, "127 hello (7-3i) M0", tVal, true},
+
+       // HTML.
+       {"html", `{{html "<script>alert(\"XSS\");</script>"}}`,
+               "&lt;script&gt;alert(&#34;XSS&#34;);&lt;/script&gt;", nil, true},
+       {"html pipeline", `{{printf "<script>alert(\"XSS\");</script>" | html}}`,
+               "&lt;script&gt;alert(&#34;XSS&#34;);&lt;/script&gt;", nil, true},
+       {"html", `{{html .PS}}`, "a string", tVal, true},
+       {"html typed nil", `{{html .NIL}}`, "&lt;nil&gt;", tVal, true},
+       {"html untyped nil", `{{html .Empty0}}`, "&lt;nil&gt;", tVal, true}, // NOTE: "&lt;no value&gt;" in text/template
+
+       // JavaScript.
+       {"js", `{{js .}}`, `It\&#39;d be nice.`, `It'd be nice.`, true},
+
+       // URL query.
+       {"urlquery", `{{"http://www.example.org/"|urlquery}}`, "http%3A%2F%2Fwww.example.org%2F", nil, true},
+
+       // Booleans
+       {"not", "{{not true}} {{not false}}", "false true", nil, true},
+       {"and", "{{and false 0}} {{and 1 0}} {{and 0 true}} {{and 1 1}}", "false 0 0 1", nil, true},
+       {"or", "{{or 0 0}} {{or 1 0}} {{or 0 true}} {{or 1 1}}", "0 1 true 1", nil, true},
+       {"boolean if", "{{if and true 1 `hi`}}TRUE{{else}}FALSE{{end}}", "TRUE", tVal, true},
+       {"boolean if not", "{{if and true 1 `hi` | not}}TRUE{{else}}FALSE{{end}}", "FALSE", nil, true},
+
+       // Indexing.
+       {"slice[0]", "{{index .SI 0}}", "3", tVal, true},
+       {"slice[1]", "{{index .SI 1}}", "4", tVal, true},
+       {"slice[HUGE]", "{{index .SI 10}}", "", tVal, false},
+       {"slice[WRONG]", "{{index .SI `hello`}}", "", tVal, false},
+       {"slice[nil]", "{{index .SI nil}}", "", tVal, false},
+       {"map[one]", "{{index .MSI `one`}}", "1", tVal, true},
+       {"map[two]", "{{index .MSI `two`}}", "2", tVal, true},
+       {"map[NO]", "{{index .MSI `XXX`}}", "0", tVal, true},
+       {"map[nil]", "{{index .MSI nil}}", "", tVal, false},
+       {"map[``]", "{{index .MSI ``}}", "0", tVal, true},
+       {"map[WRONG]", "{{index .MSI 10}}", "", tVal, false},
+       {"double index", "{{index .SMSI 1 `eleven`}}", "11", tVal, true},
+       {"nil[1]", "{{index nil 1}}", "", tVal, false},
+       {"map MI64S", "{{index .MI64S 2}}", "i642", tVal, true},
+       {"map MI32S", "{{index .MI32S 2}}", "two", tVal, true},
+       {"map MUI64S", "{{index .MUI64S 3}}", "ui643", tVal, true},
+       {"map MI8S", "{{index .MI8S 3}}", "i83", tVal, true},
+       {"map MUI8S", "{{index .MUI8S 2}}", "u82", tVal, true},
+       {"index of an interface field", "{{index .Empty3 0}}", "7", tVal, true},
+
+       // Slicing.
+       {"slice[:]", "{{slice .SI}}", "[3 4 5]", tVal, true},
+       {"slice[1:]", "{{slice .SI 1}}", "[4 5]", tVal, true},
+       {"slice[1:2]", "{{slice .SI 1 2}}", "[4]", tVal, true},
+       {"slice[-1:]", "{{slice .SI -1}}", "", tVal, false},
+       {"slice[1:-2]", "{{slice .SI 1 -2}}", "", tVal, false},
+       {"slice[1:2:-1]", "{{slice .SI 1 2 -1}}", "", tVal, false},
+       {"slice[2:1]", "{{slice .SI 2 1}}", "", tVal, false},
+       {"slice[2:2:1]", "{{slice .SI 2 2 1}}", "", tVal, false},
+       {"out of range", "{{slice .SI 4 5}}", "", tVal, false},
+       {"out of range", "{{slice .SI 2 2 5}}", "", tVal, false},
+       {"len(s) < indexes < cap(s)", "{{slice .SICap 6 10}}", "[0 0 0 0]", tVal, true},
+       {"len(s) < indexes < cap(s)", "{{slice .SICap 6 10 10}}", "[0 0 0 0]", tVal, true},
+       {"indexes > cap(s)", "{{slice .SICap 10 11}}", "", tVal, false},
+       {"indexes > cap(s)", "{{slice .SICap 6 10 11}}", "", tVal, false},
+       {"array[:]", "{{slice .AI}}", "[3 4 5]", tVal, true},
+       {"array[1:]", "{{slice .AI 1}}", "[4 5]", tVal, true},
+       {"array[1:2]", "{{slice .AI 1 2}}", "[4]", tVal, true},
+       {"string[:]", "{{slice .S}}", "xyz", tVal, true},
+       {"string[0:1]", "{{slice .S 0 1}}", "x", tVal, true},
+       {"string[1:]", "{{slice .S 1}}", "yz", tVal, true},
+       {"string[1:2]", "{{slice .S 1 2}}", "y", tVal, true},
+       {"out of range", "{{slice .S 1 5}}", "", tVal, false},
+       {"3-index slice of string", "{{slice .S 1 2 2}}", "", tVal, false},
+       {"slice of an interface field", "{{slice .Empty3 0 1}}", "[7]", tVal, true},
+
+       // Len.
+       {"slice", "{{len .SI}}", "3", tVal, true},
+       {"map", "{{len .MSI }}", "3", tVal, true},
+       {"len of int", "{{len 3}}", "", tVal, false},
+       {"len of nothing", "{{len .Empty0}}", "", tVal, false},
+       {"len of an interface field", "{{len .Empty3}}", "2", tVal, true},
+
+       // With.
+       {"with true", "{{with true}}{{.}}{{end}}", "true", tVal, true},
+       {"with false", "{{with false}}{{.}}{{else}}FALSE{{end}}", "FALSE", tVal, true},
+       {"with 1", "{{with 1}}{{.}}{{else}}ZERO{{end}}", "1", tVal, true},
+       {"with 0", "{{with 0}}{{.}}{{else}}ZERO{{end}}", "ZERO", tVal, true},
+       {"with 1.5", "{{with 1.5}}{{.}}{{else}}ZERO{{end}}", "1.5", tVal, true},
+       {"with 0.0", "{{with .FloatZero}}{{.}}{{else}}ZERO{{end}}", "ZERO", tVal, true},
+       {"with 1.5i", "{{with 1.5i}}{{.}}{{else}}ZERO{{end}}", "(0&#43;1.5i)", tVal, true},
+       {"with 0.0i", "{{with .ComplexZero}}{{.}}{{else}}ZERO{{end}}", "ZERO", tVal, true},
+       {"with emptystring", "{{with ``}}{{.}}{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"with string", "{{with `notempty`}}{{.}}{{else}}EMPTY{{end}}", "notempty", tVal, true},
+       {"with emptyslice", "{{with .SIEmpty}}{{.}}{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"with slice", "{{with .SI}}{{.}}{{else}}EMPTY{{end}}", "[3 4 5]", tVal, true},
+       {"with emptymap", "{{with .MSIEmpty}}{{.}}{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"with map", "{{with .MSIone}}{{.}}{{else}}EMPTY{{end}}", "map[one:1]", tVal, true},
+       {"with empty interface, struct field", "{{with .Empty4}}{{.V}}{{end}}", "UinEmpty", tVal, true},
+       {"with $x int", "{{with $x := .I}}{{$x}}{{end}}", "17", tVal, true},
+       {"with $x struct.U.V", "{{with $x := $}}{{$x.U.V}}{{end}}", "v", tVal, true},
+       {"with variable and action", "{{with $x := $}}{{$y := $.U.V}}{{$y}}{{end}}", "v", tVal, true},
+       {"with on typed nil interface value", "{{with .NonEmptyInterfaceTypedNil}}TRUE{{ end }}", "", tVal, true},
+
+       // Range.
+       {"range []int", "{{range .SI}}-{{.}}-{{end}}", "-3--4--5-", tVal, true},
+       {"range empty no else", "{{range .SIEmpty}}-{{.}}-{{end}}", "", tVal, true},
+       {"range []int else", "{{range .SI}}-{{.}}-{{else}}EMPTY{{end}}", "-3--4--5-", tVal, true},
+       {"range empty else", "{{range .SIEmpty}}-{{.}}-{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"range []bool", "{{range .SB}}-{{.}}-{{end}}", "-true--false-", tVal, true},
+       {"range []int method", "{{range .SI | .MAdd .I}}-{{.}}-{{end}}", "-20--21--22-", tVal, true},
+       {"range map", "{{range .MSI}}-{{.}}-{{end}}", "-1--3--2-", tVal, true},
+       {"range empty map no else", "{{range .MSIEmpty}}-{{.}}-{{end}}", "", tVal, true},
+       {"range map else", "{{range .MSI}}-{{.}}-{{else}}EMPTY{{end}}", "-1--3--2-", tVal, true},
+       {"range empty map else", "{{range .MSIEmpty}}-{{.}}-{{else}}EMPTY{{end}}", "EMPTY", tVal, true},
+       {"range empty interface", "{{range .Empty3}}-{{.}}-{{else}}EMPTY{{end}}", "-7--8-", tVal, true},
+       {"range empty nil", "{{range .Empty0}}-{{.}}-{{end}}", "", tVal, true},
+       {"range $x SI", "{{range $x := .SI}}<{{$x}}>{{end}}", "&lt;3>&lt;4>&lt;5>", tVal, true},
+       {"range $x $y SI", "{{range $x, $y := .SI}}<{{$x}}={{$y}}>{{end}}", "&lt;0=3>&lt;1=4>&lt;2=5>", tVal, true},
+       {"range $x MSIone", "{{range $x := .MSIone}}<{{$x}}>{{end}}", "&lt;1>", tVal, true},
+       {"range $x $y MSIone", "{{range $x, $y := .MSIone}}<{{$x}}={{$y}}>{{end}}", "&lt;one=1>", tVal, true},
+       {"range $x PSI", "{{range $x := .PSI}}<{{$x}}>{{end}}", "&lt;21>&lt;22>&lt;23>", tVal, true},
+       {"declare in range", "{{range $x := .PSI}}<{{$foo:=$x}}{{$x}}>{{end}}", "&lt;21>&lt;22>&lt;23>", tVal, true},
+       {"range count", `{{range $i, $x := count 5}}[{{$i}}]{{$x}}{{end}}`, "[0]a[1]b[2]c[3]d[4]e", tVal, true},
+       {"range nil count", `{{range $i, $x := count 0}}{{else}}empty{{end}}`, "empty", tVal, true},
+
+       // Cute examples.
+       {"or as if true", `{{or .SI "slice is empty"}}`, "[3 4 5]", tVal, true},
+       {"or as if false", `{{or .SIEmpty "slice is empty"}}`, "slice is empty", tVal, true},
+
+       // Error handling.
+       {"error method, error", "{{.MyError true}}", "", tVal, false},
+       {"error method, no error", "{{.MyError false}}", "false", tVal, true},
+
+       // Numbers
+       {"decimal", "{{print 1234}}", "1234", tVal, true},
+       {"decimal _", "{{print 12_34}}", "1234", tVal, true},
+       {"binary", "{{print 0b101}}", "5", tVal, true},
+       {"binary _", "{{print 0b_1_0_1}}", "5", tVal, true},
+       {"BINARY", "{{print 0B101}}", "5", tVal, true},
+       {"octal0", "{{print 0377}}", "255", tVal, true},
+       {"octal", "{{print 0o377}}", "255", tVal, true},
+       {"octal _", "{{print 0o_3_7_7}}", "255", tVal, true},
+       {"OCTAL", "{{print 0O377}}", "255", tVal, true},
+       {"hex", "{{print 0x123}}", "291", tVal, true},
+       {"hex _", "{{print 0x1_23}}", "291", tVal, true},
+       {"HEX", "{{print 0X123ABC}}", "1194684", tVal, true},
+       {"float", "{{print 123.4}}", "123.4", tVal, true},
+       {"float _", "{{print 0_0_1_2_3.4}}", "123.4", tVal, true},
+       {"hex float", "{{print +0x1.ep+2}}", "7.5", tVal, true},
+       {"hex float _", "{{print +0x_1.e_0p+0_2}}", "7.5", tVal, true},
+       {"HEX float", "{{print +0X1.EP+2}}", "7.5", tVal, true},
+       {"print multi", "{{print 1_2_3_4 7.5_00_00_00}}", "1234 7.5", tVal, true},
+       {"print multi2", "{{print 1234 0x0_1.e_0p+02}}", "1234 7.5", tVal, true},
+
+       // Fixed bugs.
+       // Must separate dot and receiver; otherwise args are evaluated with dot set to variable.
+       {"bug0", "{{range .MSIone}}{{if $.Method1 .}}X{{end}}{{end}}", "X", tVal, true},
+       // Do not loop endlessly in indirect for non-empty interfaces.
+       // The bug appears with *interface only; looped forever.
+       {"bug1", "{{.Method0}}", "M0", &iVal, true},
+       // Was taking address of interface field, so method set was empty.
+       {"bug2", "{{$.NonEmptyInterface.Method0}}", "M0", tVal, true},
+       // Struct values were not legal in with - mere oversight.
+       {"bug3", "{{with $}}{{.Method0}}{{end}}", "M0", tVal, true},
+       // Nil interface values in if.
+       {"bug4", "{{if .Empty0}}non-nil{{else}}nil{{end}}", "nil", tVal, true},
+       // Stringer.
+       {"bug5", "{{.Str}}", "foozle", tVal, true},
+       {"bug5a", "{{.Err}}", "erroozle", tVal, true},
+       // Args need to be indirected and dereferenced sometimes.
+       {"bug6a", "{{vfunc .V0 .V1}}", "vfunc", tVal, true},
+       {"bug6b", "{{vfunc .V0 .V0}}", "vfunc", tVal, true},
+       {"bug6c", "{{vfunc .V1 .V0}}", "vfunc", tVal, true},
+       {"bug6d", "{{vfunc .V1 .V1}}", "vfunc", tVal, true},
+       // Legal parse but illegal execution: non-function should have no arguments.
+       {"bug7a", "{{3 2}}", "", tVal, false},
+       {"bug7b", "{{$x := 1}}{{$x 2}}", "", tVal, false},
+       {"bug7c", "{{$x := 1}}{{3 | $x}}", "", tVal, false},
+       // Pipelined arg was not being type-checked.
+       {"bug8a", "{{3|oneArg}}", "", tVal, false},
+       {"bug8b", "{{4|dddArg 3}}", "", tVal, false},
+       // A bug was introduced that broke map lookups for lower-case names.
+       {"bug9", "{{.cause}}", "neglect", map[string]string{"cause": "neglect"}, true},
+       // Field chain starting with function did not work.
+       {"bug10", "{{mapOfThree.three}}-{{(mapOfThree).three}}", "3-3", 0, true},
+       // Dereferencing nil pointer while evaluating function arguments should not panic. Issue 7333.
+       {"bug11", "{{valueString .PS}}", "", T{}, false},
+       // 0xef gave constant type float64. Issue 8622.
+       {"bug12xe", "{{printf `%T` 0xef}}", "int", T{}, true},
+       {"bug12xE", "{{printf `%T` 0xEE}}", "int", T{}, true},
+       {"bug12Xe", "{{printf `%T` 0Xef}}", "int", T{}, true},
+       {"bug12XE", "{{printf `%T` 0XEE}}", "int", T{}, true},
+       // Chained nodes did not work as arguments. Issue 8473.
+       {"bug13", "{{print (.Copy).I}}", "17", tVal, true},
+       // Didn't protect against nil or literal values in field chains.
+       {"bug14a", "{{(nil).True}}", "", tVal, false},
+       {"bug14b", "{{$x := nil}}{{$x.anything}}", "", tVal, false},
+       {"bug14c", `{{$x := (1.0)}}{{$y := ("hello")}}{{$x.anything}}{{$y.true}}`, "", tVal, false},
+       // Didn't call validateType on function results. Issue 10800.
+       {"bug15", "{{valueString returnInt}}", "", tVal, false},
+       // Variadic function corner cases. Issue 10946.
+       {"bug16a", "{{true|printf}}", "", tVal, false},
+       {"bug16b", "{{1|printf}}", "", tVal, false},
+       {"bug16c", "{{1.1|printf}}", "", tVal, false},
+       {"bug16d", "{{'x'|printf}}", "", tVal, false},
+       {"bug16e", "{{0i|printf}}", "", tVal, false},
+       {"bug16f", "{{true|twoArgs \"xxx\"}}", "", tVal, false},
+       {"bug16g", "{{\"aaa\" |twoArgs \"bbb\"}}", "twoArgs=bbbaaa", tVal, true},
+       {"bug16h", "{{1|oneArg}}", "", tVal, false},
+       {"bug16i", "{{\"aaa\"|oneArg}}", "oneArg=aaa", tVal, true},
+       {"bug16j", "{{1+2i|printf \"%v\"}}", "(1&#43;2i)", tVal, true},
+       {"bug16k", "{{\"aaa\"|printf }}", "aaa", tVal, true},
+       {"bug17a", "{{.NonEmptyInterface.X}}", "x", tVal, true},
+       {"bug17b", "-{{.NonEmptyInterface.Method1 1234}}-", "-1234-", tVal, true},
+       {"bug17c", "{{len .NonEmptyInterfacePtS}}", "2", tVal, true},
+       {"bug17d", "{{index .NonEmptyInterfacePtS 0}}", "a", tVal, true},
+       {"bug17e", "{{range .NonEmptyInterfacePtS}}-{{.}}-{{end}}", "-a--b-", tVal, true},
+
+       // More variadic function corner cases. Some runes would get evaluated
+       // as constant floats instead of ints. Issue 34483.
+       {"bug18a", "{{eq . '.'}}", "true", '.', true},
+       {"bug18b", "{{eq . 'e'}}", "true", 'e', true},
+       {"bug18c", "{{eq . 'P'}}", "true", 'P', true},
+}
+
+func zeroArgs() string {
+       return "zeroArgs"
+}
+
+func oneArg(a string) string {
+       return "oneArg=" + a
+}
+
+func twoArgs(a, b string) string {
+       return "twoArgs=" + a + b
+}
+
+func dddArg(a int, b ...string) string {
+       return fmt.Sprintln(a, b)
+}
+
+// count returns a channel that will deliver n sequential 1-letter strings starting at "a"
+func count(n int) chan string {
+       if n == 0 {
+               return nil
+       }
+       c := make(chan string)
+       go func() {
+               for i := 0; i < n; i++ {
+                       c <- "abcdefghijklmnop"[i : i+1]
+               }
+               close(c)
+       }()
+       return c
+}
+
+// vfunc takes a *V and a V
+func vfunc(V, *V) string {
+       return "vfunc"
+}
+
+// valueString takes a string, not a pointer.
+func valueString(v string) string {
+       return "value is ignored"
+}
+
+// returnInt returns an int
+func returnInt() int {
+       return 7
+}
+
+func add(args ...int) int {
+       sum := 0
+       for _, x := range args {
+               sum += x
+       }
+       return sum
+}
+
+func echo(arg interface{}) interface{} {
+       return arg
+}
+
+func makemap(arg ...string) map[string]string {
+       if len(arg)%2 != 0 {
+               panic("bad makemap")
+       }
+       m := make(map[string]string)
+       for i := 0; i < len(arg); i += 2 {
+               m[arg[i]] = arg[i+1]
+       }
+       return m
+}
+
+func stringer(s fmt.Stringer) string {
+       return s.String()
+}
+
+func mapOfThree() interface{} {
+       return map[string]int{"three": 3}
+}
+
+func testExecute(execTests []execTest, template *Template, t *testing.T) {
+       b := new(bytes.Buffer)
+       funcs := FuncMap{
+               "add":         add,
+               "count":       count,
+               "dddArg":      dddArg,
+               "echo":        echo,
+               "makemap":     makemap,
+               "mapOfThree":  mapOfThree,
+               "oneArg":      oneArg,
+               "returnInt":   returnInt,
+               "stringer":    stringer,
+               "twoArgs":     twoArgs,
+               "typeOf":      typeOf,
+               "valueString": valueString,
+               "vfunc":       vfunc,
+               "zeroArgs":    zeroArgs,
+       }
+       for _, test := range execTests {
+               var tmpl *Template
+               var err error
+               if template == nil {
+                       tmpl, err = New(test.name).Funcs(funcs).Parse(test.input)
+               } else {
+                       tmpl, err = template.Clone()
+                       if err != nil {
+                               t.Errorf("%s: clone error: %s", test.name, err)
+                               continue
+                       }
+                       tmpl, err = tmpl.New(test.name).Funcs(funcs).Parse(test.input)
+               }
+               if err != nil {
+                       t.Errorf("%s: parse error: %s", test.name, err)
+                       continue
+               }
+               b.Reset()
+               err = tmpl.Execute(b, test.data)
+               switch {
+               case !test.ok && err == nil:
+                       t.Errorf("%s: expected error; got none", test.name)
+                       continue
+               case test.ok && err != nil:
+                       t.Errorf("%s: unexpected execute error: %s", test.name, err)
+                       continue
+               case !test.ok && err != nil:
+                       // expected error, got one
+                       if *debug {
+                               fmt.Printf("%s: %s\n\t%s\n", test.name, test.input, err)
+                       }
+               }
+               result := b.String()
+               if result != test.output {
+                       t.Errorf("%s: expected\n\t%q\ngot\n\t%q", test.name, test.output, result)
+               }
+       }
+}
+
+func TestExecute(t *testing.T) {
+       testExecute(execTests, nil, t)
+}
+
+var delimPairs = []string{
+       "", "", // default
+       "{{", "}}", // same as default
+       "|", "|", // same
+       "(日)", "(本)", // peculiar
+}
+
+func TestDelims(t *testing.T) {
+       const hello = "Hello, world"
+       var value = struct{ Str string }{hello}
+       for i := 0; i < len(delimPairs); i += 2 {
+               text := ".Str"
+               left := delimPairs[i+0]
+               trueLeft := left
+               right := delimPairs[i+1]
+               trueRight := right
+               if left == "" { // default case
+                       trueLeft = "{{"
+               }
+               if right == "" { // default case
+                       trueRight = "}}"
+               }
+               text = trueLeft + text + trueRight
+               // Now add a comment
+               text += trueLeft + "/*comment*/" + trueRight
+               // Now add  an action containing a string.
+               text += trueLeft + `"` + trueLeft + `"` + trueRight
+               // At this point text looks like `{{.Str}}{{/*comment*/}}{{"{{"}}`.
+               tmpl, err := New("delims").Delims(left, right).Parse(text)
+               if err != nil {
+                       t.Fatalf("delim %q text %q parse err %s", left, text, err)
+               }
+               var b = new(bytes.Buffer)
+               err = tmpl.Execute(b, value)
+               if err != nil {
+                       t.Fatalf("delim %q exec err %s", left, err)
+               }
+               if b.String() != hello+trueLeft {
+                       t.Errorf("expected %q got %q", hello+trueLeft, b.String())
+               }
+       }
+}
+
+// Check that an error from a method flows back to the top.
+func TestExecuteError(t *testing.T) {
+       b := new(bytes.Buffer)
+       tmpl := New("error")
+       _, err := tmpl.Parse("{{.MyError true}}")
+       if err != nil {
+               t.Fatalf("parse error: %s", err)
+       }
+       err = tmpl.Execute(b, tVal)
+       if err == nil {
+               t.Errorf("expected error; got none")
+       } else if !strings.Contains(err.Error(), myError.Error()) {
+               if *debug {
+                       fmt.Printf("test execute error: %s\n", err)
+               }
+               t.Errorf("expected myError; got %s", err)
+       }
+}
+
+const execErrorText = `line 1
+line 2
+line 3
+{{template "one" .}}
+{{define "one"}}{{template "two" .}}{{end}}
+{{define "two"}}{{template "three" .}}{{end}}
+{{define "three"}}{{index "hi" $}}{{end}}`
+
+// Check that an error from a nested template contains all the relevant information.
+func TestExecError(t *testing.T) {
+       tmpl, err := New("top").Parse(execErrorText)
+       if err != nil {
+               t.Fatal("parse error:", err)
+       }
+       var b bytes.Buffer
+       err = tmpl.Execute(&b, 5) // 5 is out of range indexing "hi"
+       if err == nil {
+               t.Fatal("expected error")
+       }
+       const want = `template: top:7:20: executing "three" at <index "hi" $>: error calling index: index out of range: 5`
+       got := err.Error()
+       if got != want {
+               t.Errorf("expected\n%q\ngot\n%q", want, got)
+       }
+}
+
+func TestJSEscaping(t *testing.T) {
+       testCases := []struct {
+               in, exp string
+       }{
+               {`a`, `a`},
+               {`'foo`, `\'foo`},
+               {`Go "jump" \`, `Go \"jump\" \\`},
+               {`Yukihiro says "今日は世界"`, `Yukihiro says \"今日は世界\"`},
+               {"unprintable \uFDFF", `unprintable \uFDFF`},
+               {`<html>`, `\u003Chtml\u003E`},
+               {`no = in attributes`, `no \u003D in attributes`},
+               {`&#x27; does not become HTML entity`, `\u0026#x27; does not become HTML entity`},
+       }
+       for _, tc := range testCases {
+               s := JSEscapeString(tc.in)
+               if s != tc.exp {
+                       t.Errorf("JS escaping [%s] got [%s] want [%s]", tc.in, s, tc.exp)
+               }
+       }
+}
+
+// A nice example: walk a binary tree.
+
+type Tree struct {
+       Val         int
+       Left, Right *Tree
+}
+
+// Use different delimiters to test Set.Delims.
+// Also test the trimming of leading and trailing spaces.
+const treeTemplate = `
+       (- define "tree" -)
+       [
+               (- .Val -)
+               (- with .Left -)
+                       (template "tree" . -)
+               (- end -)
+               (- with .Right -)
+                       (- template "tree" . -)
+               (- end -)
+       ]
+       (- end -)
+`
+
+func TestTree(t *testing.T) {
+       var tree = &Tree{
+               1,
+               &Tree{
+                       2, &Tree{
+                               3,
+                               &Tree{
+                                       4, nil, nil,
+                               },
+                               nil,
+                       },
+                       &Tree{
+                               5,
+                               &Tree{
+                                       6, nil, nil,
+                               },
+                               nil,
+                       },
+               },
+               &Tree{
+                       7,
+                       &Tree{
+                               8,
+                               &Tree{
+                                       9, nil, nil,
+                               },
+                               nil,
+                       },
+                       &Tree{
+                               10,
+                               &Tree{
+                                       11, nil, nil,
+                               },
+                               nil,
+                       },
+               },
+       }
+       tmpl, err := New("root").Delims("(", ")").Parse(treeTemplate)
+       if err != nil {
+               t.Fatal("parse error:", err)
+       }
+       var b bytes.Buffer
+       const expect = "[1[2[3[4]][5[6]]][7[8[9]][10[11]]]]"
+       // First by looking up the template.
+       err = tmpl.Lookup("tree").Execute(&b, tree)
+       if err != nil {
+               t.Fatal("exec error:", err)
+       }
+       result := b.String()
+       if result != expect {
+               t.Errorf("expected %q got %q", expect, result)
+       }
+       // Then direct to execution.
+       b.Reset()
+       err = tmpl.ExecuteTemplate(&b, "tree", tree)
+       if err != nil {
+               t.Fatal("exec error:", err)
+       }
+       result = b.String()
+       if result != expect {
+               t.Errorf("expected %q got %q", expect, result)
+       }
+}
+
+func TestExecuteOnNewTemplate(t *testing.T) {
+       // This is issue 3872.
+       New("Name").Templates()
+       // This is issue 11379.
+       // new(Template).Templates() // TODO: crashes
+       // new(Template).Parse("") // TODO: crashes
+       // new(Template).New("abc").Parse("") // TODO: crashes
+       // new(Template).Execute(nil, nil)                // TODO: crashes; returns an error (but does not crash)
+       // new(Template).ExecuteTemplate(nil, "XXX", nil) // TODO: crashes; returns an error (but does not crash)
+}
+
+const testTemplates = `{{define "one"}}one{{end}}{{define "two"}}two{{end}}`
+
+func TestMessageForExecuteEmpty(t *testing.T) {
+       // Test a truly empty template.
+       tmpl := New("empty")
+       var b bytes.Buffer
+       err := tmpl.Execute(&b, 0)
+       if err == nil {
+               t.Fatal("expected initial error")
+       }
+       got := err.Error()
+       want := `template: "empty" is an incomplete or empty template` // NOTE: text/template has extra "empty: " in message
+       if got != want {
+               t.Errorf("expected error %s got %s", want, got)
+       }
+
+       // Add a non-empty template to check that the error is helpful.
+       tmpl = New("empty")
+       tests, err := New("").Parse(testTemplates)
+       if err != nil {
+               t.Fatal(err)
+       }
+       tmpl.AddParseTree("secondary", tests.Tree)
+       err = tmpl.Execute(&b, 0)
+       if err == nil {
+               t.Fatal("expected second error")
+       }
+       got = err.Error()
+       if got != want {
+               t.Errorf("expected error %s got %s", want, got)
+       }
+       // Make sure we can execute the secondary.
+       err = tmpl.ExecuteTemplate(&b, "secondary", 0)
+       if err != nil {
+               t.Fatal(err)
+       }
+}
+
+func TestFinalForPrintf(t *testing.T) {
+       tmpl, err := New("").Parse(`{{"x" | printf}}`)
+       if err != nil {
+               t.Fatal(err)
+       }
+       var b bytes.Buffer
+       err = tmpl.Execute(&b, 0)
+       if err != nil {
+               t.Fatal(err)
+       }
+}
+
+type cmpTest struct {
+       expr  string
+       truth string
+       ok    bool
+}
+
+var cmpTests = []cmpTest{
+       {"eq true true", "true", true},
+       {"eq true false", "false", true},
+       {"eq 1+2i 1+2i", "true", true},
+       {"eq 1+2i 1+3i", "false", true},
+       {"eq 1.5 1.5", "true", true},
+       {"eq 1.5 2.5", "false", true},
+       {"eq 1 1", "true", true},
+       {"eq 1 2", "false", true},
+       {"eq `xy` `xy`", "true", true},
+       {"eq `xy` `xyz`", "false", true},
+       {"eq .Uthree .Uthree", "true", true},
+       {"eq .Uthree .Ufour", "false", true},
+       {"eq 3 4 5 6 3", "true", true},
+       {"eq 3 4 5 6 7", "false", true},
+       {"ne true true", "false", true},
+       {"ne true false", "true", true},
+       {"ne 1+2i 1+2i", "false", true},
+       {"ne 1+2i 1+3i", "true", true},
+       {"ne 1.5 1.5", "false", true},
+       {"ne 1.5 2.5", "true", true},
+       {"ne 1 1", "false", true},
+       {"ne 1 2", "true", true},
+       {"ne `xy` `xy`", "false", true},
+       {"ne `xy` `xyz`", "true", true},
+       {"ne .Uthree .Uthree", "false", true},
+       {"ne .Uthree .Ufour", "true", true},
+       {"lt 1.5 1.5", "false", true},
+       {"lt 1.5 2.5", "true", true},
+       {"lt 1 1", "false", true},
+       {"lt 1 2", "true", true},
+       {"lt `xy` `xy`", "false", true},
+       {"lt `xy` `xyz`", "true", true},
+       {"lt .Uthree .Uthree", "false", true},
+       {"lt .Uthree .Ufour", "true", true},
+       {"le 1.5 1.5", "true", true},
+       {"le 1.5 2.5", "true", true},
+       {"le 2.5 1.5", "false", true},
+       {"le 1 1", "true", true},
+       {"le 1 2", "true", true},
+       {"le 2 1", "false", true},
+       {"le `xy` `xy`", "true", true},
+       {"le `xy` `xyz`", "true", true},
+       {"le `xyz` `xy`", "false", true},
+       {"le .Uthree .Uthree", "true", true},
+       {"le .Uthree .Ufour", "true", true},
+       {"le .Ufour .Uthree", "false", true},
+       {"gt 1.5 1.5", "false", true},
+       {"gt 1.5 2.5", "false", true},
+       {"gt 1 1", "false", true},
+       {"gt 2 1", "true", true},
+       {"gt 1 2", "false", true},
+       {"gt `xy` `xy`", "false", true},
+       {"gt `xy` `xyz`", "false", true},
+       {"gt .Uthree .Uthree", "false", true},
+       {"gt .Uthree .Ufour", "false", true},
+       {"gt .Ufour .Uthree", "true", true},
+       {"ge 1.5 1.5", "true", true},
+       {"ge 1.5 2.5", "false", true},
+       {"ge 2.5 1.5", "true", true},
+       {"ge 1 1", "true", true},
+       {"ge 1 2", "false", true},
+       {"ge 2 1", "true", true},
+       {"ge `xy` `xy`", "true", true},
+       {"ge `xy` `xyz`", "false", true},
+       {"ge `xyz` `xy`", "true", true},
+       {"ge .Uthree .Uthree", "true", true},
+       {"ge .Uthree .Ufour", "false", true},
+       {"ge .Ufour .Uthree", "true", true},
+       // Mixing signed and unsigned integers.
+       {"eq .Uthree .Three", "true", true},
+       {"eq .Three .Uthree", "true", true},
+       {"le .Uthree .Three", "true", true},
+       {"le .Three .Uthree", "true", true},
+       {"ge .Uthree .Three", "true", true},
+       {"ge .Three .Uthree", "true", true},
+       {"lt .Uthree .Three", "false", true},
+       {"lt .Three .Uthree", "false", true},
+       {"gt .Uthree .Three", "false", true},
+       {"gt .Three .Uthree", "false", true},
+       {"eq .Ufour .Three", "false", true},
+       {"lt .Ufour .Three", "false", true},
+       {"gt .Ufour .Three", "true", true},
+       {"eq .NegOne .Uthree", "false", true},
+       {"eq .Uthree .NegOne", "false", true},
+       {"ne .NegOne .Uthree", "true", true},
+       {"ne .Uthree .NegOne", "true", true},
+       {"lt .NegOne .Uthree", "true", true},
+       {"lt .Uthree .NegOne", "false", true},
+       {"le .NegOne .Uthree", "true", true},
+       {"le .Uthree .NegOne", "false", true},
+       {"gt .NegOne .Uthree", "false", true},
+       {"gt .Uthree .NegOne", "true", true},
+       {"ge .NegOne .Uthree", "false", true},
+       {"ge .Uthree .NegOne", "true", true},
+       {"eq (index `x` 0) 'x'", "true", true}, // The example that triggered this rule.
+       {"eq (index `x` 0) 'y'", "false", true},
+       {"eq .V1 .V2", "true", true},
+       {"eq .Ptr .Ptr", "true", true},
+       {"eq .Ptr .NilPtr", "false", true},
+       {"eq .NilPtr .NilPtr", "true", true},
+       {"eq .Iface1 .Iface1", "true", true},
+       {"eq .Iface1 .Iface2", "false", true},
+       {"eq .Iface2 .Iface2", "true", true},
+       // Errors
+       {"eq `xy` 1", "", false},       // Different types.
+       {"eq 2 2.0", "", false},        // Different types.
+       {"lt true true", "", false},    // Unordered types.
+       {"lt 1+0i 1+0i", "", false},    // Unordered types.
+       {"eq .Ptr 1", "", false},       // Incompatible types.
+       {"eq .Ptr .NegOne", "", false}, // Incompatible types.
+       {"eq .Map .Map", "", false},    // Uncomparable types.
+       {"eq .Map .V1", "", false},     // Uncomparable types.
+}
+
+func TestComparison(t *testing.T) {
+       b := new(bytes.Buffer)
+       var cmpStruct = struct {
+               Uthree, Ufour  uint
+               NegOne, Three  int
+               Ptr, NilPtr    *int
+               Map            map[int]int
+               V1, V2         V
+               Iface1, Iface2 fmt.Stringer
+       }{
+               Uthree: 3,
+               Ufour:  4,
+               NegOne: -1,
+               Three:  3,
+               Ptr:    new(int),
+               Iface1: b,
+       }
+       for _, test := range cmpTests {
+               text := fmt.Sprintf("{{if %s}}true{{else}}false{{end}}", test.expr)
+               tmpl, err := New("empty").Parse(text)
+               if err != nil {
+                       t.Fatalf("%q: %s", test.expr, err)
+               }
+               b.Reset()
+               err = tmpl.Execute(b, &cmpStruct)
+               if test.ok && err != nil {
+                       t.Errorf("%s errored incorrectly: %s", test.expr, err)
+                       continue
+               }
+               if !test.ok && err == nil {
+                       t.Errorf("%s did not error", test.expr)
+                       continue
+               }
+               if b.String() != test.truth {
+                       t.Errorf("%s: want %s; got %s", test.expr, test.truth, b.String())
+               }
+       }
+}
+
+func TestMissingMapKey(t *testing.T) {
+       data := map[string]int{
+               "x": 99,
+       }
+       tmpl, err := New("t1").Parse("{{.x}} {{.y}}")
+       if err != nil {
+               t.Fatal(err)
+       }
+       var b bytes.Buffer
+       // By default, just get "<no value>" // NOTE: not in html/template, get empty string
+       err = tmpl.Execute(&b, data)
+       if err != nil {
+               t.Fatal(err)
+       }
+       want := "99 "
+       got := b.String()
+       if got != want {
+               t.Errorf("got %q; expected %q", got, want)
+       }
+       // Same if we set the option explicitly to the default.
+       tmpl.Option("missingkey=default")
+       b.Reset()
+       err = tmpl.Execute(&b, data)
+       if err != nil {
+               t.Fatal("default:", err)
+       }
+       got = b.String()
+       if got != want {
+               t.Errorf("got %q; expected %q", got, want)
+       }
+       // Next we ask for a zero value
+       tmpl.Option("missingkey=zero")
+       b.Reset()
+       err = tmpl.Execute(&b, data)
+       if err != nil {
+               t.Fatal("zero:", err)
+       }
+       want = "99 0"
+       got = b.String()
+       if got != want {
+               t.Errorf("got %q; expected %q", got, want)
+       }
+       // Now we ask for an error.
+       tmpl.Option("missingkey=error")
+       err = tmpl.Execute(&b, data)
+       if err == nil {
+               t.Errorf("expected error; got none")
+       }
+       // same Option, but now a nil interface: ask for an error
+       err = tmpl.Execute(&b, nil)
+       t.Log(err)
+       if err == nil {
+               t.Errorf("expected error for nil-interface; got none")
+       }
+}
+
+// Test that the error message for multiline unterminated string
+// refers to the line number of the opening quote.
+func TestUnterminatedStringError(t *testing.T) {
+       _, err := New("X").Parse("hello\n\n{{`unterminated\n\n\n\n}}\n some more\n\n")
+       if err == nil {
+               t.Fatal("expected error")
+       }
+       str := err.Error()
+       if !strings.Contains(str, "X:3: unterminated raw quoted string") {
+               t.Fatalf("unexpected error: %s", str)
+       }
+}
+
+const alwaysErrorText = "always be failing"
+
+var alwaysError = errors.New(alwaysErrorText)
+
+type ErrorWriter int
+
+func (e ErrorWriter) Write(p []byte) (int, error) {
+       return 0, alwaysError
+}
+
+func TestExecuteGivesExecError(t *testing.T) {
+       // First, a non-execution error shouldn't be an ExecError.
+       tmpl, err := New("X").Parse("hello")
+       if err != nil {
+               t.Fatal(err)
+       }
+       err = tmpl.Execute(ErrorWriter(0), 0)
+       if err == nil {
+               t.Fatal("expected error; got none")
+       }
+       if err.Error() != alwaysErrorText {
+               t.Errorf("expected %q error; got %q", alwaysErrorText, err)
+       }
+       // This one should be an ExecError.
+       tmpl, err = New("X").Parse("hello, {{.X.Y}}")
+       if err != nil {
+               t.Fatal(err)
+       }
+       err = tmpl.Execute(io.Discard, 0)
+       if err == nil {
+               t.Fatal("expected error; got none")
+       }
+       eerr, ok := err.(template.ExecError)
+       if !ok {
+               t.Fatalf("did not expect ExecError %s", eerr)
+       }
+       expect := "field X in type int"
+       if !strings.Contains(err.Error(), expect) {
+               t.Errorf("expected %q; got %q", expect, err)
+       }
+}
+
+func funcNameTestFunc() int {
+       return 0
+}
+
+func TestGoodFuncNames(t *testing.T) {
+       names := []string{
+               "_",
+               "a",
+               "a1",
+               "a1",
+               "Ӵ",
+       }
+       for _, name := range names {
+               tmpl := New("X").Funcs(
+                       FuncMap{
+                               name: funcNameTestFunc,
+                       },
+               )
+               if tmpl == nil {
+                       t.Fatalf("nil result for %q", name)
+               }
+       }
+}
+
+func TestBadFuncNames(t *testing.T) {
+       names := []string{
+               "",
+               "2",
+               "a-b",
+       }
+       for _, name := range names {
+               testBadFuncName(name, t)
+       }
+}
+
+func testBadFuncName(name string, t *testing.T) {
+       t.Helper()
+       defer func() {
+               recover()
+       }()
+       New("X").Funcs(
+               FuncMap{
+                       name: funcNameTestFunc,
+               },
+       )
+       // If we get here, the name did not cause a panic, which is how Funcs
+       // reports an error.
+       t.Errorf("%q succeeded incorrectly as function name", name)
+}
+
+func TestBlock(t *testing.T) {
+       const (
+               input   = `a({{block "inner" .}}bar({{.}})baz{{end}})b`
+               want    = `a(bar(hello)baz)b`
+               overlay = `{{define "inner"}}foo({{.}})bar{{end}}`
+               want2   = `a(foo(goodbye)bar)b`
+       )
+       tmpl, err := New("outer").Parse(input)
+       if err != nil {
+               t.Fatal(err)
+       }
+       tmpl2, err := Must(tmpl.Clone()).Parse(overlay)
+       if err != nil {
+               t.Fatal(err)
+       }
+
+       var buf bytes.Buffer
+       if err := tmpl.Execute(&buf, "hello"); err != nil {
+               t.Fatal(err)
+       }
+       if got := buf.String(); got != want {
+               t.Errorf("got %q, want %q", got, want)
+       }
+
+       buf.Reset()
+       if err := tmpl2.Execute(&buf, "goodbye"); err != nil {
+               t.Fatal(err)
+       }
+       if got := buf.String(); got != want2 {
+               t.Errorf("got %q, want %q", got, want2)
+       }
+}
+
+func TestEvalFieldErrors(t *testing.T) {
+       tests := []struct {
+               name, src string
+               value     interface{}
+               want      string
+       }{
+               {
+                       // Check that calling an invalid field on nil pointer
+                       // prints a field error instead of a distracting nil
+                       // pointer error. https://golang.org/issue/15125
+                       "MissingFieldOnNil",
+                       "{{.MissingField}}",
+                       (*T)(nil),
+                       "can't evaluate field MissingField in type *template.T",
+               },
+               {
+                       "MissingFieldOnNonNil",
+                       "{{.MissingField}}",
+                       &T{},
+                       "can't evaluate field MissingField in type *template.T",
+               },
+               {
+                       "ExistingFieldOnNil",
+                       "{{.X}}",
+                       (*T)(nil),
+                       "nil pointer evaluating *template.T.X",
+               },
+               {
+                       "MissingKeyOnNilMap",
+                       "{{.MissingKey}}",
+                       (*map[string]string)(nil),
+                       "nil pointer evaluating *map[string]string.MissingKey",
+               },
+               {
+                       "MissingKeyOnNilMapPtr",
+                       "{{.MissingKey}}",
+                       (*map[string]string)(nil),
+                       "nil pointer evaluating *map[string]string.MissingKey",
+               },
+               {
+                       "MissingKeyOnMapPtrToNil",
+                       "{{.MissingKey}}",
+                       &map[string]string{},
+                       "<nil>",
+               },
+       }
+       for _, tc := range tests {
+               t.Run(tc.name, func(t *testing.T) {
+                       tmpl := Must(New("tmpl").Parse(tc.src))
+                       err := tmpl.Execute(io.Discard, tc.value)
+                       got := "<nil>"
+                       if err != nil {
+                               got = err.Error()
+                       }
+                       if !strings.HasSuffix(got, tc.want) {
+                               t.Fatalf("got error %q, want %q", got, tc.want)
+                       }
+               })
+       }
+}
+
+func TestMaxExecDepth(t *testing.T) {
+       if testing.Short() {
+               t.Skip("skipping in -short mode")
+       }
+       tmpl := Must(New("tmpl").Parse(`{{template "tmpl" .}}`))
+       err := tmpl.Execute(io.Discard, nil)
+       got := "<nil>"
+       if err != nil {
+               got = err.Error()
+       }
+       const want = "exceeded maximum template depth"
+       if !strings.Contains(got, want) {
+               t.Errorf("got error %q; want %q", got, want)
+       }
+}
+
+func TestAddrOfIndex(t *testing.T) {
+       // golang.org/issue/14916.
+       // Before index worked on reflect.Values, the .String could not be
+       // found on the (incorrectly unaddressable) V value,
+       // in contrast to range, which worked fine.
+       // Also testing that passing a reflect.Value to tmpl.Execute works.
+       texts := []string{
+               `{{range .}}{{.String}}{{end}}`,
+               `{{with index . 0}}{{.String}}{{end}}`,
+       }
+       for _, text := range texts {
+               tmpl := Must(New("tmpl").Parse(text))
+               var buf bytes.Buffer
+               err := tmpl.Execute(&buf, reflect.ValueOf([]V{{1}}))
+               if err != nil {
+                       t.Fatalf("%s: Execute: %v", text, err)
+               }
+               if buf.String() != "&lt;1&gt;" {
+                       t.Fatalf("%s: template output = %q, want %q", text, &buf, "&lt;1&gt;")
+               }
+       }
+}
+
+func TestInterfaceValues(t *testing.T) {
+       // golang.org/issue/17714.
+       // Before index worked on reflect.Values, interface values
+       // were always implicitly promoted to the underlying value,
+       // except that nil interfaces were promoted to the zero reflect.Value.
+       // Eliminating a round trip to interface{} and back to reflect.Value
+       // eliminated this promotion, breaking these cases.
+       tests := []struct {
+               text string
+               out  string
+       }{
+               {`{{index .Nil 1}}`, "ERROR: index of untyped nil"},
+               {`{{index .Slice 2}}`, "2"},
+               {`{{index .Slice .Two}}`, "2"},
+               {`{{call .Nil 1}}`, "ERROR: call of nil"},
+               {`{{call .PlusOne 1}}`, "2"},
+               {`{{call .PlusOne .One}}`, "2"},
+               {`{{and (index .Slice 0) true}}`, "0"},
+               {`{{and .Zero true}}`, "0"},
+               {`{{and (index .Slice 1) false}}`, "false"},
+               {`{{and .One false}}`, "false"},
+               {`{{or (index .Slice 0) false}}`, "false"},
+               {`{{or .Zero false}}`, "false"},
+               {`{{or (index .Slice 1) true}}`, "1"},
+               {`{{or .One true}}`, "1"},
+               {`{{not (index .Slice 0)}}`, "true"},
+               {`{{not .Zero}}`, "true"},
+               {`{{not (index .Slice 1)}}`, "false"},
+               {`{{not .One}}`, "false"},
+               {`{{eq (index .Slice 0) .Zero}}`, "true"},
+               {`{{eq (index .Slice 1) .One}}`, "true"},
+               {`{{ne (index .Slice 0) .Zero}}`, "false"},
+               {`{{ne (index .Slice 1) .One}}`, "false"},
+               {`{{ge (index .Slice 0) .One}}`, "false"},
+               {`{{ge (index .Slice 1) .Zero}}`, "true"},
+               {`{{gt (index .Slice 0) .One}}`, "false"},
+               {`{{gt (index .Slice 1) .Zero}}`, "true"},
+               {`{{le (index .Slice 0) .One}}`, "true"},
+               {`{{le (index .Slice 1) .Zero}}`, "false"},
+               {`{{lt (index .Slice 0) .One}}`, "true"},
+               {`{{lt (index .Slice 1) .Zero}}`, "false"},
+       }
+
+       for _, tt := range tests {
+               tmpl := Must(New("tmpl").Parse(tt.text))
+               var buf bytes.Buffer
+               err := tmpl.Execute(&buf, map[string]interface{}{
+                       "PlusOne": func(n int) int {
+                               return n + 1
+                       },
+                       "Slice": []int{0, 1, 2, 3},
+                       "One":   1,
+                       "Two":   2,
+                       "Nil":   nil,
+                       "Zero":  0,
+               })
+               if strings.HasPrefix(tt.out, "ERROR:") {
+                       e := strings.TrimSpace(strings.TrimPrefix(tt.out, "ERROR:"))
+                       if err == nil || !strings.Contains(err.Error(), e) {
+                               t.Errorf("%s: Execute: %v, want error %q", tt.text, err, e)
+                       }
+                       continue
+               }
+               if err != nil {
+                       t.Errorf("%s: Execute: %v", tt.text, err)
+                       continue
+               }
+               if buf.String() != tt.out {
+                       t.Errorf("%s: template output = %q, want %q", tt.text, &buf, tt.out)
+               }
+       }
+}
+
+// Check that panics during calls are recovered and returned as errors.
+func TestExecutePanicDuringCall(t *testing.T) {
+       funcs := map[string]interface{}{
+               "doPanic": func() string {
+                       panic("custom panic string")
+               },
+       }
+       tests := []struct {
+               name    string
+               input   string
+               data    interface{}
+               wantErr string
+       }{
+               {
+                       "direct func call panics",
+                       "{{doPanic}}", (*T)(nil),
+                       `template: t:1:2: executing "t" at <doPanic>: error calling doPanic: custom panic string`,
+               },
+               {
+                       "indirect func call panics",
+                       "{{call doPanic}}", (*T)(nil),
+                       `template: t:1:7: executing "t" at <doPanic>: error calling doPanic: custom panic string`,
+               },
+               {
+                       "direct method call panics",
+                       "{{.GetU}}", (*T)(nil),
+                       `template: t:1:2: executing "t" at <.GetU>: error calling GetU: runtime error: invalid memory address or nil pointer dereference`,
+               },
+               {
+                       "indirect method call panics",
+                       "{{call .GetU}}", (*T)(nil),
+                       `template: t:1:7: executing "t" at <.GetU>: error calling GetU: runtime error: invalid memory address or nil pointer dereference`,
+               },
+               {
+                       "func field call panics",
+                       "{{call .PanicFunc}}", tVal,
+                       `template: t:1:2: executing "t" at <call .PanicFunc>: error calling call: test panic`,
+               },
+               {
+                       "method call on nil interface",
+                       "{{.NonEmptyInterfaceNil.Method0}}", tVal,
+                       `template: t:1:23: executing "t" at <.NonEmptyInterfaceNil.Method0>: nil pointer evaluating template.I.Method0`,
+               },
+       }
+       for _, tc := range tests {
+               b := new(bytes.Buffer)
+               tmpl, err := New("t").Funcs(funcs).Parse(tc.input)
+               if err != nil {
+                       t.Fatalf("parse error: %s", err)
+               }
+               err = tmpl.Execute(b, tc.data)
+               if err == nil {
+                       t.Errorf("%s: expected error; got none", tc.name)
+               } else if !strings.Contains(err.Error(), tc.wantErr) {
+                       if *debug {
+                               fmt.Printf("%s: test execute error: %s\n", tc.name, err)
+                       }
+                       t.Errorf("%s: expected error:\n%s\ngot:\n%s", tc.name, tc.wantErr, err)
+               }
+       }
+}
+
+// Issue 31810. Check that a parenthesized first argument behaves properly.
+func TestIssue31810(t *testing.T) {
+       t.Skip("broken in html/template")
+
+       // A simple value with no arguments is fine.
+       var b bytes.Buffer
+       const text = "{{ (.)  }}"
+       tmpl, err := New("").Parse(text)
+       if err != nil {
+               t.Error(err)
+       }
+       err = tmpl.Execute(&b, "result")
+       if err != nil {
+               t.Error(err)
+       }
+       if b.String() != "result" {
+               t.Errorf("%s got %q, expected %q", text, b.String(), "result")
+       }
+
+       // Even a plain function fails - need to use call.
+       f := func() string { return "result" }
+       b.Reset()
+       err = tmpl.Execute(&b, f)
+       if err == nil {
+               t.Error("expected error with no call, got none")
+       }
+
+       // Works if the function is explicitly called.
+       const textCall = "{{ (call .)  }}"
+       tmpl, err = New("").Parse(textCall)
+       b.Reset()
+       err = tmpl.Execute(&b, f)
+       if err != nil {
+               t.Error(err)
+       }
+       if b.String() != "result" {
+               t.Errorf("%s got %q, expected %q", textCall, b.String(), "result")
+       }
+}
index 13a0cd043607d48d738a53a73c434f61528b707a..356b8298ae36df752fa9f70f67ab97d0c220f6e0 100644 (file)
@@ -240,8 +240,7 @@ func htmlNameFilter(args ...interface{}) string {
        }
        s = strings.ToLower(s)
        if t := attrType(s); t != contentTypePlain {
-               // TODO: Split attr and element name part filters so we can whitelist
-               // attributes.
+               // TODO: Split attr and element name part filters so we can recognize known attributes.
                return filterFailsafe
        }
        for _, r := range s {
diff --git a/tpl/internal/go_templates/htmltemplate/multi_test.go b/tpl/internal/go_templates/htmltemplate/multi_test.go
new file mode 100644 (file)
index 0000000..fd61c3f
--- /dev/null
@@ -0,0 +1,292 @@
+// Copyright 2011 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+// Tests for multiple-template execution, copied from text/template.
+
+// +build go1.13,!windows
+
+package template
+
+import (
+       "archive/zip"
+       "bytes"
+       "os"
+       "testing"
+
+       "github.com/gohugoio/hugo/tpl/internal/go_templates/texttemplate/parse"
+)
+
+var multiExecTests = []execTest{
+       {"empty", "", "", nil, true},
+       {"text", "some text", "some text", nil, true},
+       {"invoke x", `{{template "x" .SI}}`, "TEXT", tVal, true},
+       {"invoke x no args", `{{template "x"}}`, "TEXT", tVal, true},
+       {"invoke dot int", `{{template "dot" .I}}`, "17", tVal, true},
+       {"invoke dot []int", `{{template "dot" .SI}}`, "[3 4 5]", tVal, true},
+       {"invoke dotV", `{{template "dotV" .U}}`, "v", tVal, true},
+       {"invoke nested int", `{{template "nested" .I}}`, "17", tVal, true},
+       {"variable declared by template", `{{template "nested" $x:=.SI}},{{index $x 1}}`, "[3 4 5],4", tVal, true},
+
+       // User-defined function: test argument evaluator.
+       {"testFunc literal", `{{oneArg "joe"}}`, "oneArg=joe", tVal, true},
+       {"testFunc .", `{{oneArg .}}`, "oneArg=joe", "joe", true},
+}
+
+// These strings are also in testdata/*.
+const multiText1 = `
+       {{define "x"}}TEXT{{end}}
+       {{define "dotV"}}{{.V}}{{end}}
+`
+
+const multiText2 = `
+       {{define "dot"}}{{.}}{{end}}
+       {{define "nested"}}{{template "dot" .}}{{end}}
+`
+
+func TestMultiExecute(t *testing.T) {
+       // Declare a couple of templates first.
+       template, err := New("root").Parse(multiText1)
+       if err != nil {
+               t.Fatalf("parse error for 1: %s", err)
+       }
+       _, err = template.Parse(multiText2)
+       if err != nil {
+               t.Fatalf("parse error for 2: %s", err)
+       }
+       testExecute(multiExecTests, template, t)
+}
+
+func TestParseFiles(t *testing.T) {
+       _, err := ParseFiles("DOES NOT EXIST")
+       if err == nil {
+               t.Error("expected error for non-existent file; got none")
+       }
+       template := New("root")
+       _, err = template.ParseFiles("testdata/file1.tmpl", "testdata/file2.tmpl")
+       if err != nil {
+               t.Fatalf("error parsing files: %v", err)
+       }
+       testExecute(multiExecTests, template, t)
+}
+
+func TestParseGlob(t *testing.T) {
+       _, err := ParseGlob("DOES NOT EXIST")
+       if err == nil {
+               t.Error("expected error for non-existent file; got none")
+       }
+       _, err = New("error").ParseGlob("[x")
+       if err == nil {
+               t.Error("expected error for bad pattern; got none")
+       }
+       template := New("root")
+       _, err = template.ParseGlob("testdata/file*.tmpl")
+       if err != nil {
+               t.Fatalf("error parsing files: %v", err)
+       }
+       testExecute(multiExecTests, template, t)
+}
+
+func TestParseFS(t *testing.T) {
+       fs := os.DirFS("testdata")
+
+       {
+               _, err := ParseFS(fs, "DOES NOT EXIST")
+               if err == nil {
+                       t.Error("expected error for non-existent file; got none")
+               }
+       }
+
+       {
+               template := New("root")
+               _, err := template.ParseFS(fs, "file1.tmpl", "file2.tmpl")
+               if err != nil {
+                       t.Fatalf("error parsing files: %v", err)
+               }
+               testExecute(multiExecTests, template, t)
+       }
+
+       {
+               template := New("root")
+               _, err := template.ParseFS(fs, "file*.tmpl")
+               if err != nil {
+                       t.Fatalf("error parsing files: %v", err)
+               }
+               testExecute(multiExecTests, template, t)
+       }
+}
+
+// In these tests, actual content (not just template definitions) comes from the parsed files.
+
+var templateFileExecTests = []execTest{
+       {"test", `{{template "tmpl1.tmpl"}}{{template "tmpl2.tmpl"}}`, "template1\n\ny\ntemplate2\n\nx\n", 0, true},
+}
+
+func TestParseFilesWithData(t *testing.T) {
+       template, err := New("root").ParseFiles("testdata/tmpl1.tmpl", "testdata/tmpl2.tmpl")
+       if err != nil {
+               t.Fatalf("error parsing files: %v", err)
+       }
+       testExecute(templateFileExecTests, template, t)
+}
+
+func TestParseGlobWithData(t *testing.T) {
+       template, err := New("root").ParseGlob("testdata/tmpl*.tmpl")
+       if err != nil {
+               t.Fatalf("error parsing files: %v", err)
+       }
+       testExecute(templateFileExecTests, template, t)
+}
+
+func TestParseZipFS(t *testing.T) {
+       z, err := zip.OpenReader("testdata/fs.zip")
+       if err != nil {
+               t.Fatalf("error parsing zip: %v", err)
+       }
+       template, err := New("root").ParseFS(z, "tmpl*.tmpl")
+       if err != nil {
+               t.Fatalf("error parsing files: %v", err)
+       }
+       testExecute(templateFileExecTests, template, t)
+}
+
+const (
+       cloneText1 = `{{define "a"}}{{template "b"}}{{template "c"}}{{end}}`
+       cloneText2 = `{{define "b"}}b{{end}}`
+       cloneText3 = `{{define "c"}}root{{end}}`
+       cloneText4 = `{{define "c"}}clone{{end}}`
+)
+
+// Issue 7032
+func TestAddParseTreeToUnparsedTemplate(t *testing.T) {
+       master := "{{define \"master\"}}{{end}}"
+       tmpl := New("master")
+       tree, err := parse.Parse("master", master, "", "", nil)
+       if err != nil {
+               t.Fatalf("unexpected parse err: %v", err)
+       }
+       masterTree := tree["master"]
+       tmpl.AddParseTree("master", masterTree) // used to panic
+}
+
+func TestRedefinition(t *testing.T) {
+       var tmpl *Template
+       var err error
+       if tmpl, err = New("tmpl1").Parse(`{{define "test"}}foo{{end}}`); err != nil {
+               t.Fatalf("parse 1: %v", err)
+       }
+       if _, err = tmpl.Parse(`{{define "test"}}bar{{end}}`); err != nil {
+               t.Fatalf("got error %v, expected nil", err)
+       }
+       if _, err = tmpl.New("tmpl2").Parse(`{{define "test"}}bar{{end}}`); err != nil {
+               t.Fatalf("got error %v, expected nil", err)
+       }
+}
+
+// Issue 10879
+func TestEmptyTemplateCloneCrash(t *testing.T) {
+       t1 := New("base")
+       t1.Clone() // used to panic
+}
+
+// Issue 10910, 10926
+func TestTemplateLookUp(t *testing.T) {
+       t.Skip("broken on html/template") // TODO
+       t1 := New("foo")
+       if t1.Lookup("foo") != nil {
+               t.Error("Lookup returned non-nil value for undefined template foo")
+       }
+       t1.New("bar")
+       if t1.Lookup("bar") != nil {
+               t.Error("Lookup returned non-nil value for undefined template bar")
+       }
+       t1.Parse(`{{define "foo"}}test{{end}}`)
+       if t1.Lookup("foo") == nil {
+               t.Error("Lookup returned nil value for defined template")
+       }
+}
+
+func TestParse(t *testing.T) {
+       // In multiple calls to Parse with the same receiver template, only one call
+       // can contain text other than space, comments, and template definitions
+       t1 := New("test")
+       if _, err := t1.Parse(`{{define "test"}}{{end}}`); err != nil {
+               t.Fatalf("parsing test: %s", err)
+       }
+       if _, err := t1.Parse(`{{define "test"}}{{/* this is a comment */}}{{end}}`); err != nil {
+               t.Fatalf("parsing test: %s", err)
+       }
+       if _, err := t1.Parse(`{{define "test"}}foo{{end}}`); err != nil {
+               t.Fatalf("parsing test: %s", err)
+       }
+}
+
+func TestEmptyTemplate(t *testing.T) {
+       cases := []struct {
+               defn []string
+               in   string
+               want string
+       }{
+               {[]string{"x", "y"}, "", "y"},
+               {[]string{""}, "once", ""},
+               {[]string{"", ""}, "twice", ""},
+               {[]string{"{{.}}", "{{.}}"}, "twice", "twice"},
+               {[]string{"{{/* a comment */}}", "{{/* a comment */}}"}, "comment", ""},
+               {[]string{"{{.}}", ""}, "twice", "twice"}, // TODO: should want "" not "twice"
+       }
+
+       for i, c := range cases {
+               root := New("root")
+
+               var (
+                       m   *Template
+                       err error
+               )
+               for _, d := range c.defn {
+                       m, err = root.New(c.in).Parse(d)
+                       if err != nil {
+                               t.Fatal(err)
+                       }
+               }
+               buf := &bytes.Buffer{}
+               if err := m.Execute(buf, c.in); err != nil {
+                       t.Error(i, err)
+                       continue
+               }
+               if buf.String() != c.want {
+                       t.Errorf("expected string %q: got %q", c.want, buf.String())
+               }
+       }
+}
+
+// Issue 19249 was a regression in 1.8 caused by the handling of empty
+// templates added in that release, which got different answers depending
+// on the order templates appeared in the internal map.
+func TestIssue19294(t *testing.T) {
+       // The empty block in "xhtml" should be replaced during execution
+       // by the contents of "stylesheet", but if the internal map associating
+       // names with templates is built in the wrong order, the empty block
+       // looks non-empty and this doesn't happen.
+       var inlined = map[string]string{
+               "stylesheet": `{{define "stylesheet"}}stylesheet{{end}}`,
+               "xhtml":      `{{block "stylesheet" .}}{{end}}`,
+       }
+       all := []string{"stylesheet", "xhtml"}
+       for i := 0; i < 100; i++ {
+               res, err := New("title.xhtml").Parse(`{{template "xhtml" .}}`)
+               if err != nil {
+                       t.Fatal(err)
+               }
+               for _, name := range all {
+                       _, err := res.New(name).Parse(inlined[name])
+                       if err != nil {
+                               t.Fatal(err)
+                       }
+               }
+               var buf bytes.Buffer
+               res.Execute(&buf, 0)
+               if buf.String() != "stylesheet" {
+                       t.Fatalf("iteration %d: got %q; expected %q", i, buf.String(), "stylesheet")
+               }
+       }
+}
index aa65d9cb129efee161bb504a782fa64cd1c26770..132a4102b04fa5dee3d97cde26002fa4e349343c 100644 (file)
@@ -7,7 +7,9 @@ package template
 import (
        "fmt"
        "io"
+       "io/fs"
        "io/ioutil"
+       "path"
        "path/filepath"
        "sync"
 
@@ -385,7 +387,7 @@ func Must(t *Template, err error) *Template {
 // For instance, ParseFiles("a/foo", "b/foo") stores "b/foo" as the template
 // named "foo", while "a/foo" is unavailable.
 func ParseFiles(filenames ...string) (*Template, error) {
-       return parseFiles(nil, filenames...)
+       return parseFiles(nil, readFileOS, filenames...)
 }
 
 // ParseFiles parses the named files and associates the resulting templates with
@@ -397,12 +399,12 @@ func ParseFiles(filenames ...string) (*Template, error) {
 //
 // ParseFiles returns an error if t or any associated template has already been executed.
 func (t *Template) ParseFiles(filenames ...string) (*Template, error) {
-       return parseFiles(t, filenames...)
+       return parseFiles(t, readFileOS, filenames...)
 }
 
 // parseFiles is the helper for the method and function. If the argument
 // template is nil, it is created from the first file.
-func parseFiles(t *Template, filenames ...string) (*Template, error) {
+func parseFiles(t *Template, readFile func(string) (string, []byte, error), filenames ...string) (*Template, error) {
        if err := t.checkCanParse(); err != nil {
                return nil, err
        }
@@ -412,12 +414,11 @@ func parseFiles(t *Template, filenames ...string) (*Template, error) {
                return nil, fmt.Errorf("html/template: no files named in call to ParseFiles")
        }
        for _, filename := range filenames {
-               b, err := ioutil.ReadFile(filename)
+               name, b, err := readFile(filename)
                if err != nil {
                        return nil, err
                }
                s := string(b)
-               name := filepath.Base(filename)
                // First template becomes return value if not already defined,
                // and we use that one for subsequent New calls to associate
                // all the templates together. Also, if this file has the same name
@@ -480,7 +481,7 @@ func parseGlob(t *Template, pattern string) (*Template, error) {
        if len(filenames) == 0 {
                return nil, fmt.Errorf("html/template: pattern matches no files: %#q", pattern)
        }
-       return parseFiles(t, filenames...)
+       return parseFiles(t, readFileOS, filenames...)
 }
 
 // IsTrue reports whether the value is 'true', in the sense of not the zero of its type,
@@ -489,3 +490,48 @@ func parseGlob(t *Template, pattern string) (*Template, error) {
 func IsTrue(val interface{}) (truth, ok bool) {
        return template.IsTrue(val)
 }
+
+// ParseFS is like ParseFiles or ParseGlob but reads from the file system fs
+// instead of the host operating system's file system.
+// It accepts a list of glob patterns.
+// (Note that most file names serve as glob patterns matching only themselves.)
+func ParseFS(fs fs.FS, patterns ...string) (*Template, error) {
+       return parseFS(nil, fs, patterns)
+}
+
+// ParseFS is like ParseFiles or ParseGlob but reads from the file system fs
+// instead of the host operating system's file system.
+// It accepts a list of glob patterns.
+// (Note that most file names serve as glob patterns matching only themselves.)
+func (t *Template) ParseFS(fs fs.FS, patterns ...string) (*Template, error) {
+       return parseFS(t, fs, patterns)
+}
+
+func parseFS(t *Template, fsys fs.FS, patterns []string) (*Template, error) {
+       var filenames []string
+       for _, pattern := range patterns {
+               list, err := fs.Glob(fsys, pattern)
+               if err != nil {
+                       return nil, err
+               }
+               if len(list) == 0 {
+                       return nil, fmt.Errorf("template: pattern matches no files: %#q", pattern)
+               }
+               filenames = append(filenames, list...)
+       }
+       return parseFiles(t, readFileFS(fsys), filenames...)
+}
+
+func readFileOS(file string) (name string, b []byte, err error) {
+       name = filepath.Base(file)
+       b, err = ioutil.ReadFile(file)
+       return
+}
+
+func readFileFS(fsys fs.FS) func(string) (string, []byte, error) {
+       return func(file string) (name string, b []byte, err error) {
+               name = path.Base(file)
+               b, err = fs.ReadFile(fsys, file)
+               return
+       }
+}
index 589e6912a247c004b58345628121ddae13f8524e..562d50b229b495b36cbe6072c5d46bc200f51463 100644 (file)
@@ -13,10 +13,11 @@ import (
        "testing"
 
        . "github.com/gohugoio/hugo/tpl/internal/go_templates/htmltemplate"
+       "github.com/gohugoio/hugo/tpl/internal/go_templates/texttemplate/parse" // https://golang.org/issue/12996
 )
 
 func TestTemplateClone(t *testing.T) {
-       // https://golang.org/issue/12996
+
        orig := New("name")
        clone, err := orig.Clone()
        if err != nil {
@@ -163,6 +164,21 @@ func TestStringsInScriptsWithJsonContentTypeAreCorrectlyEscaped(t *testing.T) {
        }
 }
 
+func TestSkipEscapeComments(t *testing.T) {
+       c := newTestCase(t)
+       tr := parse.New("root")
+       tr.Mode = parse.ParseComments
+       newT, err := tr.Parse("{{/* A comment */}}{{ 1 }}{{/* Another comment */}}", "", "", make(map[string]*parse.Tree))
+       if err != nil {
+               t.Fatalf("Cannot parse template text: %v", err)
+       }
+       c.root, err = c.root.AddParseTree("root", newT)
+       if err != nil {
+               t.Fatalf("Cannot add parse tree to template: %v", err)
+       }
+       c.mustExecute(c.root, nil, "1")
+}
+
 type testCase struct {
        t    *testing.T
        root *Template
index f5ea398fb887b1a19dc16b773adb80c586be32d9..5bd91591a94db3d6a12ed7d30a009b08503ec27e 100644 (file)
@@ -45,12 +45,8 @@ func HasGoBuild() bool {
                return false
        }
        switch runtime.GOOS {
-       case "android", "js":
+       case "android", "js", "ios":
                return false
-       case "darwin":
-               if runtime.GOARCH == "arm64" {
-                       return false
-               }
        }
        return true
 }
@@ -124,12 +120,8 @@ func GoTool() (string, error) {
 // using os.StartProcess or (more commonly) exec.Command.
 func HasExec() bool {
        switch runtime.GOOS {
-       case "js":
+       case "js", "ios":
                return false
-       case "darwin":
-               if runtime.GOARCH == "arm64" {
-                       return false
-               }
        }
        return true
 }
@@ -137,10 +129,8 @@ func HasExec() bool {
 // HasSrc reports whether the entire source tree is available under GOROOT.
 func HasSrc() bool {
        switch runtime.GOOS {
-       case "darwin":
-               if runtime.GOARCH == "arm64" {
-                       return false
-               }
+       case "ios":
+               return false
        }
        return true
 }
@@ -204,6 +194,32 @@ func MustHaveCGO(t testing.TB) {
        }
 }
 
+// CanInternalLink reports whether the current system can link programs with
+// internal linking.
+// (This is the opposite of cmd/internal/sys.MustLinkExternal. Keep them in sync.)
+func CanInternalLink() bool {
+       switch runtime.GOOS {
+       case "android":
+               if runtime.GOARCH != "arm64" {
+                       return false
+               }
+       case "ios":
+               if runtime.GOARCH == "arm64" {
+                       return false
+               }
+       }
+       return true
+}
+
+// MustInternalLink checks that the current system can link programs with internal
+// linking.
+// If not, MustInternalLink calls t.Skip with an explanation.
+func MustInternalLink(t testing.TB) {
+       if !CanInternalLink() {
+               t.Skipf("skipping test: internal linking on %s/%s is not supported", runtime.GOOS, runtime.GOARCH)
+       }
+}
+
 // HasSymlink reports whether the current system can use os.Symlink.
 func HasSymlink() bool {
        ok, _ := hasSymlink()
@@ -272,3 +288,23 @@ func CleanCmdEnv(cmd *exec.Cmd) *exec.Cmd {
        }
        return cmd
 }
+
+// CPUIsSlow reports whether the CPU running the test is suspected to be slow.
+func CPUIsSlow() bool {
+       switch runtime.GOARCH {
+       case "arm", "mips", "mipsle", "mips64", "mips64le":
+               return true
+       }
+       return false
+}
+
+// SkipIfShortAndSlow skips t if -short is set and the CPU running the test is
+// suspected to be slow.
+//
+// (This is useful for CPU-intensive tests that otherwise complete quickly.)
+func SkipIfShortAndSlow(t testing.TB) {
+       if testing.Short() && CPUIsSlow() {
+               t.Helper()
+               t.Skipf("skipping test in -short mode on %s", runtime.GOARCH)
+       }
+}
index 4b0efd2df87ad38012eb92aba3ca83563737b1f8..7b3029433690c94b932bbda092392c3599a7b53e 100644 (file)
@@ -40,16 +40,17 @@ More intricate examples appear below.
 Text and spaces
 
 By default, all text between actions is copied verbatim when the template is
-executed. For example, the string " items are made of " in the example above appears
-on standard output when the program is run.
-
-However, to aid in formatting template source code, if an action's left delimiter
-(by default "{{") is followed immediately by a minus sign and ASCII space character
-("{{- "), all trailing white space is trimmed from the immediately preceding text.
-Similarly, if the right delimiter ("}}") is preceded by a space and minus sign
-(" -}}"), all leading white space is trimmed from the immediately following text.
-In these trim markers, the ASCII space must be present; "{{-3}}" parses as an
-action containing the number -3.
+executed. For example, the string " items are made of " in the example above
+appears on standard output when the program is run.
+
+However, to aid in formatting template source code, if an action's left
+delimiter (by default "{{") is followed immediately by a minus sign and white
+space, all trailing white space is trimmed from the immediately preceding text.
+Similarly, if the right delimiter ("}}") is preceded by white space and a minus
+sign, all leading white space is trimmed from the immediately following text.
+In these trim markers, the white space must be present:
+"{{- 3}}" is like "{{3}}" but trims the immediately preceding text, while
+"{{-3}}" parses as an action containing the number -3.
 
 For instance, when executing the template whose source is
 
index 879cd08846ea70d3baf5d2d3a3b2b1954df868d5..ca6a1a07146466fc589ac24300b282bb1af03430 100644 (file)
@@ -256,6 +256,7 @@ func (s *state) walk(dot reflect.Value, node parse.Node) {
                if len(node.Pipe.Decl) == 0 {
                        s.printValue(node, val)
                }
+       case *parse.CommentNode:
        case *parse.IfNode:
                s.walkIfOrWith(parse.NodeIf, dot, node.Pipe, node.List, node.ElseList)
        case *parse.ListNode:
index 940a1de6a0543072bcc3a2e867d1cf0282bdf0ef..3f16b6eb6ca19f7bbdb4249e288bd538342778c5 100644 (file)
@@ -11,7 +11,7 @@ import (
        "errors"
        "flag"
        "fmt"
-       "io/ioutil"
+       "io"
        "reflect"
        "strings"
        "testing"
@@ -1297,7 +1297,7 @@ func TestUnterminatedStringError(t *testing.T) {
                t.Fatal("expected error")
        }
        str := err.Error()
-       if !strings.Contains(str, "X:3: unexpected unterminated raw quoted string") {
+       if !strings.Contains(str, "X:3: unterminated raw quoted string") {
                t.Fatalf("unexpected error: %s", str)
        }
 }
@@ -1330,7 +1330,7 @@ func TestExecuteGivesExecError(t *testing.T) {
        if err != nil {
                t.Fatal(err)
        }
-       err = tmpl.Execute(ioutil.Discard, 0)
+       err = tmpl.Execute(io.Discard, 0)
        if err == nil {
                t.Fatal("expected error; got none")
        }
@@ -1476,7 +1476,7 @@ func TestEvalFieldErrors(t *testing.T) {
        for _, tc := range tests {
                t.Run(tc.name, func(t *testing.T) {
                        tmpl := Must(New("tmpl").Parse(tc.src))
-                       err := tmpl.Execute(ioutil.Discard, tc.value)
+                       err := tmpl.Execute(io.Discard, tc.value)
                        got := "<nil>"
                        if err != nil {
                                got = err.Error()
@@ -1493,7 +1493,7 @@ func TestMaxExecDepth(t *testing.T) {
                t.Skip("skipping in -short mode")
        }
        tmpl := Must(New("tmpl").Parse(`{{template "tmpl" .}}`))
-       err := tmpl.Execute(ioutil.Discard, nil)
+       err := tmpl.Execute(io.Discard, nil)
        got := "<nil>"
        if err != nil {
                got = err.Error()
index c9e890078c4ea03c002ae7b5449273e4da384685..8269fa28c50d3d0b9c94898969a49f0f3d15369c 100644 (file)
@@ -8,7 +8,9 @@ package template
 
 import (
        "fmt"
+       "io/fs"
        "io/ioutil"
+       "path"
        "path/filepath"
 )
 
@@ -35,7 +37,7 @@ func Must(t *Template, err error) *Template {
 // For instance, ParseFiles("a/foo", "b/foo") stores "b/foo" as the template
 // named "foo", while "a/foo" is unavailable.
 func ParseFiles(filenames ...string) (*Template, error) {
-       return parseFiles(nil, filenames...)
+       return parseFiles(nil, readFileOS, filenames...)
 }
 
 // ParseFiles parses the named files and associates the resulting templates with
@@ -51,23 +53,22 @@ func ParseFiles(filenames ...string) (*Template, error) {
 // the last one mentioned will be the one that results.
 func (t *Template) ParseFiles(filenames ...string) (*Template, error) {
        t.init()
-       return parseFiles(t, filenames...)
+       return parseFiles(t, readFileOS, filenames...)
 }
 
 // parseFiles is the helper for the method and function. If the argument
 // template is nil, it is created from the first file.
-func parseFiles(t *Template, filenames ...string) (*Template, error) {
+func parseFiles(t *Template, readFile func(string) (string, []byte, error), filenames ...string) (*Template, error) {
        if len(filenames) == 0 {
                // Not really a problem, but be consistent.
                return nil, fmt.Errorf("template: no files named in call to ParseFiles")
        }
        for _, filename := range filenames {
-               b, err := ioutil.ReadFile(filename)
+               name, b, err := readFile(filename)
                if err != nil {
                        return nil, err
                }
                s := string(b)
-               name := filepath.Base(filename)
                // First template becomes return value if not already defined,
                // and we use that one for subsequent New calls to associate
                // all the templates together. Also, if this file has the same name
@@ -126,5 +127,51 @@ func parseGlob(t *Template, pattern string) (*Template, error) {
        if len(filenames) == 0 {
                return nil, fmt.Errorf("template: pattern matches no files: %#q", pattern)
        }
-       return parseFiles(t, filenames...)
+       return parseFiles(t, readFileOS, filenames...)
+}
+
+// ParseFS is like ParseFiles or ParseGlob but reads from the file system fsys
+// instead of the host operating system's file system.
+// It accepts a list of glob patterns.
+// (Note that most file names serve as glob patterns matching only themselves.)
+func ParseFS(fsys fs.FS, patterns ...string) (*Template, error) {
+       return parseFS(nil, fsys, patterns)
+}
+
+// ParseFS is like ParseFiles or ParseGlob but reads from the file system fsys
+// instead of the host operating system's file system.
+// It accepts a list of glob patterns.
+// (Note that most file names serve as glob patterns matching only themselves.)
+func (t *Template) ParseFS(fsys fs.FS, patterns ...string) (*Template, error) {
+       t.init()
+       return parseFS(t, fsys, patterns)
+}
+
+func parseFS(t *Template, fsys fs.FS, patterns []string) (*Template, error) {
+       var filenames []string
+       for _, pattern := range patterns {
+               list, err := fs.Glob(fsys, pattern)
+               if err != nil {
+                       return nil, err
+               }
+               if len(list) == 0 {
+                       return nil, fmt.Errorf("template: pattern matches no files: %#q", pattern)
+               }
+               filenames = append(filenames, list...)
+       }
+       return parseFiles(t, readFileFS(fsys), filenames...)
+}
+
+func readFileOS(file string) (name string, b []byte, err error) {
+       name = filepath.Base(file)
+       b, err = ioutil.ReadFile(file)
+       return
+}
+
+func readFileFS(fsys fs.FS) func(string) (string, []byte, error) {
+       return func(file string) (name string, b []byte, err error) {
+               name = path.Base(file)
+               b, err = fs.ReadFile(fsys, file)
+               return
+       }
 }
diff --git a/tpl/internal/go_templates/texttemplate/link_test.go b/tpl/internal/go_templates/texttemplate/link_test.go
new file mode 100644 (file)
index 0000000..3db8c21
--- /dev/null
@@ -0,0 +1,66 @@
+// Copyright 2019 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+// +build go1.13
+
+package template_test
+
+import (
+       "bytes"
+       "github.com/gohugoio/hugo/tpl/internal/go_templates/testenv"
+       "io/ioutil"
+       "os"
+       "os/exec"
+       "path/filepath"
+       "testing"
+)
+
+// Issue 36021: verify that text/template doesn't prevent the linker from removing
+// unused methods.
+func _TestLinkerGC(t *testing.T) {
+       if testing.Short() {
+               t.Skip("skipping in short mode")
+       }
+       testenv.MustHaveGoBuild(t)
+       const prog = `package main
+
+import (
+       _ "text/template"
+)
+
+type T struct{}
+
+func (t *T) Unused() { println("THIS SHOULD BE ELIMINATED") }
+func (t *T) Used() {}
+
+var sink *T
+
+func main() {
+       var t T
+       sink = &t
+       t.Used()
+}
+`
+       td, err := ioutil.TempDir("", "text_template_TestDeadCodeElimination")
+       if err != nil {
+               t.Fatal(err)
+       }
+       defer os.RemoveAll(td)
+
+       if err := ioutil.WriteFile(filepath.Join(td, "x.go"), []byte(prog), 0644); err != nil {
+               t.Fatal(err)
+       }
+       cmd := exec.Command(testenv.GoToolPath(t), "build", "-o", "x.exe", "x.go")
+       cmd.Dir = td
+       if out, err := cmd.CombinedOutput(); err != nil {
+               t.Fatalf("go build: %v, %s", err, out)
+       }
+       slurp, err := ioutil.ReadFile(filepath.Join(td, "x.exe"))
+       if err != nil {
+               t.Fatal(err)
+       }
+       if bytes.Contains(slurp, []byte("THIS SHOULD BE ELIMINATED")) {
+               t.Error("binary contains code that should be deadcode eliminated")
+       }
+}
index 7323be379515dc0b22421ec093f64083b311ae6c..99c2f7759781755391c925e0ab6451a0d243c661 100644 (file)
@@ -12,6 +12,7 @@ import (
        "bytes"
        "fmt"
        "github.com/gohugoio/hugo/tpl/internal/go_templates/texttemplate/parse"
+       "os"
        "testing"
 )
 
@@ -155,6 +156,35 @@ func TestParseGlob(t *testing.T) {
        testExecute(multiExecTests, template, t)
 }
 
+func TestParseFS(t *testing.T) {
+       fs := os.DirFS("testdata")
+
+       {
+               _, err := ParseFS(fs, "DOES NOT EXIST")
+               if err == nil {
+                       t.Error("expected error for non-existent file; got none")
+               }
+       }
+
+       {
+               template := New("root")
+               _, err := template.ParseFS(fs, "file1.tmpl", "file2.tmpl")
+               if err != nil {
+                       t.Fatalf("error parsing files: %v", err)
+               }
+               testExecute(multiExecTests, template, t)
+       }
+
+       {
+               template := New("root")
+               _, err := template.ParseFS(fs, "file*.tmpl")
+               if err != nil {
+                       t.Fatalf("error parsing files: %v", err)
+               }
+               testExecute(multiExecTests, template, t)
+       }
+}
+
 // In these tests, actual content (not just template definitions) comes from the parsed files.
 
 var templateFileExecTests = []execTest{
@@ -361,6 +391,7 @@ func TestEmptyTemplate(t *testing.T) {
                in   string
                want string
        }{
+               {[]string{"x", "y"}, "", "y"},
                {[]string{""}, "once", ""},
                {[]string{"", ""}, "twice", ""},
                {[]string{"{{.}}", "{{.}}"}, "twice", "twice"},
index 30371f28626761633106b1dc4460833cff83a9a6..6784071b1118d16f752d5b39e8467af013881f7e 100644 (file)
@@ -41,6 +41,7 @@ const (
        itemBool                         // boolean constant
        itemChar                         // printable ASCII character; grab bag for comma etc.
        itemCharConstant                 // character constant
+       itemComment                      // comment text
        itemComplex                      // complex constant (1+2i); imaginary is just a number
        itemAssign                       // equals ('=') introducing an assignment
        itemDeclare                      // colon-equals (':=') introducing a declaration
@@ -91,15 +92,14 @@ const eof = -1
 // If the action begins "{{- " rather than "{{", then all space/tab/newlines
 // preceding the action are trimmed; conversely if it ends " -}}" the
 // leading spaces are trimmed. This is done entirely in the lexer; the
-// parser never sees it happen. We require an ASCII space to be
-// present to avoid ambiguity with things like "{{-3}}". It reads
+// parser never sees it happen. We require an ASCII space (' ', \t, \r, \n)
+// to be present to avoid ambiguity with things like "{{-3}}". It reads
 // better with the space present anyway. For simplicity, only ASCII
-// space does the job.
+// does the job.
 const (
-       spaceChars      = " \t\r\n" // These are the space characters defined by Go itself.
-       leftTrimMarker  = "- "      // Attached to left delimiter, trims trailing spaces from preceding text.
-       rightTrimMarker = " -"      // Attached to right delimiter, trims leading spaces from following text.
-       trimMarkerLen   = Pos(len(leftTrimMarker))
+       spaceChars    = " \t\r\n"  // These are the space characters defined by Go itself.
+       trimMarker    = '-'        // Attached to left/right delimiter, trims trailing spaces from preceding/following text.
+       trimMarkerLen = Pos(1 + 1) // marker plus space before or after
 )
 
 // stateFn represents the state of the scanner as a function that returns the next state.
@@ -107,18 +107,18 @@ type stateFn func(*lexer) stateFn
 
 // lexer holds the state of the scanner.
 type lexer struct {
-       name           string    // the name of the input; used only for error reports
-       input          string    // the string being scanned
-       leftDelim      string    // start of action
-       rightDelim     string    // end of action
-       trimRightDelim string    // end of action with trim marker
-       pos            Pos       // current position in the input
-       start          Pos       // start position of this item
-       width          Pos       // width of last rune read from input
-       items          chan item // channel of scanned items
-       parenDepth     int       // nesting depth of ( ) exprs
-       line           int       // 1+number of newlines seen
-       startLine      int       // start line of this item
+       name        string    // the name of the input; used only for error reports
+       input       string    // the string being scanned
+       leftDelim   string    // start of action
+       rightDelim  string    // end of action
+       emitComment bool      // emit itemComment tokens.
+       pos         Pos       // current position in the input
+       start       Pos       // start position of this item
+       width       Pos       // width of last rune read from input
+       items       chan item // channel of scanned items
+       parenDepth  int       // nesting depth of ( ) exprs
+       line        int       // 1+number of newlines seen
+       startLine   int       // start line of this item
 }
 
 // next returns the next rune in the input.
@@ -203,7 +203,7 @@ func (l *lexer) drain() {
 }
 
 // lex creates a new scanner for the input string.
-func lex(name, input, left, right string) *lexer {
+func lex(name, input, left, right string, emitComment bool) *lexer {
        if left == "" {
                left = leftDelim
        }
@@ -211,14 +211,14 @@ func lex(name, input, left, right string) *lexer {
                right = rightDelim
        }
        l := &lexer{
-               name:           name,
-               input:          input,
-               leftDelim:      left,
-               rightDelim:     right,
-               trimRightDelim: rightTrimMarker + right,
-               items:          make(chan item),
-               line:           1,
-               startLine:      1,
+               name:        name,
+               input:       input,
+               leftDelim:   left,
+               rightDelim:  right,
+               emitComment: emitComment,
+               items:       make(chan item),
+               line:        1,
+               startLine:   1,
        }
        go l.run()
        return l
@@ -248,7 +248,7 @@ func lexText(l *lexer) stateFn {
                ldn := Pos(len(l.leftDelim))
                l.pos += Pos(x)
                trimLength := Pos(0)
-               if strings.HasPrefix(l.input[l.pos+ldn:], leftTrimMarker) {
+               if hasLeftTrimMarker(l.input[l.pos+ldn:]) {
                        trimLength = rightTrimLength(l.input[l.start:l.pos])
                }
                l.pos -= trimLength
@@ -277,7 +277,7 @@ func rightTrimLength(s string) Pos {
 
 // atRightDelim reports whether the lexer is at a right delimiter, possibly preceded by a trim marker.
 func (l *lexer) atRightDelim() (delim, trimSpaces bool) {
-       if strings.HasPrefix(l.input[l.pos:], l.trimRightDelim) { // With trim marker.
+       if hasRightTrimMarker(l.input[l.pos:]) && strings.HasPrefix(l.input[l.pos+trimMarkerLen:], l.rightDelim) { // With trim marker.
                return true, true
        }
        if strings.HasPrefix(l.input[l.pos:], l.rightDelim) { // Without trim marker.
@@ -294,7 +294,7 @@ func leftTrimLength(s string) Pos {
 // lexLeftDelim scans the left delimiter, which is known to be present, possibly with a trim marker.
 func lexLeftDelim(l *lexer) stateFn {
        l.pos += Pos(len(l.leftDelim))
-       trimSpace := strings.HasPrefix(l.input[l.pos:], leftTrimMarker)
+       trimSpace := hasLeftTrimMarker(l.input[l.pos:])
        afterMarker := Pos(0)
        if trimSpace {
                afterMarker = trimMarkerLen
@@ -323,6 +323,9 @@ func lexComment(l *lexer) stateFn {
        if !delim {
                return l.errorf("comment ends before closing delimiter")
        }
+       if l.emitComment {
+               l.emit(itemComment)
+       }
        if trimSpace {
                l.pos += trimMarkerLen
        }
@@ -336,7 +339,7 @@ func lexComment(l *lexer) stateFn {
 
 // lexRightDelim scans the right delimiter, which is known to be present, possibly with a trim marker.
 func lexRightDelim(l *lexer) stateFn {
-       trimSpace := strings.HasPrefix(l.input[l.pos:], rightTrimMarker)
+       trimSpace := hasRightTrimMarker(l.input[l.pos:])
        if trimSpace {
                l.pos += trimMarkerLen
                l.ignore()
@@ -363,7 +366,7 @@ func lexInsideAction(l *lexer) stateFn {
                return l.errorf("unclosed left paren")
        }
        switch r := l.next(); {
-       case r == eof || isEndOfLine(r):
+       case r == eof:
                return l.errorf("unclosed action")
        case isSpace(r):
                l.backup() // Put space back in case we have " -}}".
@@ -433,7 +436,7 @@ func lexSpace(l *lexer) stateFn {
        }
        // Be careful about a trim-marked closing delimiter, which has a minus
        // after a space. We know there is a space, so check for the '-' that might follow.
-       if strings.HasPrefix(l.input[l.pos-1:], l.trimRightDelim) {
+       if hasRightTrimMarker(l.input[l.pos-1:]) && strings.HasPrefix(l.input[l.pos-1+trimMarkerLen:], l.rightDelim) {
                l.backup() // Before the space.
                if numSpaces == 1 {
                        return lexRightDelim // On the delim, so go right to that.
@@ -520,7 +523,7 @@ func lexFieldOrVariable(l *lexer, typ itemType) stateFn {
 // day to implement arithmetic.
 func (l *lexer) atTerminator() bool {
        r := l.peek()
-       if isSpace(r) || isEndOfLine(r) {
+       if isSpace(r) {
                return true
        }
        switch r {
@@ -651,15 +654,18 @@ Loop:
 
 // isSpace reports whether r is a space character.
 func isSpace(r rune) bool {
-       return r == ' ' || r == '\t'
-}
-
-// isEndOfLine reports whether r is an end-of-line character.
-func isEndOfLine(r rune) bool {
-       return r == '\r' || r == '\n'
+       return r == ' ' || r == '\t' || r == '\r' || r == '\n'
 }
 
 // isAlphaNumeric reports whether r is an alphabetic, digit, or underscore.
 func isAlphaNumeric(r rune) bool {
        return r == '_' || unicode.IsLetter(r) || unicode.IsDigit(r)
 }
+
+func hasLeftTrimMarker(s string) bool {
+       return len(s) >= 2 && s[0] == trimMarker && isSpace(rune(s[1]))
+}
+
+func hasRightTrimMarker(s string) bool {
+       return len(s) >= 2 && isSpace(rune(s[0])) && s[1] == trimMarker
+}
index caadc52ed984cd3bb2e90deedf893cdb730883e1..7ba4ee2b809caeb45fab4d746317e3d74be89b93 100644 (file)
@@ -17,6 +17,7 @@ var itemName = map[itemType]string{
        itemBool:         "bool",
        itemChar:         "char",
        itemCharConstant: "charconst",
+       itemComment:      "comment",
        itemComplex:      "complex",
        itemDeclare:      ":=",
        itemEOF:          "EOF",
@@ -92,6 +93,7 @@ var lexTests = []lexTest{
        {"text", `now is the time`, []item{mkItem(itemText, "now is the time"), tEOF}},
        {"text with comment", "hello-{{/* this is a comment */}}-world", []item{
                mkItem(itemText, "hello-"),
+               mkItem(itemComment, "/* this is a comment */"),
                mkItem(itemText, "-world"),
                tEOF,
        }},
@@ -313,6 +315,7 @@ var lexTests = []lexTest{
        }},
        {"trimming spaces before and after comment", "hello- {{- /* hello */ -}} -world", []item{
                mkItem(itemText, "hello-"),
+               mkItem(itemComment, "/* hello */"),
                mkItem(itemText, "-world"),
                tEOF,
        }},
@@ -322,7 +325,7 @@ var lexTests = []lexTest{
                tLeft,
                mkItem(itemError, "unrecognized character in action: U+0001"),
        }},
-       {"unclosed action", "{{\n}}", []item{
+       {"unclosed action", "{{", []item{
                tLeft,
                mkItem(itemError, "unclosed action"),
        }},
@@ -391,7 +394,7 @@ var lexTests = []lexTest{
 
 // collect gathers the emitted items into a slice.
 func collect(t *lexTest, left, right string) (items []item) {
-       l := lex(t.name, t.input, left, right)
+       l := lex(t.name, t.input, left, right, true)
        for {
                item := l.nextItem()
                items = append(items, item)
@@ -531,7 +534,7 @@ func TestPos(t *testing.T) {
 func TestShutdown(t *testing.T) {
        // We need to duplicate template.Parse here to hold on to the lexer.
        const text = "erroneous{{define}}{{else}}1234"
-       lexer := lex("foo", text, "{{", "}}")
+       lexer := lex("foo", text, "{{", "}}", false)
        _, err := New("root").parseLexer(lexer)
        if err == nil {
                t.Fatalf("expected error")
index 1c116ea6fab3dbf594306b9846dbf97f74715243..177482f9b26059b5183e29b0146985b796dc134d 100644 (file)
@@ -70,6 +70,7 @@ const (
        NodeTemplate                   // A template invocation action.
        NodeVariable                   // A $ variable.
        NodeWith                       // A with action.
+       NodeComment                    // A comment.
 )
 
 // Nodes.
@@ -149,6 +150,38 @@ func (t *TextNode) Copy() Node {
        return &TextNode{tr: t.tr, NodeType: NodeText, Pos: t.Pos, Text: append([]byte{}, t.Text...)}
 }
 
+// CommentNode holds a comment.
+type CommentNode struct {
+       NodeType
+       Pos
+       tr   *Tree
+       Text string // Comment text.
+}
+
+func (t *Tree) newComment(pos Pos, text string) *CommentNode {
+       return &CommentNode{tr: t, NodeType: NodeComment, Pos: pos, Text: text}
+}
+
+func (c *CommentNode) String() string {
+       var sb strings.Builder
+       c.writeTo(&sb)
+       return sb.String()
+}
+
+func (c *CommentNode) writeTo(sb *strings.Builder) {
+       sb.WriteString("{{")
+       sb.WriteString(c.Text)
+       sb.WriteString("}}")
+}
+
+func (c *CommentNode) tree() *Tree {
+       return c.tr
+}
+
+func (c *CommentNode) Copy() Node {
+       return &CommentNode{tr: c.tr, NodeType: NodeComment, Pos: c.Pos, Text: c.Text}
+}
+
 // PipeNode holds a pipeline with optional declaration
 type PipeNode struct {
        NodeType
@@ -349,7 +382,7 @@ func (i *IdentifierNode) Copy() Node {
        return NewIdentifier(i.Ident).SetTree(i.tr).SetPos(i.Pos)
 }
 
-// AssignNode holds a list of variable names, possibly with chained field
+// VariableNode holds a list of variable names, possibly with chained field
 // accesses. The dollar sign is part of the (first) name.
 type VariableNode struct {
        NodeType
index c9b80f4a24836846a54bba632c75c13fe9310169..5e6e512eb4314cd9aaa6c6555ba9a2bc095916a1 100644 (file)
@@ -21,16 +21,26 @@ type Tree struct {
        Name      string    // name of the template represented by the tree.
        ParseName string    // name of the top-level template during parsing, for error messages.
        Root      *ListNode // top-level root of the tree.
+       Mode      Mode      // parsing mode.
        text      string    // text parsed to create the template (or its parent)
        // Parsing only; cleared after parse.
-       funcs     []map[string]interface{}
-       lex       *lexer
-       token     [3]item // three-token lookahead for parser.
-       peekCount int
-       vars      []string // variables defined at the moment.
-       treeSet   map[string]*Tree
+       funcs      []map[string]interface{}
+       lex        *lexer
+       token      [3]item // three-token lookahead for parser.
+       peekCount  int
+       vars       []string // variables defined at the moment.
+       treeSet    map[string]*Tree
+       actionLine int // line of left delim starting action
+       mode       Mode
 }
 
+// A mode value is a set of flags (or 0). Modes control parser behavior.
+type Mode uint
+
+const (
+       ParseComments Mode = 1 << iota // parse comments and add them to AST
+)
+
 // Copy returns a copy of the Tree. Any parsing state is discarded.
 func (t *Tree) Copy() *Tree {
        if t == nil {
@@ -178,6 +188,16 @@ func (t *Tree) expectOneOf(expected1, expected2 itemType, context string) item {
 
 // unexpected complains about the token and terminates processing.
 func (t *Tree) unexpected(token item, context string) {
+       if token.typ == itemError {
+               extra := ""
+               if t.actionLine != 0 && t.actionLine != token.line {
+                       extra = fmt.Sprintf(" in action started at %s:%d", t.ParseName, t.actionLine)
+                       if strings.HasSuffix(token.val, " action") {
+                               extra = extra[len(" in action"):] // avoid "action in action"
+                       }
+               }
+               t.errorf("%s%s", token, extra)
+       }
        t.errorf("unexpected %s in %s", token, context)
 }
 
@@ -220,7 +240,8 @@ func (t *Tree) stopParse() {
 func (t *Tree) Parse(text, leftDelim, rightDelim string, treeSet map[string]*Tree, funcs ...map[string]interface{}) (tree *Tree, err error) {
        defer t.recover(&err)
        t.ParseName = t.Name
-       t.startParse(funcs, lex(t.Name, text, leftDelim, rightDelim), treeSet)
+       emitComment := t.Mode&ParseComments != 0
+       t.startParse(funcs, lex(t.Name, text, leftDelim, rightDelim, emitComment), treeSet)
        t.text = text
        t.parse()
        t.add()
@@ -240,12 +261,14 @@ func (t *Tree) add() {
        }
 }
 
-// IsEmptyTree reports whether this tree (node) is empty of everything but space.
+// IsEmptyTree reports whether this tree (node) is empty of everything but space or comments.
 func IsEmptyTree(n Node) bool {
        switch n := n.(type) {
        case nil:
                return true
        case *ActionNode:
+       case *CommentNode:
+               return true
        case *IfNode:
        case *ListNode:
                for _, node := range n.Nodes {
@@ -276,6 +299,7 @@ func (t *Tree) parse() {
                        if t.nextNonSpace().typ == itemDefine {
                                newT := New("definition") // name will be updated once we know it.
                                newT.text = t.text
+                               newT.Mode = t.Mode
                                newT.ParseName = t.ParseName
                                newT.startParse(t.funcs, t.lex, t.treeSet)
                                newT.parseDefinition()
@@ -331,19 +355,27 @@ func (t *Tree) itemList() (list *ListNode, next Node) {
 }
 
 // textOrAction:
-//     text | action
+//     text | comment | action
 func (t *Tree) textOrAction() Node {
        switch token := t.nextNonSpace(); token.typ {
        case itemText:
                return t.newText(token.pos, token.val)
        case itemLeftDelim:
+               t.actionLine = token.line
+               defer t.clearActionLine()
                return t.action()
+       case itemComment:
+               return t.newComment(token.pos, token.val)
        default:
                t.unexpected(token, "input")
        }
        return nil
 }
 
+func (t *Tree) clearActionLine() {
+       t.actionLine = 0
+}
+
 // Action:
 //     control
 //     command ("|" command)*
@@ -369,12 +401,12 @@ func (t *Tree) action() (n Node) {
        t.backup()
        token := t.peek()
        // Do not pop variables; they persist until "end".
-       return t.newAction(token.pos, token.line, t.pipeline("command"))
+       return t.newAction(token.pos, token.line, t.pipeline("command", itemRightDelim))
 }
 
 // Pipeline:
 //     declarations? command ('|' command)*
-func (t *Tree) pipeline(context string) (pipe *PipeNode) {
+func (t *Tree) pipeline(context string, end itemType) (pipe *PipeNode) {
        token := t.peekNonSpace()
        pipe = t.newPipeline(token.pos, token.line, nil)
        // Are there declarations or assignments?
@@ -415,12 +447,9 @@ decls:
        }
        for {
                switch token := t.nextNonSpace(); token.typ {
-               case itemRightDelim, itemRightParen:
+               case end:
                        // At this point, the pipeline is complete
                        t.checkPipeline(pipe, context)
-                       if token.typ == itemRightParen {
-                               t.backup()
-                       }
                        return
                case itemBool, itemCharConstant, itemComplex, itemDot, itemField, itemIdentifier,
                        itemNumber, itemNil, itemRawString, itemString, itemVariable, itemLeftParen:
@@ -449,7 +478,7 @@ func (t *Tree) checkPipeline(pipe *PipeNode, context string) {
 
 func (t *Tree) parseControl(allowElseIf bool, context string) (pos Pos, line int, pipe *PipeNode, list, elseList *ListNode) {
        defer t.popVars(len(t.vars))
-       pipe = t.pipeline(context)
+       pipe = t.pipeline(context, itemRightDelim)
        var next Node
        list, next = t.itemList()
        switch next.Type() {
@@ -535,10 +564,11 @@ func (t *Tree) blockControl() Node {
 
        token := t.nextNonSpace()
        name := t.parseTemplateName(token, context)
-       pipe := t.pipeline(context)
+       pipe := t.pipeline(context, itemRightDelim)
 
        block := New(name) // name will be updated once we know it.
        block.text = t.text
+       block.Mode = t.Mode
        block.ParseName = t.ParseName
        block.startParse(t.funcs, t.lex, t.treeSet)
        var end Node
@@ -564,7 +594,7 @@ func (t *Tree) templateControl() Node {
        if t.nextNonSpace().typ != itemRightDelim {
                t.backup()
                // Do not pop variables; they persist until "end".
-               pipe = t.pipeline(context)
+               pipe = t.pipeline(context, itemRightDelim)
        }
        return t.newTemplate(token.pos, token.line, name, pipe)
 }
@@ -598,13 +628,12 @@ func (t *Tree) command() *CommandNode {
                switch token := t.next(); token.typ {
                case itemSpace:
                        continue
-               case itemError:
-                       t.errorf("%s", token.val)
                case itemRightDelim, itemRightParen:
                        t.backup()
                case itemPipe:
+                       // nothing here; break loop below
                default:
-                       t.errorf("unexpected %s in operand", token)
+                       t.unexpected(token, "operand")
                }
                break
        }
@@ -659,8 +688,6 @@ func (t *Tree) operand() Node {
 // A nil return means the next item is not a term.
 func (t *Tree) term() Node {
        switch token := t.nextNonSpace(); token.typ {
-       case itemError:
-               t.errorf("%s", token.val)
        case itemIdentifier:
                if !t.hasFunction(token.val) {
                        t.errorf("function %q not defined", token.val)
@@ -683,11 +710,7 @@ func (t *Tree) term() Node {
                }
                return number
        case itemLeftParen:
-               pipe := t.pipeline("parenthesized pipeline")
-               if token := t.next(); token.typ != itemRightParen {
-                       t.errorf("unclosed right paren: unexpected %s", token)
-               }
-               return pipe
+               return t.pipeline("parenthesized pipeline", itemRightParen)
        case itemString, itemRawString:
                s, err := strconv.Unquote(token.val)
                if err != nil {
index 79e7bb5ae5faeb0830a65f692c6cf31b8c9f7dcd..0433a87db83c10f2a5ae3f4332c2938336f7df35 100644 (file)
@@ -252,6 +252,13 @@ var parseTests = []parseTest{
        {"comment trim left and right", "x \r\n\t{{- /* */ -}}\n\n\ty", noError, `"x""y"`},
        {"block definition", `{{block "foo" .}}hello{{end}}`, noError,
                `{{template "foo" .}}`},
+
+       {"newline in assignment", "{{ $x \n := \n 1 \n }}", noError, "{{$x := 1}}"},
+       {"newline in empty action", "{{\n}}", hasError, "{{\n}}"},
+       {"newline in pipeline", "{{\n\"x\"\n|\nprintf\n}}", noError, `{{"x" | printf}}`},
+       {"newline in comment", "{{/*\nhello\n*/}}", noError, ""},
+       {"newline in comment", "{{-\n/*\nhello\n*/\n-}}", noError, ""},
+
        // Errors.
        {"unclosed action", "hello{{range", hasError, ""},
        {"unmatched end", "{{end}}", hasError, ""},
@@ -350,6 +357,30 @@ func TestParseCopy(t *testing.T) {
        testParse(true, t)
 }
 
+func TestParseWithComments(t *testing.T) {
+       textFormat = "%q"
+       defer func() { textFormat = "%s" }()
+       tests := [...]parseTest{
+               {"comment", "{{/*\n\n\n*/}}", noError, "{{/*\n\n\n*/}}"},
+               {"comment trim left", "x \r\n\t{{- /* hi */}}", noError, `"x"{{/* hi */}}`},
+               {"comment trim right", "{{/* hi */ -}}\n\n\ty", noError, `{{/* hi */}}"y"`},
+               {"comment trim left and right", "x \r\n\t{{- /* */ -}}\n\n\ty", noError, `"x"{{/* */}}"y"`},
+       }
+       for _, test := range tests {
+               t.Run(test.name, func(t *testing.T) {
+                       tr := New(test.name)
+                       tr.Mode = ParseComments
+                       tmpl, err := tr.Parse(test.input, "", "", make(map[string]*Tree))
+                       if err != nil {
+                               t.Errorf("%q: expected error; got none", test.name)
+                       }
+                       if result := tmpl.Root.String(); result != test.result {
+                               t.Errorf("%s=(%q): got\n\t%v\nexpected\n\t%v", test.name, test.input, result, test.result)
+                       }
+               })
+       }
+}
+
 type isEmptyTest struct {
        name  string
        input string
@@ -360,6 +391,7 @@ var isEmptyTests = []isEmptyTest{
        {"empty", ``, true},
        {"nonempty", `hello`, false},
        {"spaces only", " \t\n \t\n", true},
+       {"comment only", "{{/* comment */}}", true},
        {"definition", `{{define "x"}}something{{end}}`, true},
        {"definitions and space", "{{define `x`}}something{{end}}\n\n{{define `y`}}something{{end}}\n\n", true},
        {"definitions and text", "{{define `x`}}something{{end}}\nx\n{{define `y`}}something{{end}}\ny\n", false},
@@ -403,23 +435,38 @@ var errorTests = []parseTest{
        // Check line numbers are accurate.
        {"unclosed1",
                "line1\n{{",
-               hasError, `unclosed1:2: unexpected unclosed action in command`},
+               hasError, `unclosed1:2: unclosed action`},
        {"unclosed2",
                "line1\n{{define `x`}}line2\n{{",
-               hasError, `unclosed2:3: unexpected unclosed action in command`},
+               hasError, `unclosed2:3: unclosed action`},
+       {"unclosed3",
+               "line1\n{{\"x\"\n\"y\"\n",
+               hasError, `unclosed3:4: unclosed action started at unclosed3:2`},
+       {"unclosed4",
+               "{{\n\n\n\n\n",
+               hasError, `unclosed4:6: unclosed action started at unclosed4:1`},
+       {"var1",
+               "line1\n{{\nx\n}}",
+               hasError, `var1:3: function "x" not defined`},
        // Specific errors.
        {"function",
                "{{foo}}",
                hasError, `function "foo" not defined`},
-       {"comment",
+       {"comment1",
                "{{/*}}",
-               hasError, `unclosed comment`},
+               hasError, `comment1:1: unclosed comment`},
+       {"comment2",
+               "{{/*\nhello\n}}",
+               hasError, `comment2:1: unclosed comment`},
        {"lparen",
                "{{.X (1 2 3}}",
                hasError, `unclosed left paren`},
        {"rparen",
-               "{{.X 1 2 3)}}",
-               hasError, `unexpected ")"`},
+               "{{.X 1 2 3 ) }}",
+               hasError, `unexpected ")" in command`},
+       {"rparen2",
+               "{{(.X 1 2 3",
+               hasError, `unclosed action`},
        {"space",
                "{{`x`3}}",
                hasError, `in operand`},
@@ -465,7 +512,7 @@ var errorTests = []parseTest{
                hasError, `missing value for parenthesized pipeline`},
        {"multilinerawstring",
                "{{ $v := `\n` }} {{",
-               hasError, `multilinerawstring:2: unexpected unclosed action`},
+               hasError, `multilinerawstring:2: unclosed action`},
        {"rangeundefvar",
                "{{range $k}}{{end}}",
                hasError, `undefined variable`},