fix(Solution): Udpated to latest Neuroglia Framework packages

fix(Sdk): Fixed property attributes to solve improper ordering of serialized properties
fix(Sdk): Fixed all enumerations by assigning a value to their fields, thus allowing for default values (0)

Author: cdavernas

src/ServerlessWorkflow.Sdk.UnitTests/Cases/Services/WorkflowReaderTests.cs

@@ -17,13 +17,17 @@
 using FluentAssertions;
 using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
+using Newtonsoft.Json.Serialization;
 using ServerlessWorkflow.Sdk.Models;
 using ServerlessWorkflow.Sdk.Services.IO;
+using ServerlessWorkflow.Sdk.UnitTests.Data.Factories;
 using System;
 using System.Collections.Generic;
 using System.IO;
+using System.Linq;
 using System.Net.Http;
 using System.Net.Http.Headers;
+using System.Reflection;
 using System.Threading.Tasks;
 using Xunit;
 

src/ServerlessWorkflow.Sdk.UnitTests/Data/Factories/WorkflowDefinitionFactory.cs

@@ -19,6 +19,8 @@ public static WorkflowDefinition Create()
                 .WithId("fake-workflow")
                 .WithVersion("0.1.0-fake")
                 .WithName("Fake Workflow")
+                .WithAnnotation("group=/synapse")
+                .WithMetadata("group", "synapse")
                 .WithDataInputSchema(new JSchemaGenerator().Generate(typeof(TestInputData)))
                 .AddFunction(function =>
                     function.WithName("fake-function")

src/ServerlessWorkflow.Sdk/ActionType.cs

@@ -30,17 +30,17 @@ public enum ActionType
         /// Indicates an action that invokes a function
         /// </summary>
         [EnumMember(Value = "function")]
-        Function,
+        Function = 1,
         /// <summary>
         /// Indicates an action that executes a cloud event trigger
         /// </summary>
         [EnumMember(Value = "trigger")]
-        Trigger,
+        Trigger = 2,
         /// <summary>
         /// Indicates an action that executes a subflow
         /// </summary>
         [EnumMember(Value = "subflow")]
-        Subflow
+        Subflow = 4
     }
 
 }

src/ServerlessWorkflow.Sdk/AuthenticationScheme.cs

@@ -29,17 +29,17 @@ public enum AuthenticationScheme
         /// Indicates the basic (username/password) authentication scheme
         /// </summary>
         [EnumMember(Value = "basic")]
-        Basic,
+        Basic = 1,
         /// <summary>
         /// Indicates the bearer (JwT) authentication scheme
         /// </summary>
         [EnumMember(Value = "bearer")]
-        Bearer,
+        Bearer = 2,
         /// <summary>
         /// Indicates the OAuth 2 authentication scheme
         /// </summary>
         [EnumMember(Value = "oauth2")]
-        OAuth2
+        OAuth2 = 4
     }
 
 }

src/ServerlessWorkflow.Sdk/ConditionType.cs

@@ -30,12 +30,12 @@ public enum ConditionType
         /// Indicates a transition condition
         /// </summary>
         [EnumMember(Value = "transition")]
-        Transition,
+        Transition = 1,
         /// <summary>
         /// Indicates an end condition
         /// </summary>
         [EnumMember(Value = "end")]
-        End
+        End = 2
     }
 
 }

src/ServerlessWorkflow.Sdk/EventKind.cs

@@ -30,12 +30,12 @@ public enum EventKind
         /// Indicates an event to consume
         /// </summary>
         [EnumMember(Value = "consumed")]
-        Consumed,
+        Consumed = 1,
         /// <summary>
         /// Indicates an event to produce
         /// </summary>
         [EnumMember(Value = "produced")]
-        Produced
+        Produced = 2
     }
 
 }

src/ServerlessWorkflow.Sdk/FunctionType.cs

@@ -28,32 +28,32 @@ public enum FunctionType
         /// Indicates a REST function
         /// </summary>
         [EnumMember(Value = "rest")]
-        Rest,
+        Rest = 1,
         /// <summary>
         /// Indicates an Remote Procedure Call (RPC)
         /// </summary>
         [EnumMember(Value = "rpc")]
