diff options
author | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2017-03-26 19:34:30 +0200 |
---|---|---|
committer | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2017-03-27 15:43:56 +0200 |
commit | 930a3df1b79d21f005a63918b89cc01ae5a4cd9e (patch) | |
tree | 07955b204d91ee67857beb1af2a6d49da4386086 /output/layout_test.go | |
parent | e49a2b83ad9825a978ecbf0ff5fd9b7331690c17 (diff) |
hugolib, output: Restrict Render to regular Pages
Using it for list pages doesn't work and has potential weird side-effects.
The user probably meant to range over .Site.ReqularPages, and that is now marked clearly in the log.
Diffstat (limited to 'output/layout_test.go')
-rw-r--r-- | output/layout_test.go | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/output/layout_test.go b/output/layout_test.go index b38bd9c13..e59a16fcb 100644 --- a/output/layout_test.go +++ b/output/layout_test.go @@ -68,8 +68,9 @@ func TestLayout(t *testing.T) { t.Run(this.name, func(t *testing.T) { l := NewLayoutHandler(this.hasTheme) - layouts := l.For(this.d, this.layoutOverride, this.tp) + layouts, err := l.For(this.d, this.layoutOverride, this.tp) + require.NoError(t, err) require.NotNil(t, layouts) require.True(t, len(layouts) >= len(this.expect)) // Not checking the complete list for now ... @@ -83,6 +84,10 @@ func TestLayout(t *testing.T) { }) } + l := NewLayoutHandler(false) + _, err := l.For(LayoutDescriptor{Kind: "taxonomyTerm", Section: "tag"}, "override", RSSFormat) + require.Error(t, err) + } func BenchmarkLayout(b *testing.B) { @@ -90,7 +95,8 @@ func BenchmarkLayout(b *testing.B) { l := NewLayoutHandler(false) for i := 0; i < b.N; i++ { - layouts := l.For(descriptor, "", HTMLFormat) + layouts, err := l.For(descriptor, "", HTMLFormat) + require.NoError(b, err) require.NotEmpty(b, layouts) } } |