Skip to content

Commit

Permalink
tpl/cast: Handle template.HTML and friends in ToInt
Browse files Browse the repository at this point in the history
Also add tests for ToInt and ToString.

Resolves #3308
  • Loading branch information
moorereason authored and bep committed May 22, 2017
1 parent f41f728 commit 4113693
Show file tree
Hide file tree
Showing 2 changed files with 97 additions and 0 deletions.
14 changes: 14 additions & 0 deletions tpl/cast/cast.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,8 @@
package cast

import (
"html/template"

_cast "github.com/spf13/cast"
)

Expand All @@ -28,6 +30,18 @@ type Namespace struct {

// ToInt converts the given value to an int.
func (ns *Namespace) ToInt(v interface{}) (int, error) {
switch vv := v.(type) {
case template.HTML:
v = string(vv)
case template.CSS:
v = string(vv)
case template.HTMLAttr:
v = string(vv)
case template.JS:
v = string(vv)
case template.JSStr:
v = string(vv)
}
return _cast.ToIntE(v)
}

Expand Down
83 changes: 83 additions & 0 deletions tpl/cast/cast_test.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,83 @@
// Copyright 2017 The Hugo Authors. All rights reserved.
//
// Licensed under the Apache License, Version 2.0 (the "License");
// you may not use this file except in compliance with the License.
// You may obtain a copy of the License at
// http://www.apache.org/licenses/LICENSE-2.0
//
// Unless required by applicable law or agreed to in writing, software
// distributed under the License is distributed on an "AS IS" BASIS,
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
// See the License for the specific language governing permissions and
// limitations under the License.

package cast

import (
"fmt"
"html/template"
"testing"

"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
)

func TestToInt(t *testing.T) {
t.Parallel()

ns := New()

for i, test := range []struct {
v interface{}
expect interface{}
}{
{"1", 1},
{template.HTML("2"), 2},
{template.CSS("3"), 3},
{template.HTMLAttr("4"), 4},
{template.JS("5"), 5},
{template.JSStr("6"), 6},
{"a", false},
{t, false},
} {
errMsg := fmt.Sprintf("[%d] %v", i, test.v)

result, err := ns.ToInt(test.v)

if b, ok := test.expect.(bool); ok && !b {
require.Error(t, err, errMsg)
continue
}

require.NoError(t, err, errMsg)
assert.Equal(t, test.expect, result, errMsg)
}
}

func TestToString(t *testing.T) {
t.Parallel()

ns := New()

for i, test := range []struct {
v interface{}
expect interface{}
}{
{1, "1"},
{template.HTML("2"), "2"},
{"a", "a"},
{t, false},
} {
errMsg := fmt.Sprintf("[%d] %v", i, test.v)

result, err := ns.ToString(test.v)

if b, ok := test.expect.(bool); ok && !b {
require.Error(t, err, errMsg)
continue
}

require.NoError(t, err, errMsg)
assert.Equal(t, test.expect, result, errMsg)
}
}

0 comments on commit 4113693

Please sign in to comment.