Pipeline: Don't wrap snapshot in template engine in "Data"

Signed-off-by: Richard Kosegi <richard.kosegi@gmail.com>
rkosegi · Aug 13, 2024 · 3803ae8 · 3803ae8
1 parent bfb0197
commit 3803ae8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/pipeline/template_op.go b/pipeline/template_op.go
@@ -41,9 +41,7 @@ func (ts *TemplateOp) Do(ctx ActionContext) error {
 	if len(ts.Path) == 0 {
 		return ErrPathEmpty
 	}
-	val, err := ctx.TemplateEngine().Render(ts.Template, map[string]interface{}{
-		"Data": ctx.Snapshot(),
-	})
+	val, err := ctx.TemplateEngine().Render(ts.Template, ctx.Snapshot())
 	ctx.Data().AddValueAt(ts.Path, dom.LeafNode(val))
 	return err
 }

diff --git a/pipeline/template_op_test.go b/pipeline/template_op_test.go
@@ -32,7 +32,7 @@ func TestExecuteTemplateOp(t *testing.T) {
 	gd = b.Container()
 	gd.AddValueAt("root.leaf1", dom.LeafNode(123456))
 	ts = TemplateOp{
-		Template: `{{ (mul .Data.root.leaf1 2) | quote }}`,
+		Template: `{{ (mul .root.leaf1 2) | quote }}`,
 		Path:     "result.x1",
 	}
 	assert.NoError(t, New(WithData(gd)).Execute(&ts))