Fix copying custom attributes with array constructor arguments

Author: Methuselah96

ImpromptuInterface/src/EmitProxy/ActLikeMaker.cs

@@ -1433,11 +1433,36 @@ public static CustomAttributeBuilder ToCustomAttributeBuilder(CustomAttributeDat
         {
             return new CustomAttributeBuilder(
                 customAttributeData.Constructor,
-                customAttributeData.ConstructorArguments.Select(arg => arg.Value).ToArray(),
-                customAttributeData.NamedArguments.Where(arg => arg.MemberInfo.MemberType == MemberTypes.Property).Select(arg => arg.MemberInfo).Cast<PropertyInfo>().ToArray(),
-                customAttributeData.NamedArguments.Where(arg => arg.MemberInfo.MemberType == MemberTypes.Property).Select(arg => arg.TypedValue.Value).ToArray(),
-                customAttributeData.NamedArguments.Where(arg => arg.MemberInfo.MemberType == MemberTypes.Field).Select(arg => arg.MemberInfo).Cast<FieldInfo>().ToArray(),
-                customAttributeData.NamedArguments.Where(arg => arg.MemberInfo.MemberType == MemberTypes.Field).Select(arg => arg.TypedValue.Value).ToArray());
+                customAttributeData.ConstructorArguments
+                    .Select(
+                        arg =>
+                            arg.Value?.GetType()
+                            == typeof(ReadOnlyCollection<CustomAttributeTypedArgument>)
+                                ? ((ReadOnlyCollection<CustomAttributeTypedArgument>)arg.Value)
+                                .Select(arrayArg => arrayArg.Value)
+                                .ToArray()
+                                : arg.Value
+                    )
+                    .ToArray(),
+                customAttributeData.NamedArguments
+                    .Where(arg => arg.MemberInfo.MemberType == MemberTypes.Property)
+                    .Select(arg => arg.MemberInfo)
+                    .Cast<PropertyInfo>()
+                    .ToArray(),
+                customAttributeData.NamedArguments
+                    .Where(arg => arg.MemberInfo.MemberType == MemberTypes.Property)
+                    .Select(arg => arg.TypedValue.Value)
+                    .ToArray(),
+                customAttributeData.NamedArguments
+                    .Where(arg => arg.MemberInfo.MemberType == MemberTypes.Field)
+                    .Select(arg => arg.MemberInfo)
+                    .Cast<FieldInfo>()
+                    .ToArray(),
+                customAttributeData.NamedArguments
+                    .Where(arg => arg.MemberInfo.MemberType == MemberTypes.Field)
+                    .Select(arg => arg.TypedValue.Value)
+                    .ToArray()
+            );
         }
 
 

Tests/UnitTestImpromptuInterface/Basic.cs

@@ -444,11 +444,30 @@ public void PropertyAttributeTest()
         {
             dynamic tNew = new ExpandoObject();
             ISimpeleSetClassProps tActsLike = Impromptu.ActLike<ISimpeleSetClassProps>(tNew);
-            var attribute = (DisplayNameAttribute)tActsLike.GetType().GetProperty(nameof(ISimpeleSetClassProps.Prop1)).GetCustomAttribute(typeof(DisplayNameAttribute), true);
+            var attribute = (DisplayNameAttribute)
+                tActsLike
+                    .GetType()
+                    .GetProperty(nameof(ISimpeleSetClassProps.Prop1))
+                    .GetCustomAttribute(typeof(DisplayNameAttribute), true);
             Assert.NotNull(attribute);
             Assert.AreEqual("testDisplayName", attribute.DisplayName);
         }
 
+        [Test]
+        public void PropertyAttributeWithArrayArgumentTest()
+        {
+            dynamic tNew = new ExpandoObject();
+            ISimpeleSetClassProps tActsLike = Impromptu.ActLike<ISimpeleSetClassProps>(tNew);
+            var attribute = (DisplayNamesAttribute)
+                tActsLike
+                    .GetType()
+                    .GetProperty(nameof(ISimpeleSetClassProps.Prop2))
+                    .GetCustomAttribute(typeof(DisplayNamesAttribute), true);
+            Assert.NotNull(attribute);
+            Assert.AreEqual("testDisplayName1", attribute.DisplayNames[0]);
+            Assert.AreEqual("testDisplayName2", attribute.DisplayNames[1]);
+        }
+
         [Test]
         public void EventDynamicPropertyTest()
         {

Tests/UnitTestImpromptuInterface/Support/SupportDefinitions.cs

@@ -102,11 +102,22 @@ public interface ISimpeleClassProps
         Guid Prop3 { get; }
     }
 
+    public class DisplayNamesAttribute : Attribute
+    {
+        public string[] DisplayNames { get; }
+
+        public DisplayNamesAttribute(params string[] displayNames)
+        {
+            DisplayNames = displayNames;
+        }
+    }
+
     public interface ISimpeleSetClassProps
     {
         [DisplayName("testDisplayName")]
         string Prop1 { set ; }
 
+        [DisplayNames("testDisplayName1", "testDisplayName2")]
         long Prop2 { set; }
 
         Guid Prop3 { set; }