-        Rpc,
+        Rpc = 2,
         /// <summary>
         /// Indicates a GraphQL function
         /// </summary>
         [EnumMember(Value = "graphql")]
-        GraphQL,
+        GraphQL = 4,
         /// <summary>
         /// Indicates an OData function
         /// </summary>
         [EnumMember(Value = "odata")]
-        OData,
+        OData = 8,
         /// <summary>
         /// Indicates an expression function
         /// </summary>
         [EnumMember(Value = "expression")]
-        Expression,
+        Expression = 16,
         /// <summary>
         /// Indicates an Async API function
         /// </summary>
         [EnumMember(Value = "asyncapi")]
-        AsyncApi
+        AsyncApi = 32
     }
 
 }

src/ServerlessWorkflow.Sdk/Models/ActionDefinition.cs

@@ -30,6 +30,13 @@ namespace ServerlessWorkflow.Sdk.Models
     public class ActionDefinition
     {
 
+        /// <summary>
+        /// Gets/sets the unique action definition name
+        /// </summary>
+        [ProtoMember(1)]
+        [DataMember(Order = 1)]
+        public virtual string? Name { get; set; }
+
         /// <summary>
         /// Gets the <see cref="ActionDefinition"/>'s type
         /// </summary>
@@ -51,13 +58,6 @@ public virtual ActionType Type
             }
         }
 
-        /// <summary>
-        /// Gets/sets the unique action definition name
-        /// </summary>
-        [ProtoMember(1)]
-        [DataMember(Order = 1)]
-        public virtual string? Name { get; set; }
-
         /// <summary>
         /// Gets/sets a <see cref="OneOf{T1, T2}"/> that represents the function to invoke
         /// </summary>

src/ServerlessWorkflow.Sdk/Models/StateDefinition.cs

@@ -52,28 +52,28 @@ protected StateDefinition(StateType type)
         }
 
         /// <summary>
-        /// Gets the <see cref="StateDefinition"/>'s type
+        /// Gets/sets the <see cref="StateDefinition"/>'s id
         /// </summary>
-        [YamlMember]
         [ProtoMember(1)]
         [DataMember(Order = 1)]
-        public virtual StateType Type { get; protected set; }
+        public virtual string? Id { get; set; }
 
         /// <summary>
         /// Gets/sets the <see cref="StateDefinition"/>'s id
         /// </summary>
+        [Required]
+        [Newtonsoft.Json.JsonRequired]
         [ProtoMember(2)]
         [DataMember(Order = 2)]
-        public virtual string? Id { get; set; }
+        public virtual string Name { get; set; } = null!;
 
         /// <summary>
-        /// Gets/sets the <see cref="StateDefinition"/>'s id
+        /// Gets the <see cref="StateDefinition"/>'s type
         /// </summary>
-        [Required]
-        [Newtonsoft.Json.JsonRequired]
+        [YamlMember]
         [ProtoMember(3)]
         [DataMember(Order = 3)]
-        public virtual string Name { get; set; } = null!;
+        public virtual StateType Type { get; protected set; }
 
         /// <summary>
         /// Gets/sets the filter to apply to the <see cref="StateDefinition"/>'s input and output data
@@ -155,13 +155,34 @@ public virtual Uri? DataInputSchemaUri
         [DataMember(Order = 6, Name = "onErrors")]
         public virtual List<ErrorHandlerDefinition>? Errors { get; set; }
 
+        /// <summary>
+        /// Gets/sets the id of the <see cref="StateDefinition"/> used to compensate the <see cref="StateDefinition"/>
+        /// </summary>
+        [ProtoMember(9)]
+        [DataMember(Order = 9)]
+        public virtual string? CompensatedBy { get; set; }
+
+        /// <summary>
+        /// Gets/sets a boolean indicating whether or not the <see cref="StateDefinition"/> is used for compensating another <see cref="StateDefinition"/>
+        /// </summary>
+        [ProtoMember(10)]
+        [DataMember(Order = 10)]
+        public virtual bool UsedForCompensation { get; set; }
+
+        /// <summary>
+        /// Gets/sets the <see cref="StateDefinition"/>'s metadata
+        /// </summary>
+        [ProtoMember(11)]
+        [DataMember(Order = 11)]
+        public virtual DynamicObject? Metadata { get; set; }
+
         /// <summary>
         /// Gets/sets the <see cref="OneOf{T1, T2}"/> that represents the <see cref="StateDefinition"/>'s <see cref="TransitionDefinition"/>
         /// </summary>
