hugolib: Camelcase ALL_CAPS vars and const in tests
authorbogem <albertnigma@gmail.com>
Mon, 21 Nov 2016 22:02:19 +0000 (23:02 +0100)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Wed, 23 Nov 2016 08:13:00 +0000 (09:13 +0100)
See #2014

hugolib/path_separators_test.go
hugolib/sitemap_test.go

index 013837279ac1b94d9b927a23bef176aa4730bcba..1597184401e42b9b6621c3123478d907f556332d 100644 (file)
@@ -19,14 +19,14 @@ import (
        "testing"
 )
 
-var SIMPLE_PAGE_YAML = `---
+var simplePageYAML = `---
 contenttype: ""
 ---
 Sample Text
 `
 
 func TestDegenerateMissingFolderInPageFilename(t *testing.T) {
-       p, err := NewPageFrom(strings.NewReader(SIMPLE_PAGE_YAML), filepath.Join("foobar"))
+       p, err := NewPageFrom(strings.NewReader(simplePageYAML), filepath.Join("foobar"))
        if err != nil {
                t.Fatalf("Error in NewPageFrom")
        }
@@ -48,9 +48,9 @@ func TestNewPageWithFilePath(t *testing.T) {
        }
 
        for i, el := range toCheck {
-               p, err := NewPageFrom(strings.NewReader(SIMPLE_PAGE_YAML), el.input)
+               p, err := NewPageFrom(strings.NewReader(simplePageYAML), el.input)
                if err != nil {
-                       t.Errorf("[%d] Reading from SIMPLE_PAGE_YAML resulted in an error: %s", i, err)
+                       t.Errorf("[%d] Reading from simplePageYAML resulted in an error: %s", i, err)
                }
                if p.Section() != el.section {
                        t.Errorf("[%d] Section incorrect page %s. got %s but expected %s", i, el.input, p.Section(), el.section)
index 72fcb2048c902e00b063b31989687036c554ca66..aba47ae7dddf3ce6544fa083915932592b9697a9 100644 (file)
@@ -23,7 +23,7 @@ import (
        "github.com/spf13/viper"
 )
 
-const SITEMAP_TEMPLATE = `<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
+const sitemapTemplate = `<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
   {{ range .Data.Pages }}
   <url>
     <loc>{{ .Permalink }}</loc>
@@ -56,7 +56,7 @@ func doTestSitemapOutput(t *testing.T, internal bool) {
                }
 
        } else {
-               if err := buildAndRenderSite(s, "sitemap.xml", SITEMAP_TEMPLATE); err != nil {
+               if err := buildAndRenderSite(s, "sitemap.xml", sitemapTemplate); err != nil {
                        t.Fatalf("Failed to build site: %s", err)
                }
        }