fix(action.go): only append an output to an operation if it applies t…

…o the action

action/action.go

@@ -213,7 +213,10 @@ func opFromClaim(action string, stateless bool, c *claim.Claim, ii bundle.Invoca
 	var outputs []string
 	if c.Bundle.Outputs != nil {
 		for _, v := range c.Bundle.Outputs {
-			outputs = append(outputs, v.Path)
+			// Only add to list if output applies to the provided action
+			if v.AppliesTo(action) {
+				outputs = append(outputs, v.Path)
+			}
 		}
 	}
 

action/action_test.go

@@ -350,6 +350,33 @@ func TestOpFromClaim_MissingRequiredParamSpecificToAction(t *testing.T) {
 	})
 }
 
+func TestOpFromClaim_NotApplicableToAction(t *testing.T) {
+	c := newClaim()
+	c.Bundle = mockBundle()
+	invocImage := c.Bundle.InvocationImages[0]
+
+	c.Bundle.Outputs = map[string]bundle.Output{
+		"some-output": {
+			Path:    "/path/to/some-output",
+			ApplyTo: []string{"install"},
+		},
+	}
+
+	t.Run("output is added to the operation when it applies to the action", func(t *testing.T) {
+		op, err := opFromClaim("install", stateful, c, invocImage, mockSet)
+		assert.NoError(t, err)
+		gotOutputs := op.Outputs
+		assert.Contains(t, gotOutputs, "/path/to/some-output", "some-output should be listed in op.Outputs")
+	})
+
+	t.Run("output not added to the operation when it doesn't apply to the action", func(t *testing.T) {
+		op, err := opFromClaim("uninstall", stateful, c, invocImage, mockSet)
+		assert.NoError(t, err)
+		gotOutputs := op.Outputs
+		assert.NotContains(t, gotOutputs, "/path/to/some-output", "some-output should not be listed in op.Outputs")
+	})
+}
+
 func TestSetOutputsOnClaim(t *testing.T) {
 	c := newClaim()
 	c.Bundle = mockBundle()