Add MessagePackMemberAttribute.Name to customize keys of map. related to #38

Author: yfakariya

src/MsgPack/Serialization/DataMemberContract.cs

@@ -160,7 +160,7 @@ public DataMemberContract( MemberInfo member, MessagePackMemberAttribute attribu
 				throw new SerializationException( String.Format( CultureInfo.CurrentCulture, "The member ID cannot be negative. The member is '{0}' in the '{1}' type.", member.Name, member.DeclaringType ) );
 			}
 
-			this._name = member.Name;
+			this._name = String.IsNullOrEmpty( attribute.Name ) ? member.Name : attribute.Name;
 			this._nilImplication = attribute.NilImplication;
 			this._id = attribute.Id;
 		}

src/MsgPack/Serialization/MessagePackMemberAttribute.cs

@@ -48,6 +48,20 @@ public int Id
 			get { return this._id; }
 		}
 
+		private string _name;
+
+		/// <summary>
+		///		Gets or sets the name of this member.
+		/// </summary>
+		/// <value>
+		///		The name which will be used in map key on serialized MessagePack stream.
+		/// </value>
+		public string Name
+		{
+			get { return this._name; }
+			set { this._name = value; }
+		}
+
 		private NilImplication _nilImplication;
 
 		/// <summary>

test/MsgPack.UnitTest/Serialization/SerializationTargetTest.cs

@@ -74,12 +74,12 @@ public void TestPlain()
 		[Test]
 		public void TestAnnotated()
 		{
-			TestCore<AnnotatedClass>( 
-				PublicProperty, NonPublicProperty, PublicField, NonPublicField, 
+			TestCore<AnnotatedClass>(
+				PublicProperty, NonPublicProperty, PublicField, NonPublicField,
 #if !NETFX_CORE && !WINDOWS_PHONE
-				NonSerializedPublicField, NonSerializedNonPublicField, 
+ NonSerializedPublicField, NonSerializedNonPublicField,
 #endif
-				CollectionReadOnlyProperty );
+ CollectionReadOnlyProperty );
 		}
 
 		[Test]
@@ -89,15 +89,29 @@ public void TestDataMember()
 			TestCore<DataMamberClass>(
 				PublicProperty, NonPublicProperty, PublicField, NonPublicField,
 #if !NETFX_CORE && !WINDOWS_PHONE
-				NonSerializedPublicField, NonSerializedNonPublicField, 
+ NonSerializedPublicField, NonSerializedNonPublicField,
 #endif
-				CollectionReadOnlyProperty );
+ CollectionReadOnlyProperty );
+		}
+
+		[Test]
+		public void TestAliasInMessagePackMember()
+		{
+			var target = SerializationTarget.GetTargetMembers( typeof( AnnotatedClass ) );
+			Assert.That( target.Any( m => m.Contract.Name == "Alias" && m.Contract.Name != m.Member.Name ) );
+		}
+
+		[Test]
+		public void TestAliasInDataMember()
+		{
+			var target = SerializationTarget.GetTargetMembers( typeof( DataMamberClass ) );
+			Assert.That( target.Any( m => m.Contract.Name == "Alias" && m.Contract.Name != m.Member.Name ) );
 		}
 
 		private static void TestCore<T>( params string[] expectedMemberNames )
 		{
 			var expected = expectedMemberNames.OrderBy( n => n ).ToArray();
-			var actual = SerializationTarget.GetTargetMembers( typeof( T ) ).OrderBy( m => m.Contract.Name ).Select( m => m.Contract.Name ).ToArray();
+			var actual = SerializationTarget.GetTargetMembers( typeof( T ) ).OrderBy( m => m.Member.Name ).Select( m => m.Member.Name ).ToArray();
 			Assert.That( actual, Is.EqualTo( expected ), String.Join( ", ", actual ) );
 		}
 

test/MsgPack.UnitTest/Serialization/SerializationTargets.cs

@@ -79,7 +79,7 @@ public class AnnotatedClass
 	{
 		public static int StaticProperty { get; set; }
 
-		[MessagePackMember( 0 )]
+		[MessagePackMember( 0, Name = "Alias" )]
 		public int PublicProperty { get; set; }
 		[MessagePackMember( 1 )]
 		public int PublicField;
@@ -166,7 +166,7 @@ public class DataMamberClass
 	{
 		public static int StaticProperty { get; set; }
 
-		[DataMember( Order = 0 )]
+		[DataMember( Order = 0, Name = "Alias" )]
 		public int PublicProperty { get; set; }
 		[DataMember( Order = 1 )]
 		public int PublicField;