Merge pull request MessagePack-CSharp#1155 from AArnott/docCtorPlusSetter

Add test to document ctor+property setter behavior

Author: AArnott

src/MessagePack.UnityClient/Assets/Scripts/Tests/ShareTests/DynamicObjectResolverTests.cs

@@ -84,6 +84,18 @@ public void DeserializerSetsMissingPropertiesToDefaultValue_OrdinalKey()
             Assert.Null(instance.Prop2);
         }
 
+        [Fact]
+        public void CtorParameterAndPropertySetterExists()
+        {
+            var m1 = new ClassWithPropertySetterAndDummyCtor(999) { MyProperty = 100 };
+            byte[] bin = MessagePackSerializer.Serialize(m1);
+            var m2 = MessagePackSerializer.Deserialize<ClassWithPropertySetterAndDummyCtor>(bin);
+
+            // In this version of the deserializer, we expect the property setter to be invoked
+            // and reaffirm the value already passed to the constructor.
+            Assert.Equal(m1.MyProperty, m2.MyProperty);
+        }
+
         /// <summary>
         /// Verifies that virtual and overridden properties do not cause the dynamic resolver to malfunction.
         /// </summary>
@@ -357,6 +369,17 @@ public class DerivedClass : BaseClass
             [DataMember]
             public string Name { get; set; }
         }
+
+        [MessagePackObject(true)]
+        public class ClassWithPropertySetterAndDummyCtor
+        {
+            public int MyProperty { get; set; }
+
+            public ClassWithPropertySetterAndDummyCtor(int myProperty)
+            {
+                // This constructor intentionally left blank.
+            }
+        }
     }
 }