summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--helpers/general_test.go12
-rw-r--r--hugolib/pagination_test.go2
-rw-r--r--hugolib/path_separators_test.go8
-rw-r--r--hugolib/path_separators_windows_test.go2
-rw-r--r--hugolib/shortcode_test.go10
-rw-r--r--parser/frontmatter_test.go2
-rw-r--r--tpl/template_funcs_test.go15
7 files changed, 26 insertions, 25 deletions
diff --git a/helpers/general_test.go b/helpers/general_test.go
index 0147822c7..31c478200 100644
--- a/helpers/general_test.go
+++ b/helpers/general_test.go
@@ -28,7 +28,7 @@ func TestGuessType(t *testing.T) {
} {
result := GuessType(this.in)
if result != this.expect {
- t.Errorf("[%d] GuessType guessed wrong, expected %s, got %s", i, this.expect, result)
+ t.Errorf("[%d] got %s but expected %s", i, result, this.expect)
}
}
}
@@ -109,7 +109,7 @@ func TestReaderContains(t *testing.T) {
for i, this := range append(containsBenchTestData, containsAdditionalTestData...) {
result := ReaderContains(StringToReader(this.v1), this.v2)
if result != this.expect {
- t.Errorf("[%d] Got %t but expected %t", i, result, this.expect)
+ t.Errorf("[%d] got %t but expected %t", i, result, this.expect)
}
}
@@ -123,7 +123,7 @@ func BenchmarkReaderContains(b *testing.B) {
for i, this := range containsBenchTestData {
result := ReaderContains(StringToReader(this.v1), this.v2)
if result != this.expect {
- b.Errorf("[%d] Got %t but expected %t", i, result, this.expect)
+ b.Errorf("[%d] got %t but expected %t", i, result, this.expect)
}
}
}
@@ -140,7 +140,7 @@ func _BenchmarkReaderContains(b *testing.B) {
}
result := bytes.Contains(bs, this.v2)
if result != this.expect {
- b.Errorf("[%d] Got %t but expected %t", i, result, this.expect)
+ b.Errorf("[%d] got %t but expected %t", i, result, this.expect)
}
}
}
@@ -167,10 +167,10 @@ func TestInStringArrayCaseSensitive(t *testing.T) {
{"Albert", true},
{"ALBERT", false},
}
- for _, in := range data {
+ for i, in := range data {
output := InStringArray(array, in.input)
if output != in.expected {
- t.Errorf("TestInStringArrayCase failed. Expected %t. Got %t.", in.expected, output)
+ t.Errorf("[%d] got %t but expected %t", i, output, in.expected)
}
}
}
diff --git a/hugolib/pagination_test.go b/hugolib/pagination_test.go
index f42197ba7..5d7dcc3e5 100644
--- a/hugolib/pagination_test.go
+++ b/hugolib/pagination_test.go
@@ -301,7 +301,7 @@ func TestProbablyEqualPageLists(t *testing.T) {
result := probablyEqualPageLists(this.v1, this.v2)
if result != this.expect {
- t.Errorf("[%d] Got %t but expected %t", i, result, this.expect)
+ t.Errorf("[%d] got %t but expected %t", i, result, this.expect)
}
}
diff --git a/hugolib/path_separators_test.go b/hugolib/path_separators_test.go
index e9af0e922..b45d5f331 100644
--- a/hugolib/path_separators_test.go
+++ b/hugolib/path_separators_test.go
@@ -34,13 +34,13 @@ func TestNewPageWithFilePath(t *testing.T) {
{filepath.Join("content", "dub", "sub", "foobar.html"), "dub", L("dub/single.html", "_default/single.html")},
}
- for _, el := range toCheck {
+ for i, el := range toCheck {
p, err := NewPageFrom(strings.NewReader(SIMPLE_PAGE_YAML), el.input)
if err != nil {
- t.Errorf("Reading from SIMPLE_PAGE_YAML resulted in an error: %s", err)
+ t.Errorf("[%d] Reading from SIMPLE_PAGE_YAML resulted in an error: %s", i, err)
}
if p.Section() != el.section {
- t.Errorf("Section not set to %s for page %s. Got: %s", el.section, el.input, p.Section())
+ t.Errorf("[%d] Section incorrect page %s. got %s but expected %s", i, el.input, p.Section(), el.section)
}
for _, y := range el.layout {
@@ -48,7 +48,7 @@ func TestNewPageWithFilePath(t *testing.T) {
}
if !listEqual(p.Layout(), el.layout) {
- t.Errorf("Layout incorrect. Expected: '%s', Got: '%s'", el.layout, p.Layout())
+ t.Errorf("[%d] Layout incorrect. got '%s' but expected '%s'", i, p.Layout(), el.layout)
}
}
}
diff --git a/hugolib/path_separators_windows_test.go b/hugolib/path_separators_windows_test.go
index 4d8e0c48b..8b2cbacfe 100644
--- a/hugolib/path_separators_windows_test.go
+++ b/hugolib/path_separators_windows_test.go
@@ -13,6 +13,6 @@ const (
func TestTemplatePathSeparator(t *testing.T) {
tmpl := new(tpl.GoHTMLTemplate)
if name := tmpl.GenerateTemplateNameFrom(win_base, win_path); name != "sub1/index.html" {
- t.Fatalf("Template name incorrect. Expected: %s, Got: %s", "sub1/index.html", name)
+ t.Fatalf("Template name incorrect. got %s but expected %s", name, "sub1/index.html")
}
}
diff --git a/hugolib/shortcode_test.go b/hugolib/shortcode_test.go
index 742a02ffe..3aa9b1ed2 100644
--- a/hugolib/shortcode_test.go
+++ b/hugolib/shortcode_test.go
@@ -27,7 +27,7 @@ func CheckShortCodeMatch(t *testing.T, input, expected string, template tpl.Temp
}
if output != expected {
- t.Fatalf("Shortcode render didn't match. Expected: %q, Got: %q", expected, output)
+ t.Fatalf("Shortcode render didn't match. got %q but exxpected %q", output, expected)
}
}
@@ -260,8 +260,8 @@ func TestExtractShortcodes(t *testing.T) {
} else {
r, _ := regexp.Compile(this.expectErrorMsg)
if !r.MatchString(err.Error()) {
- t.Fatalf("[%d] %s: ExtractShortcodes didn't return an expected error message, expected %s got %s",
- i, this.name, this.expectErrorMsg, err.Error())
+ t.Fatalf("[%d] %s: ExtractShortcodes didn't return an expected error message, got %s but expected %s",
+ i, this.name, err.Error(), this.expectErrorMsg)
}
}
continue
@@ -289,7 +289,7 @@ func TestExtractShortcodes(t *testing.T) {
}
if !r.MatchString(content) {
- t.Fatalf("[%d] %s: Shortcode extract didn't match. Expected: %q, Got: %q", i, this.name, expected, content)
+ t.Fatalf("[%d] %s: Shortcode extract didn't match. got %q but expected %q", i, this.name, content, expected)
}
for placeHolder, sc := range shortCodes {
@@ -305,7 +305,7 @@ func TestExtractShortcodes(t *testing.T) {
if this.expectShortCodes != "" {
shortCodesAsStr := fmt.Sprintf("map%q", collectAndSortShortcodes(shortCodes))
if !strings.Contains(shortCodesAsStr, this.expectShortCodes) {
- t.Fatalf("[%d] %s: Short codes not as expected, got %s - expected to contain %s", i, this.name, shortCodesAsStr, this.expectShortCodes)
+ t.Fatalf("[%d] %s: Short codes not as expected, got %s but expected %s", i, this.name, shortCodesAsStr, this.expectShortCodes)
}
}
}
diff --git a/parser/frontmatter_test.go b/parser/frontmatter_test.go
index 33db0a24a..c9c3c0eaa 100644
--- a/parser/frontmatter_test.go
+++ b/parser/frontmatter_test.go
@@ -19,7 +19,7 @@ func TestFormatToLeadRune(t *testing.T) {
result := FormatToLeadRune(this.kind)
if result != this.expect {
- t.Errorf("[%d] Got %q but expected %q", i, result, this.expect)
+ t.Errorf("[%d] got %q but expected %q", i, result, this.expect)
}
}
}
diff --git a/tpl/template_funcs_test.go b/tpl/template_funcs_test.go
index 3a51e741d..b1d611c11 100644
--- a/tpl/template_funcs_test.go
+++ b/tpl/template_funcs_test.go
@@ -271,7 +271,7 @@ func TestIn(t *testing.T) {
result := In(this.v1, this.v2)
if result != this.expect {
- t.Errorf("[%d] Got %v but expected %v", i, result, this.expect)
+ t.Errorf("[%d] got %v but expected %v", i, result, this.expect)
}
}
}
@@ -310,7 +310,7 @@ func TestSlicestr(t *testing.T) {
continue
}
if !reflect.DeepEqual(result, this.expect) {
- t.Errorf("[%d] Got %s but expected %s", i, result, this.expect)
+ t.Errorf("[%d] got %s but expected %s", i, result, this.expect)
}
}
}
@@ -350,7 +350,7 @@ func TestSubstr(t *testing.T) {
continue
}
if !reflect.DeepEqual(result, this.expect) {
- t.Errorf("[%d] Got %s but expected %s", i, result, this.expect)
+ t.Errorf("[%d] got %s but expected %s", i, result, this.expect)
}
}
}
@@ -380,7 +380,7 @@ func TestSplit(t *testing.T) {
continue
}
if !reflect.DeepEqual(result, this.expect) {
- t.Errorf("[%d] Got %s but expected %s", i, result, this.expect)
+ t.Errorf("[%d] got %s but expected %s", i, result, this.expect)
}
}
}
@@ -413,20 +413,20 @@ func TestIntersect(t *testing.T) {
continue
}
if !reflect.DeepEqual(results, this.expect) {
- t.Errorf("[%d] Got %v but expected %v", i, results, this.expect)
+ t.Errorf("[%d] got %v but expected %v", i, results, this.expect)
}
}
_, err1 := Intersect("not an array or slice", []string{"a"})
if err1 == nil {
- t.Error("Excpected error for non array as first arg")
+ t.Error("Expected error for non array as first arg")
}
_, err2 := Intersect([]string{"a"}, "not an array or slice")
if err2 == nil {
- t.Error("Excpected error for non array as second arg")
+ t.Error("Expected error for non array as second arg")
}
}
@@ -1225,3 +1225,4 @@ func TestSafeURL(t *testing.T) {
}
}
}
+