ovh
diff --git a/‎sdk/exportentities/workflow.go
Lines changed: 11 additions & 5 deletions b/‎sdk/exportentities/workflow.go
Lines changed: 11 additions & 5 deletions
diff --git a/‎sdk/exportentities/workflow_test.go
Lines changed: 74 additions & 0 deletions b/‎sdk/exportentities/workflow_test.go
Lines changed: 74 additions & 0 deletions
@@ -370,12 +370,18 @@ func NewWorkflow(w sdk.Workflow, opts ...WorkflowOptions) (Workflow, error) {
 				if m == nil {
 					return exportedWorkflow, sdk.WrapError(sdk.ErrNotFound, "unable to find hook model %s", h.HookModelName)
 				}
+				pipHook := HookEntry{
+					Model:      h.HookModelName,
+					Ref:        h.Ref,
+					Config:     h.Config.Values(m.DefaultConfig),
+					Conditions: &h.Conditions,
+				}
+
+				if h.Conditions.LuaScript == "" && len(h.Conditions.PlainConditions) == 0 {
+					pipHook.Conditions = nil
+				}
 
-				exportedWorkflow.Hooks[n.Name] = append(exportedWorkflow.Hooks[n.Name], HookEntry{
-					Model:  h.HookModelName,
-					Ref:    h.Ref,
-					Config: h.Config.Values(m.DefaultConfig),
-				})
+				exportedWorkflow.Hooks[n.Name] = append(exportedWorkflow.Hooks[n.Name], pipHook)
 			}
 		}
 	}
 
@@ -305,6 +305,80 @@ func TestWorkflow_GetWorkflow(t *testing.T) {
 				},
 			},
 		},
+		// hook conditions
+		{
+			name: "Workflow with multiple nodes should display hook conditions",
+			fields: fields{
+				Workflow: map[string]exportentities.NodeEntry{
+					"root": {
+						PipelineName: "pipeline-root",
+					},
+					"child": {
+						PipelineName: "pipeline-child",
+						DependsOn:    []string{"root"},
+						OneAtATime:   &exportentities.True,
+					},
+				},
+				Hooks: map[string][]exportentities.HookEntry{
+					"root": []exportentities.HookEntry{{
+						Model: "Scheduler",
+						Config: map[string]string{
+							"crontab": "* * * * *",
+							"payload": "{}",
+						},
+						Conditions: &sdk.WorkflowNodeConditions{
+							LuaScript: "return true",
+						},
+					}},
+				},
+			},
+			wantErr: false,
+			want: sdk.Workflow{
+				HistoryLength: sdk.DefaultHistoryLength,
+				WorkflowData: &sdk.WorkflowData{
+					Node: sdk.Node{
+						Name: "root",
+						Type: "pipeline",
+						Hooks: []sdk.NodeHook{
+							{
+								HookModelName: "Scheduler",
+								Conditions: sdk.WorkflowNodeConditions{
+									LuaScript: "return true",
+								},
+								Config: sdk.WorkflowNodeHookConfig{
+									"crontab": sdk.WorkflowNodeHookConfigValue{
+										Value:        "* * * * *",
+										Configurable: true,
+										Type:         sdk.HookConfigTypeString,
+									},
+									"payload": sdk.WorkflowNodeHookConfigValue{
+										Value:        "{}",
+										Configurable: true,
+										Type:         sdk.HookConfigTypeString,
+									},
+								},
+							},
+						},
+						Context: &sdk.NodeContext{
+							PipelineName: "pipeline-root",
+						},
+						Triggers: []sdk.NodeTrigger{
+							{
+								ChildNode: sdk.Node{
+									Name: "child",
+									Ref:  "child",
+									Type: "pipeline",
+									Context: &sdk.NodeContext{
+										PipelineName: "pipeline-child",
+										Mutex:        true,
+									},
+								},
+							},
+						},
+					},
+				},
+			},
+		},
 		// root(pipeline-root) -> child(pipeline-child)
 		{
 			name: "Complexe workflow without joins and mutex should not raise an error",
Original file line number	Diff line number	Diff line change
`@@ -370,12 +370,18 @@ func NewWorkflow(w sdk.Workflow, opts ...WorkflowOptions) (Workflow, error) {`
`370`	`370`	`if m == nil {`
`371`	`371`	`return exportedWorkflow, sdk.WrapError(sdk.ErrNotFound, "unable to find hook model %s", h.HookModelName)`
`372`	`372`	`}`
	`373`	`+ pipHook := HookEntry{`
	`374`	`+ Model: h.HookModelName,`
	`375`	`+ Ref: h.Ref,`
	`376`	`+ Config: h.Config.Values(m.DefaultConfig),`
	`377`	`+ Conditions: &h.Conditions,`
	`378`	`+ }`
	`379`	`+`
	`380`	`+ if h.Conditions.LuaScript == "" && len(h.Conditions.PlainConditions) == 0 {`
	`381`	`+ pipHook.Conditions = nil`
	`382`	`+ }`
`373`	`383`
`374`		`- exportedWorkflow.Hooks[n.Name] = append(exportedWorkflow.Hooks[n.Name], HookEntry{`
`375`		`- Model: h.HookModelName,`
`376`		`- Ref: h.Ref,`
`377`		`- Config: h.Config.Values(m.DefaultConfig),`
`378`		`- })`
	`384`	`+ exportedWorkflow.Hooks[n.Name] = append(exportedWorkflow.Hooks[n.Name], pipHook)`
`379`	`385`	`}`
`380`	`386`	`}`
`381`	`387`	`}`