msb-public
/
helm
mirror of https://github.com/helm/helm
1
0
Code Issues Projects Releases Wiki Activity

fix(tests): prefix template names with "templates/"

Browse Source
pull/1435/head
Adnan Abdulhussein 8 years ago
parent d19dd9f3cf
commit f97dbe33da
3 changed files with 55 additions and 55 deletions
Show Stats Download Patch File Download Diff File

2
cmd/helm/helm_test.go
Unescape View File

@ -79,7 +79,7 @@ func releaseMock(opts *releaseOptions) *release.Release {
Version: "0.1.0-beta.1", Version: "0.1.0-beta.1",
}, },
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "foo.tpl", Data: []byte(mockManifest)}, {Name: "templates/foo.tpl", Data: []byte(mockManifest)},
}, },
} }
} }

62
cmd/tiller/release_server_test.go
Unescape View File

@ -85,12 +85,12 @@ func chartStub() *chart.Chart {
}, },
// This adds basic templates, partials, and hooks. // This adds basic templates, partials, and hooks.
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "goodbye", Data: []byte("goodbye: world")}, {Name: "templates/goodbye", Data: []byte("goodbye: world")},
{Name: "empty", Data: []byte("")}, {Name: "templates/empty", Data: []byte("")},
{Name: "with-partials", Data: []byte(`hello: {{ template "_planet" . }}`)}, {Name: "templates/with-partials", Data: []byte(`hello: {{ template "_planet" . }}`)},
{Name: "partials/_planet", Data: []byte(`{{define "_planet"}}Earth{{end}}`)}, {Name: "templates/partials/_planet", Data: []byte(`{{define "_planet"}}Earth{{end}}`)},
{Name: "hooks", Data: []byte(manifestWithHook)}, {Name: "templates/hooks", Data: []byte(manifestWithHook)},
}, },
} }
} }
@ -212,8 +212,8 @@ func TestInstallRelease(t *testing.T) {
Chart: &chart.Chart{ Chart: &chart.Chart{
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "hooks", Data: []byte(manifestWithHook)}, {Name: "templates/hooks", Data: []byte(manifestWithHook)},
}, },
}, },
} }
@ -257,7 +257,7 @@ func TestInstallRelease(t *testing.T) {
t.Errorf("Expected manifest in %v", res) t.Errorf("Expected manifest in %v", res)
} }
if !strings.Contains(rel.Manifest, "---\n# Source: hello/hello\nhello: world") { if !strings.Contains(rel.Manifest, "---\n# Source: hello/templates/hello\nhello: world") {
t.Errorf("unexpected output: %s", rel.Manifest) t.Errorf("unexpected output: %s", rel.Manifest)
} }
} }
@ -272,9 +272,9 @@ func TestInstallReleaseWithNotes(t *testing.T) {
Chart: &chart.Chart{ Chart: &chart.Chart{
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "hooks", Data: []byte(manifestWithHook)}, {Name: "templates/hooks", Data: []byte(manifestWithHook)},
{Name: "NOTES.txt", Data: []byte(notesText)}, {Name: "templates/NOTES.txt", Data: []byte(notesText)},
}, },
}, },
} }
@ -322,7 +322,7 @@ func TestInstallReleaseWithNotes(t *testing.T) {
t.Errorf("Expected manifest in %v", res) t.Errorf("Expected manifest in %v", res)
} }
if !strings.Contains(rel.Manifest, "---\n# Source: hello/hello\nhello: world") { if !strings.Contains(rel.Manifest, "---\n# Source: hello/templates/hello\nhello: world") {
t.Errorf("unexpected output: %s", rel.Manifest) t.Errorf("unexpected output: %s", rel.Manifest)
} }
} }
@ -337,9 +337,9 @@ func TestInstallReleaseWithNotesRendered(t *testing.T) {
Chart: &chart.Chart{ Chart: &chart.Chart{
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "hooks", Data: []byte(manifestWithHook)}, {Name: "templates/hooks", Data: []byte(manifestWithHook)},
{Name: "NOTES.txt", Data: []byte(notesText + " {{.Release.Name}}")}, {Name: "templates/NOTES.txt", Data: []byte(notesText + " {{.Release.Name}}")},
}, },
}, },
} }
@ -388,7 +388,7 @@ func TestInstallReleaseWithNotesRendered(t *testing.T) {
t.Errorf("Expected manifest in %v", res) t.Errorf("Expected manifest in %v", res)
} }
if !strings.Contains(rel.Manifest, "---\n# Source: hello/hello\nhello: world") { if !strings.Contains(rel.Manifest, "---\n# Source: hello/templates/hello\nhello: world") {
t.Errorf("unexpected output: %s", rel.Manifest) t.Errorf("unexpected output: %s", rel.Manifest)
} }
} }
@ -403,17 +403,17 @@ func TestInstallReleaseWithChartAndDependencyNotes(t *testing.T) {
Chart: &chart.Chart{ Chart: &chart.Chart{
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "hooks", Data: []byte(manifestWithHook)}, {Name: "templates/hooks", Data: []byte(manifestWithHook)},
{Name: "NOTES.txt", Data: []byte(notesText)}, {Name: "templates/NOTES.txt", Data: []byte(notesText)},
}, },
Dependencies: []*chart.Chart{ Dependencies: []*chart.Chart{
{ {
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "hooks", Data: []byte(manifestWithHook)}, {Name: "templates/hooks", Data: []byte(manifestWithHook)},
{Name: "NOTES.txt", Data: []byte(notesText + " child")}, {Name: "templates/NOTES.txt", Data: []byte(notesText + " child")},
}, },
}, },
}, },
@ -456,11 +456,11 @@ func TestInstallReleaseDryRun(t *testing.T) {
t.Errorf("Expected release name.") t.Errorf("Expected release name.")
} }
if !strings.Contains(res.Release.Manifest, "---\n# Source: hello/hello\nhello: world") { if !strings.Contains(res.Release.Manifest, "---\n# Source: hello/templates/hello\nhello: world") {
t.Errorf("unexpected output: %s", res.Release.Manifest) t.Errorf("unexpected output: %s", res.Release.Manifest)
} }
if !strings.Contains(res.Release.Manifest, "---\n# Source: hello/goodbye\ngoodbye: world") { if !strings.Contains(res.Release.Manifest, "---\n# Source: hello/templates/goodbye\ngoodbye: world") {
t.Errorf("unexpected output: %s", res.Release.Manifest) t.Errorf("unexpected output: %s", res.Release.Manifest)
} }
@ -569,8 +569,8 @@ func TestUpdateRelease(t *testing.T) {
Chart: &chart.Chart{ Chart: &chart.Chart{
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "hooks", Data: []byte(manifestWithUpgradeHooks)}, {Name: "templates/hooks", Data: []byte(manifestWithUpgradeHooks)},
}, },
}, },
} }
@ -625,7 +625,7 @@ func TestUpdateRelease(t *testing.T) {
t.Errorf("Expected manifest in %v", res) t.Errorf("Expected manifest in %v", res)
} }
if !strings.Contains(updated.Manifest, "---\n# Source: hello/hello\nhello: world") { if !strings.Contains(updated.Manifest, "---\n# Source: hello/templates/hello\nhello: world") {
t.Errorf("unexpected output: %s", rel.Manifest) t.Errorf("unexpected output: %s", rel.Manifest)
} }
@ -647,7 +647,7 @@ func TestUpdateReleaseFailure(t *testing.T) {
Chart: &chart.Chart{ Chart: &chart.Chart{
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "something", Data: []byte("hello: world")}, {Name: "templates/something", Data: []byte("hello: world")},
}, },
}, },
} }
@ -715,8 +715,8 @@ func TestUpdateReleaseNoHooks(t *testing.T) {
Chart: &chart.Chart{ Chart: &chart.Chart{
Metadata: &chart.Metadata{Name: "hello"}, Metadata: &chart.Metadata{Name: "hello"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "hello", Data: []byte("hello: world")}, {Name: "templates/hello", Data: []byte("hello: world")},
{Name: "hooks", Data: []byte(manifestWithUpgradeHooks)}, {Name: "templates/hooks", Data: []byte(manifestWithUpgradeHooks)},
}, },
}, },
} }

