fix: base_output not used if the plugin Output is not a map[string]interface{}

Signed-off-by: Romain Beuque <[email protected]>

Author: rbeuque74

engine/engine_test.go

@@ -646,6 +646,23 @@ func TestEmptyStringInput(t *testing.T) {
 	assert.Equal(t, "", output["foo"])
 }
 
+func TestBaseOutputNoOutput(t *testing.T) {
+	input := map[string]interface{}{}
+	res, err := createResolution("no-output.yaml", input, nil)
+	require.NotNil(t, res)
+	require.Nil(t, err)
+
+	res, err = runResolution(res)
+
+	require.Nilf(t, err, "got error %s", err)
+	require.NotNil(t, res)
+	assert.Equal(t, resolution.StateDone, res.State)
+	assert.Equal(t, step.StateDone, res.Steps["stepOne"].State)
+
+	output := res.Steps["stepOne"].Output.(map[string]interface{})
+	assert.Equal(t, "buzz", output["foobar"])
+}
+
 func TestScriptPlugin(t *testing.T) {
 	argv := "world"
 	res, err := createResolution("execScript.yaml", map[string]interface{}{"argv": argv}, nil)

engine/step/step.go

@@ -266,13 +266,13 @@ func Run(st *Step, baseConfig map[string]json.RawMessage, values *values.Values,
 		default:
 			st.Output, st.Metadata, err = runner.Exec(st.Name, baseCfgRaw, config, ctx)
 			if baseOutput != nil {
-				marshaled, err := json.Marshal(st.Output)
-				if err == nil {
-					err = utils.JSONnumberUnmarshal(bytes.NewReader(marshaled), &baseOutput)
+				if st.Output != nil {
+					marshaled, err := json.Marshal(st.Output)
 					if err == nil {
-						st.Output = baseOutput
+						_ = utils.JSONnumberUnmarshal(bytes.NewReader(marshaled), &baseOutput)
 					}
 				}
+				st.Output = baseOutput
 			}
 			st.Payload = st.Output // FIXME deprecate
 			if err != nil {

engine/templates_tests/no-output.yaml

@@ -0,0 +1,13 @@
+name: no-output
+description: A template with step returning no output
+title_format: "[test] no output"
+auto_runnable: true
+steps:
+    stepOne:
+        description: first step
+        idempotent: true
+        retry_pattern: seconds
+        action:
+            type: echo
+            base_output: {foobar: "buzz"}
+            configuration: {}