Set EventRef.ResultEventRef to optional (#239)

* Set EventRef.ResultEventRef to optional

Signed-off-by: Dmitrii Tikhomirov <[email protected]>

* removed validate declaration and updated tests

Signed-off-by: Dmitrii Tikhomirov <[email protected]>

---------

Signed-off-by: Dmitrii Tikhomirov <[email protected]>

Author: treblereel

model/event.go

@@ -96,8 +96,8 @@ type EventRef struct {
 	// +kubebuilder:validation:Required
 	TriggerEventRef string `json:"triggerEventRef" validate:"required"`
 	// Reference to the unique name of a 'consumed' event definition
-	// +kubebuilder:validation:Required
-	ResultEventRef string `json:"resultEventRef" validate:"required"`
+	// +optional
+	ResultEventRef string `json:"resultEventRef"`
 	// Maximum amount of time (ISO 8601 format) to wait for the result event. If not defined it be set to the
 	// actionExecutionTimeout
 	// +optional

model/event_validator_test.go

@@ -175,11 +175,9 @@ func TestEventRefStructLevelValidation(t *testing.T) {
 			Model: func() Workflow {
 				model := baseWorkflow.DeepCopy()
 				model.States[0].OperationState.Actions[0].EventRef.TriggerEventRef = ""
-				model.States[0].OperationState.Actions[0].EventRef.ResultEventRef = ""
 				return *model
 			},
-			Err: `workflow.states[0].actions[0].eventRef.triggerEventRef is required
-workflow.states[0].actions[0].eventRef.resultEventRef is required`,
+			Err: `workflow.states[0].actions[0].eventRef.triggerEventRef is required`,
 		},
 		{
 			Desp: "exists",