fix: #1344 convert destObj to the correct property type before settin…

src/WorkflowCore.DSL/Services/DefinitionLoader.cs

@@ -405,7 +405,7 @@ void acn(IStepBody pStep, object pData, IStepExecutionContext pContext)
                         stack.Push(child);
                 }
 
-                stepProperty.SetValue(pStep, destObj);
+                stepProperty.SetValue(pStep, destObj.ToObject(stepProperty.PropertyType));
             }
             return acn;
         }

test/WorkflowCore.IntegrationTests/Scenarios/StoredJsonScenario.cs

@@ -84,5 +84,23 @@ public void should_execute_json_workflow_with_dynamic_data()
             data["Counter6"].Should().Be(1);
             data["Counter10"].Should().Be(1);
         }
+
+
+        [Fact]
+        public void should_execute_json_workflow_with_complex_input_type()
+        {
+            var initialData = new FlowData();
+            var workflowId = StartWorkflow(TestAssets.Utils.GetTestDefinitionJsonComplexInputProperty(), initialData);
+            WaitForWorkflowToComplete(workflowId, TimeSpan.FromSeconds(30));
+
+            var data = GetData<FlowData>(workflowId);
+            GetStatus(workflowId).Should().Be(WorkflowStatus.Complete);
+            UnhandledStepErrors.Count.Should().Be(0);
+            data.Assignee.Should().NotBeNull();
+            data.Assignee.Name.Should().Be("John Doe");
+            data.Assignee.UnitInfo.Should().NotBeNull();
+            data.Assignee.UnitInfo.Name.Should().Be("IT Department");
+
+        }
     }
 }

test/WorkflowCore.TestAssets/DataTypes/FlowData.cs

@@ -0,0 +1,10 @@
+using System;
+using System.Linq;
+using WorkflowCore.TestAssets.Steps;
+
+namespace WorkflowCore.TestAssets.DataTypes;
+
+public class FlowData
+{
+    public AssigneeInfo Assignee { get; set; } = new();
+}

test/WorkflowCore.TestAssets/Steps/AssignTask.cs

@@ -0,0 +1,27 @@
+using System;
+using System.Linq;
+using WorkflowCore.Interface;
+using WorkflowCore.Models;
+using WorkflowCore.TestAssets.DataTypes;
+
+namespace WorkflowCore.TestAssets.Steps;
+
+public class AssignTask : StepBody
+{       
+    public AssigneeInfo Assignee { get; set; }
+
+    public override ExecutionResult Run(IStepExecutionContext context)
+    {
+        if (context.Workflow.Data is FlowData flowData)
+        {
+            flowData.Assignee = new AssigneeInfo
+            {
+                Id = Assignee.Id,
+                Name = Assignee.Name,
+                MemberType = Assignee.MemberType,
+                UnitInfo = Assignee.UnitInfo
+            };
+        }
+        return ExecutionResult.Next();
+    }
+}

test/WorkflowCore.TestAssets/Steps/AssigneeInfo.cs

@@ -0,0 +1,20 @@
+using System;
+using System.Linq;
+
+namespace WorkflowCore.TestAssets.Steps;
+
+public class AssigneeInfo
+{
+    public int Id { get; set; }
+    public string Name { get; set; }
+    public int MemberType { get; set; }
+
+    public UnitInfo UnitInfo { get; set; }
+}
+
+public class UnitInfo
+{
+    public int Id { get; set; }
+    public string Name { get; set; }
+    public int UnitType { get; set; }
+}

test/WorkflowCore.TestAssets/Utils.cs

@@ -39,6 +39,12 @@ public static string GetTestDefinitionJsonMissingInputProperty()
         {
             return File.ReadAllText("stored-def-missing-input-property.json");
         }
+
+
+        public static string GetTestDefinitionJsonComplexInputProperty()
+        {
+            return File.ReadAllText("def-complex-input-property.json");
+        }
     }
 }
 

test/WorkflowCore.TestAssets/WorkflowCore.TestAssets.csproj

@@ -13,6 +13,7 @@
     <None Remove="stored-definition.yaml" />
     <None Remove="stored-dynamic-definition.json" />
     <None Remove="stored-dynamic-definition.yaml" />
+    <None Remove="def-complex-input-property.json" />
   </ItemGroup>
 
   <ItemGroup>
@@ -31,6 +32,9 @@
     <EmbeddedResource Include="stored-def-missing-input-property.json">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </EmbeddedResource>
+    <EmbeddedResource Include="def-complex-input-property.json">
+      <CopyToOutputDirectory>Always</CopyToOutputDirectory>
+    </EmbeddedResource>
   </ItemGroup>
 
   <ItemGroup>

test/WorkflowCore.TestAssets/def-complex-input-property.json

@@ -0,0 +1,24 @@
+{
+  "Id": "FlowWithComplexInputType",
+  "Version": 1,
+  "DataType": "WorkflowCore.TestAssets.DataTypes.FlowData, WorkflowCore.TestAssets",
+  "Steps": [
+    {
+      "Id": "AssignTask",
+      "StepType": "WorkflowCore.TestAssets.Steps.AssignTask, WorkflowCore.TestAssets",
+      "Inputs": {
+        "Assignee": {
+          "id": 1,
+          "name": "John Doe",
+          "memberType": 1,
+          "unitInfo": {
+            "id": 1,
+            "name": "IT Department",
+            "unitType": 1
+          }
+        }
+      },
+      "Reason": "3"
+    }
+  ]
+}