Commit 3ee70291 authored by Sam Leavens's avatar Sam Leavens

ref(pkg/chartutil): Capitalize YAML

Capitalize "yaml" in error message. Remove empty var from test error message.
parent 9b8928ed
...@@ -391,7 +391,7 @@ func istable(v interface{}) bool { ...@@ -391,7 +391,7 @@ func istable(v interface{}) bool {
// PathValue takes a yaml path with . notation and returns the value if exists // PathValue takes a yaml path with . notation and returns the value if exists
func (v Values) PathValue(ypath string) (interface{}, error) { func (v Values) PathValue(ypath string) (interface{}, error) {
if len(ypath) == 0 { if len(ypath) == 0 {
return nil, errors.New("yaml path string cannot be zero length") return nil, errors.New("YAML path string cannot be zero length")
} }
yps := strings.Split(ypath, ".") yps := strings.Split(ypath, ".")
if len(yps) == 1 { if len(yps) == 1 {
......
...@@ -447,8 +447,8 @@ chapter: ...@@ -447,8 +447,8 @@ chapter:
if _, err := d.PathValue("chapter.doesntexist.one"); err == nil { if _, err := d.PathValue("chapter.doesntexist.one"); err == nil {
t.Errorf("Non-existent key in middle of path should return error: %s\n%v", err, d) t.Errorf("Non-existent key in middle of path should return error: %s\n%v", err, d)
} }
if _, err := d.PathValue(""); err == nil { if _, err := d.PathValue(""); err != nil {
t.Errorf("Empty path should return error: %s\n%v", err, d) t.Errorf("Empty path should return error: \n%v", d)
} }
if v, err := d.PathValue("title"); err == nil { if v, err := d.PathValue("title"); err == nil {
if v != "Moby Dick" { if v != "Moby Dick" {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment