shithub: hugo

Download patch

ref: 5374242ff7b6cc1677baf9a1dd2b835d8ee6d18f
parent: c510140c0c55394d5b699c62f60df53ef8a164c7
author: Noah Campbell <[email protected]>
date: Wed Sep 18 07:52:30 EDT 2013

More expressive --check output

--- a/hugolib/planner.go
+++ b/hugolib/planner.go
@@ -14,11 +14,16 @@
 		fmt.Fprintf(out, "%s", p.FileName)
 		if p.IsRenderable() {
 			fmt.Fprintf(out, " (renderer: markdown)")
+		} else {
+			fmt.Fprintf(out, " (renderer: n/a)")
 		}
+		if s.Tmpl != nil {
+		fmt.Fprintf(out, " (layout: %s, exists: %t)", p.Layout(), s.Tmpl.Lookup(p.Layout()) != nil)
+	}
 		fmt.Fprintf(out, "\n")
 		fmt.Fprintf(out, " canonical => ")
 		if s.Target == nil {
-			fmt.Fprintf(out, "%s\n", "!no target specified!")
+			fmt.Fprintf(out, "%s\n\n", "!no target specified!")
 			continue
 		}
 
@@ -39,6 +44,7 @@
 			}
 			fmt.Fprintf(out, " %s => %s\n", alias, aliasTrans)
 		}
+		fmt.Fprintln(out)
 	}
 	return
 }
--- a/hugolib/site_show_plan_test.go
+++ b/hugolib/site_show_plan_test.go
@@ -17,6 +17,7 @@
 var fakeSource = []byteSource{
 	{"foo/bar/file.md", []byte(SIMPLE_PAGE)},
 	{"alias/test/file1.md", []byte(ALIAS_DOC_1)},
+	{"section/somecontent.html", []byte(RENDER_NO_FRONT_MATTER)},
 }
 
 type inMemorySource struct {
@@ -54,8 +55,9 @@
 		Source: &inMemorySource{fakeSource},
 	}
 	must(s.CreatePages())
-	expected := "foo/bar/file.md (renderer: markdown)\n canonical => !no target specified!\n" +
-		"alias/test/file1.md (renderer: markdown)\n canonical => !no target specified!\n"
+	expected := "foo/bar/file.md (renderer: markdown)\n canonical => !no target specified!\n\n" +
+		"alias/test/file1.md (renderer: markdown)\n canonical => !no target specified!\n\n" +
+		"section/somecontent.html (renderer: n/a)\n canonical => !no target specified!\n\n"
 	checkShowPlanExpected(t, s, expected)
 }
 
@@ -66,11 +68,13 @@
 		Alias:  new(target.HTMLRedirectAlias),
 	}
 	must(s.CreatePages())
-	expected := "foo/bar/file.md (renderer: markdown)\n canonical => foo/bar/file/index.html\n" +
+	expected := "foo/bar/file.md (renderer: markdown)\n canonical => foo/bar/file/index.html\n\n" +
 		"alias/test/file1.md (renderer: markdown)\n" +
 		" canonical => alias/test/file1/index.html\n" +
 		" alias1/ => alias1/index.html\n" +
-		" alias-2/ => alias-2/index.html\n"
+		" alias-2/ => alias-2/index.html\n\n" +
+		"section/somecontent.html (renderer: n/a)\n canonical => section/somecontent/index.html\n\n"
+
 	checkShowPlanExpected(t, s, expected)
 }
 
@@ -81,11 +85,12 @@
 		Alias:  new(target.HTMLRedirectAlias),
 	}
 	s.CreatePages()
-	expected := "foo/bar/file.md (renderer: markdown)\n canonical => foo/bar/file.html\n" +
+	expected := "foo/bar/file.md (renderer: markdown)\n canonical => foo/bar/file.html\n\n" +
 		"alias/test/file1.md (renderer: markdown)\n" +
 		" canonical => alias/test/file1.html\n" +
 		" alias1/ => alias1/index.html\n" +
-		" alias-2/ => alias-2/index.html\n"
+		" alias-2/ => alias-2/index.html\n\n" +
+		"section/somecontent.html (renderer: n/a)\n canonical => section/somecontent.html\n\n"
 	checkShowPlanExpected(t, s, expected)
 }
 
@@ -97,10 +102,11 @@
 	}
 
 	must(s.CreatePages())
-	expected := "foo/bar/file.md (renderer: markdown)\n canonical => ../public/foo/bar/file/index.html\n" +
+	expected := "foo/bar/file.md (renderer: markdown)\n canonical => ../public/foo/bar/file/index.html\n\n" +
 		"alias/test/file1.md (renderer: markdown)\n" +
 		" canonical => ../public/alias/test/file1/index.html\n" +
 		" alias1/ => ../public/alias1/index.html\n" +
-		" alias-2/ => ../public/alias-2/index.html\n"
+		" alias-2/ => ../public/alias-2/index.html\n\n" +
+		"section/somecontent.html (renderer: n/a)\n canonical => ../public/section/somecontent/index.html\n\n"
 	checkShowPlanExpected(t, s, expected)
 }