minor engine unit test fix

pkg/engine/engine_test.go

@@ -290,20 +290,24 @@ global:
 		t.Fatalf("failed to render templates: %s", err)
 	}
 
-	if out["top/"+outerpath] != "Gather ye rosebuds while ye may" {
+	fullouterpath := "top/" + outerpath
-		t.Errorf("Unexpected outer: %q", out[outerpath])
+	if out[fullouterpath] != "Gather ye rosebuds while ye may" {
+		t.Errorf("Unexpected outer: %q", out[fullouterpath])
 	}
 
-	if out["top/charts/herrick/"+innerpath] != "Old time is still a-flyin'" {
+	fullinnerpath := "top/charts/herrick/" + innerpath
-		t.Errorf("Unexpected inner: %q", out[innerpath])
+	if out[fullinnerpath] != "Old time is still a-flyin'" {
+		t.Errorf("Unexpected inner: %q", out[fullinnerpath])
 	}
 
-	if out["top/charts/herrick/charts/deepest/"+deepestpath] != "And this same flower that smiles to-day" {
+	fulldeepestpath := "top/charts/herrick/charts/deepest/" + deepestpath
-		t.Errorf("Unexpected deepest: %q", out[deepestpath])
+	if out[fulldeepestpath] != "And this same flower that smiles to-day" {
+		t.Errorf("Unexpected deepest: %q", out[fulldeepestpath])
 	}
 
-	if out["top/charts/herrick/charts/deepest/"+checkrelease] != "Tomorrow will be dyin" {
+	fullcheckrelease := "top/charts/herrick/charts/deepest/" + checkrelease
-		t.Errorf("Unexpected release: %q", out[checkrelease])
+	if out[fullcheckrelease] != "Tomorrow will be dyin" {
+		t.Errorf("Unexpected release: %q", out[fullcheckrelease])
 	}
 }