From 29aa9baecd2242bc86706c3fc09bc32cf854cff0 Mon Sep 17 00:00:00 2001 From: Anthony Fok Date: Tue, 8 Sep 2015 23:27:55 -0600 Subject: [PATCH] Complete the Url-to-URL transition in tests See #959 --- hugolib/site_test.go | 38 +++++++++++++++++++------------------- utils/utils_test.go | 2 +- 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/hugolib/site_test.go b/hugolib/site_test.go index 19df704d6..705abe0c1 100644 --- a/hugolib/site_test.go +++ b/hugolib/site_test.go @@ -279,26 +279,26 @@ func TestDraftAndFutureRender(t *testing.T) { // Issue #957 func TestCrossrefs(t *testing.T) { hugofs.DestinationFS = new(afero.MemMapFs) - for _, uglyUrls := range []bool{true, false} { + for _, uglyURLs := range []bool{true, false} { for _, relative := range []bool{true, false} { - doTestCrossrefs(t, relative, uglyUrls) + doTestCrossrefs(t, relative, uglyURLs) } } } -func doTestCrossrefs(t *testing.T, relative, uglyUrls bool) { +func doTestCrossrefs(t *testing.T, relative, uglyURLs bool) { viper.Reset() defer viper.Reset() - baseUrl := "http://foo/bar" + baseURL := "http://foo/bar" viper.Set("DefaultExtension", "html") - viper.Set("baseurl", baseUrl) - viper.Set("UglyURLs", uglyUrls) + viper.Set("baseurl", baseURL) + viper.Set("UglyURLs", uglyURLs) viper.Set("verbose", true) var refShortcode string var expectedBase string - var expectedUrlSuffix string + var expectedURLSuffix string var expectedPathSuffix string if relative { @@ -306,14 +306,14 @@ func doTestCrossrefs(t *testing.T, relative, uglyUrls bool) { expectedBase = "/bar" } else { refShortcode = "ref" - expectedBase = baseUrl + expectedBase = baseURL } - if uglyUrls { - expectedUrlSuffix = ".html" + if uglyURLs { + expectedURLSuffix = ".html" expectedPathSuffix = ".html" } else { - expectedUrlSuffix = "/" + expectedURLSuffix = "/" expectedPathSuffix = "/index.html" } @@ -331,7 +331,7 @@ THE END.`, refShortcode))}, s := &Site{ Source: &source.InMemorySource{ByteSource: sources}, - Targets: targetList{Page: &target.PagePub{UglyURLs: uglyUrls}}, + Targets: targetList{Page: &target.PagePub{UglyURLs: uglyURLs}}, } s.initializeSiteInfo() @@ -345,8 +345,8 @@ THE END.`, refShortcode))}, doc string expected string }{ - {filepath.FromSlash(fmt.Sprintf("sect/doc1%s", expectedPathSuffix)), fmt.Sprintf("

Ref 2: %s/sect/doc2%s

\n", expectedBase, expectedUrlSuffix)}, - {filepath.FromSlash(fmt.Sprintf("sect/doc2%s", expectedPathSuffix)), fmt.Sprintf("

Ref 1:

\n\n%s/sect/doc1%s\n\n

THE END.

\n", expectedBase, expectedUrlSuffix)}, + {filepath.FromSlash(fmt.Sprintf("sect/doc1%s", expectedPathSuffix)), fmt.Sprintf("

Ref 2: %s/sect/doc2%s

\n", expectedBase, expectedURLSuffix)}, + {filepath.FromSlash(fmt.Sprintf("sect/doc2%s", expectedPathSuffix)), fmt.Sprintf("

Ref 1:

\n\n%s/sect/doc1%s\n\n

THE END.

\n", expectedBase, expectedURLSuffix)}, } for _, test := range tests { @@ -366,14 +366,14 @@ THE END.`, refShortcode))}, } // Issue #939 -func Test404ShouldAlwaysHaveUglyUrls(t *testing.T) { +func Test404ShouldAlwaysHaveUglyURLs(t *testing.T) { hugofs.DestinationFS = new(afero.MemMapFs) for _, uglyURLs := range []bool{true, false} { - doTest404ShouldAlwaysHaveUglyUrls(t, uglyURLs) + doTest404ShouldAlwaysHaveUglyURLs(t, uglyURLs) } } -func doTest404ShouldAlwaysHaveUglyUrls(t *testing.T, uglyURLs bool) { +func doTest404ShouldAlwaysHaveUglyURLs(t *testing.T, uglyURLs bool) { viper.Reset() defer viper.Reset() @@ -590,7 +590,7 @@ func TestSkipRender(t *testing.T) { } } -func TestAbsUrlify(t *testing.T) { +func TestAbsURLify(t *testing.T) { viper.Reset() defer viper.Reset() @@ -648,7 +648,7 @@ func TestAbsUrlify(t *testing.T) { } if content != expected { - t.Errorf("AbsUrlify content expected:\n%q\ngot\n%q", expected, content) + t.Errorf("AbsURLify content expected:\n%q\ngot\n%q", expected, content) } } } diff --git a/utils/utils_test.go b/utils/utils_test.go index 7c47ef970..4ec2a4eec 100644 --- a/utils/utils_test.go +++ b/utils/utils_test.go @@ -10,7 +10,7 @@ func TestCutUsageMessage(t *testing.T) { cutMessage string }{ {"", ""}, - {" Usage of hugo: \n -b, --baseUrl=...", ""}, + {" Usage of hugo: \n -b, --baseURL=...", ""}, {"Some error Usage of hugo: \n", "Some error"}, {"Usage of hugo: \n -b --baseU", ""}, {"CRITICAL error for usage of hugo ", "CRITICAL error for usage of hugo"},