46
pkg/engine/engine_test.go
Unescape View File

@ -77,9 +77,9 @@ func TestRender(t *testing.T) {
Version: "1.2.3", Version: "1.2.3",
}, },
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "test1", Data: []byte("{{.outer | title }} {{.inner | title}}")}, {Name: "templates/test1", Data: []byte("{{.outer | title }} {{.inner | title}}")},
{Name: "test2", Data: []byte("{{.global.callme | lower }}")}, {Name: "templates/test2", Data: []byte("{{.global.callme | lower }}")},
{Name: "test3", Data: []byte("{{.noValue}}")}, {Name: "templates/test3", Data: []byte("{{.noValue}}")},
}, },
Values: &chart.Config{ Values: &chart.Config{
Raw: "outer: DEFAULT\ninner: DEFAULT", Raw: "outer: DEFAULT\ninner: DEFAULT",
@ -105,16 +105,16 @@ global:
} }
expect := "Spouter Inn" expect := "Spouter Inn"
if out["moby/test1"] != expect { if out["moby/templates/test1"] != expect {
t.Errorf("Expected %q, got %q", expect, out["test1"]) t.Errorf("Expected %q, got %q", expect, out["test1"])
} }
expect = "ishmael" expect = "ishmael"
if out["moby/test2"] != expect { if out["moby/templates/test2"] != expect {
t.Errorf("Expected %q, got %q", expect, out["test2"]) t.Errorf("Expected %q, got %q", expect, out["test2"])
} }
expect = "" expect = ""
if out["moby/test3"] != expect { if out["moby/templates/test3"] != expect {
t.Errorf("Expected %q, got %q", expect, out["test3"]) t.Errorf("Expected %q, got %q", expect, out["test3"])
} }
@ -186,20 +186,20 @@ func TestAllTemplates(t *testing.T) {
ch1 := &chart.Chart{ ch1 := &chart.Chart{
Metadata: &chart.Metadata{Name: "ch1"}, Metadata: &chart.Metadata{Name: "ch1"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "foo", Data: []byte("foo")}, {Name: "templates/foo", Data: []byte("foo")},
{Name: "bar", Data: []byte("bar")}, {Name: "templates/bar", Data: []byte("bar")},
}, },
Dependencies: []*chart.Chart{ Dependencies: []*chart.Chart{
{ {
Metadata: &chart.Metadata{Name: "laboratory mice"}, Metadata: &chart.Metadata{Name: "laboratory mice"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "pinky", Data: []byte("pinky")}, {Name: "templates/pinky", Data: []byte("pinky")},
{Name: "brain", Data: []byte("brain")}, {Name: "templates/brain", Data: []byte("brain")},
}, },
Dependencies: []*chart.Chart{{ Dependencies: []*chart.Chart{{
Metadata: &chart.Metadata{Name: "same thing we do every night"}, Metadata: &chart.Metadata{Name: "same thing we do every night"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "innermost", Data: []byte("innermost")}, {Name: "templates/innermost", Data: []byte("innermost")},
}}, }},
}, },
}, },
@ -220,13 +220,13 @@ func TestRenderDependency(t *testing.T) {
ch := &chart.Chart{ ch := &chart.Chart{
Metadata: &chart.Metadata{Name: "outerchart"}, Metadata: &chart.Metadata{Name: "outerchart"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "outer", Data: []byte(toptpl)}, {Name: "templates/outer", Data: []byte(toptpl)},
}, },
Dependencies: []*chart.Chart{ Dependencies: []*chart.Chart{
{ {
Metadata: &chart.Metadata{Name: "innerchart"}, Metadata: &chart.Metadata{Name: "innerchart"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "inner", Data: []byte(deptpl)}, {Name: "templates/inner", Data: []byte(deptpl)},
}, },
}, },
}, },
@ -243,7 +243,7 @@ func TestRenderDependency(t *testing.T) {
} }
expect := "Hello World" expect := "Hello World"
if out["outerchart/outer"] != expect { if out["outerchart/templates/outer"] != expect {
t.Errorf("Expected %q, got %q", expect, out["outer"]) t.Errorf("Expected %q, got %q", expect, out["outer"])
} }
@ -346,8 +346,8 @@ func TestRenderBuiltinValues(t *testing.T) {
inner := &chart.Chart{ inner := &chart.Chart{
Metadata: &chart.Metadata{Name: "Latium"}, Metadata: &chart.Metadata{Name: "Latium"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "Lavinia", Data: []byte(`{{.Template.Name}}{{.Chart.Name}}{{.Release.Name}}`)}, {Name: "templates/Lavinia", Data: []byte(`{{.Template.Name}}{{.Chart.Name}}{{.Release.Name}}`)},
{Name: "From", Data: []byte(`{{.Files.author | printf "%s"}} {{.Files.Get "book/title.txt"}}`)}, {Name: "templates/From", Data: []byte(`{{.Files.author | printf "%s"}} {{.Files.Get "book/title.txt"}}`)},
}, },
Values: &chart.Config{Raw: ``}, Values: &chart.Config{Raw: ``},
Dependencies: []*chart.Chart{}, Dependencies: []*chart.Chart{},
@ -360,7 +360,7 @@ func TestRenderBuiltinValues(t *testing.T) {
outer := &chart.Chart{ outer := &chart.Chart{
Metadata: &chart.Metadata{Name: "Troy"}, Metadata: &chart.Metadata{Name: "Troy"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "Aeneas", Data: []byte(`{{.Template.Name}}{{.Chart.Name}}{{.Release.Name}}`)}, {Name: "templates/Aeneas", Data: []byte(`{{.Template.Name}}{{.Chart.Name}}{{.Release.Name}}`)},
}, },
Values: &chart.Config{Raw: ``}, Values: &chart.Config{Raw: ``},
Dependencies: []*chart.Chart{inner}, Dependencies: []*chart.Chart{inner},
@ -382,9 +382,9 @@ func TestRenderBuiltinValues(t *testing.T) {
} }
expects := map[string]string{ expects := map[string]string{
"Troy/charts/Latium/Lavinia": "Troy/charts/Latium/LaviniaLatiumAeneid", "Troy/charts/Latium/templates/Lavinia": "Troy/charts/Latium/templates/LaviniaLatiumAeneid",
"Troy/Aeneas": "Troy/AeneasTroyAeneid", "Troy/templates/Aeneas": "Troy/templates/AeneasTroyAeneid",
"Troy/charts/Latium/From": "Virgil Aeneid", "Troy/charts/Latium/templates/From": "Virgil Aeneid",
} }
for file, expect := range expects { for file, expect := range expects {
if out[file] != expect { if out[file] != expect {
@ -398,8 +398,8 @@ func TestAlterFuncMap(t *testing.T) {
c := &chart.Chart{ c := &chart.Chart{
Metadata: &chart.Metadata{Name: "conrad"}, Metadata: &chart.Metadata{Name: "conrad"},
Templates: []*chart.Template{ Templates: []*chart.Template{
{Name: "quote", Data: []byte(`{{include "conrad/_partial" . | indent 2}} dead.`)}, {Name: "templates/quote", Data: []byte(`{{include "conrad/templates/_partial" . | indent 2}} dead.`)},
{Name: "_partial", Data: []byte(`{{.Release.Name}} - he`)}, {Name: "templates/_partial", Data: []byte(`{{.Release.Name}} - he`)},
}, },
Values: &chart.Config{Raw: ``}, Values: &chart.Config{Raw: ``},
Dependencies: []*chart.Chart{}, Dependencies: []*chart.Chart{},
@ -419,7 +419,7 @@ func TestAlterFuncMap(t *testing.T) {
} }
expect := " Mistah Kurtz - he dead." expect := " Mistah Kurtz - he dead."
if got := out["conrad/quote"]; got != expect { if got := out["conrad/templates/quote"]; got != expect {
t.Errorf("Expected %q, got %q (%v)", expect, got, out) t.Errorf("Expected %q, got %q (%v)", expect, got, out)
} }
} }

Write Preview
Loading…
Cancel
Save