Change-Id: I1da1d718609eb6a7b78d29b173ec780bde22c687
Reviewed-on: https://go-review.googlesource.com/16422
Reviewed-by: Ralph Corderoy <ralph@inputplus.co.uk>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Andrew Gerrand <adg@golang.org>
        }
        for i, test := range nilValueTestData {
                if _, err := Marshal(test); err == nil {
-                       t.Fatal("#%d: successfully marshaled nil value", i)
+                       t.Fatalf("#%d: successfully marshaled nil value", i)
                }
        }
 }
 
        // Clone t0 as t4. Redefining the "lhs" template should not fail.
        t4 := Must(t0.Clone())
        if _, err := t4.Parse(`{{define "lhs"}} OK {{end}}`); err != nil {
-               t.Error(`redefine "lhs": got err %v want non-nil`, err)
+               t.Errorf(`redefine "lhs": got err %v want nil`, err)
        }
        // Cloning t1 should fail as it has been executed.
        if _, err := t1.Clone(); err == nil {