-        [ProtoMember(7, Name = "transition")]
-        [DataMember(Order = 7, Name = "transition")]
+        [ProtoMember(999999999, Name = "transition")]
+        [DataMember(Order = 999999999, Name = "transition")]
         [YamlMember(Alias = "transition")]
-        [Newtonsoft.Json.JsonProperty(PropertyName = "transition"), Newtonsoft.Json.JsonConverter(typeof(Newtonsoft.Json.Converters.OneOfConverter<TransitionDefinition, string>))]
+        [Newtonsoft.Json.JsonProperty(PropertyName = "transition", Order = 999999999), Newtonsoft.Json.JsonConverter(typeof(Newtonsoft.Json.Converters.OneOfConverter<TransitionDefinition, string>))]
         [System.Text.Json.Serialization.JsonPropertyName("transition"), System.Text.Json.Serialization.JsonConverter(typeof(System.Text.Json.Serialization.Converters.OneOfConverter<TransitionDefinition, string>))]
         protected virtual OneOf<TransitionDefinition, string>? TransitionValue { get; set; }
 
@@ -218,10 +239,10 @@ public virtual string? TransitionToStateName
         /// <summary>
         /// Gets/sets the <see cref="OneOf{T1, T2}"/> that represents the <see cref="StateDefinition"/>'s <see cref="EndDefinition"/>
         /// </summary>
-        [ProtoMember(4, Name = "end")]
-        [DataMember(Order = 4, Name = "end")]
+        [ProtoMember(999999999, Name = "end")]
+        [DataMember(Order = 999999999, Name = "end")]
         [YamlMember(Alias = "end")]
-        [Newtonsoft.Json.JsonProperty(PropertyName = "end"), Newtonsoft.Json.JsonConverter(typeof(Newtonsoft.Json.Converters.OneOfConverter<EndDefinition, bool>))]
+        [Newtonsoft.Json.JsonProperty(PropertyName = "end", Order = 999999999), Newtonsoft.Json.JsonConverter(typeof(Newtonsoft.Json.Converters.OneOfConverter<EndDefinition, bool>))]
         [System.Text.Json.Serialization.JsonPropertyName("end"), System.Text.Json.Serialization.JsonConverter(typeof(System.Text.Json.Serialization.Converters.OneOfConverter<EndDefinition, bool>))]
         protected virtual OneOf<EndDefinition, bool>? EndValue { get; set; }
 
@@ -239,7 +260,7 @@ public virtual EndDefinition? End
             {
                 if (this.EndValue?.T1Value == null
                     && (this.EndValue != null && this.EndValue.T2Value))
-                    return new() {  };
+                    return new() { };
                 else
                     return this.EndValue?.T1Value;
             }
@@ -275,27 +296,6 @@ public virtual bool IsEnd
             }
         }
 
-        /// <summary>
-        /// Gets/sets the id of the <see cref="StateDefinition"/> used to compensate the <see cref="StateDefinition"/>
-        /// </summary>
-        [ProtoMember(9)]
-        [DataMember(Order = 9)]
-        public virtual string? CompensatedBy { get; set; }
-
-        /// <summary>
-        /// Gets/sets a boolean indicating whether or not the <see cref="StateDefinition"/> is used for compensating another <see cref="StateDefinition"/>
-        /// </summary>
-        [ProtoMember(10)]
-        [DataMember(Order = 10)]
-        public virtual bool UsedForCompensation { get; set; }
-
-        /// <summary>
-        /// Gets/sets the <see cref="StateDefinition"/>'s metadata
-        /// </summary>
-        [ProtoMember(11)]
-        [DataMember(Order = 11)]
-        public virtual DynamicObject? Metadata { get; set; }
-
         /// <inheritdoc/>
         public override string ToString()
         {