Merge pull request #275 from jrasell/missing-key-zero

Use missingkey=zero rather than error in template rendering.

Author: jrasell

template/render_test.go

@@ -2,7 +2,6 @@ package template
 
 import (
 	"os"
-	"strings"
 	"testing"
 
 	nomad "github.com/hashicorp/nomad/api"
@@ -108,14 +107,4 @@ func TestTemplater_RenderTemplate(t *testing.T) {
 	if *job.TaskGroups[0].Name != testEnvValue {
 		t.Fatalf("expected %s but got %v", testEnvValue, *job.TaskGroups[0].Name)
 	}
-
-	// Test var-args only render.
-	fVars["job_name"] = testJobName
-	_, err = RenderTemplate("test-fixtures/missing_var.nomad", []string{}, "", &fVars)
-	if err == nil {
-		t.Fatal("expected err to not be nil")
-	}
-	if !strings.Contains(err.Error(), "binary_url") {
-		t.Fatal("expected err to mention missing var (binary_url)")
-	}
 }

template/template.go

@@ -28,7 +28,7 @@ const (
 func (t *tmpl) newTemplate() *template.Template {
 	tmpl := template.New("jobTemplate")
 	tmpl.Delims(leftDelim, rightDelim)
-	tmpl.Option("missingkey=error")
+	tmpl.Option("missingkey=zero")
 	tmpl.Funcs(funcMap(t.consulClient))
 	return tmpl
 }