multiple invalid properties

Co-authored-by: Simon MᶜKenzie <17579442+simonmckenzie@users.noreply.github.com>

Author: lbargaoanu

src/UnitTests/ConfigurationValidation.cs

@@ -17,7 +17,7 @@ protected override MapperConfiguration CreateConfiguration() =>
 
     [Fact]
     public void Should_throw_unmapped_member_and_mismatched_type_exceptions()
-    {
+    {
         new Action(AssertConfigurationIsValid)
             .ShouldThrow<AggregateException>()
             .ShouldSatisfyAllConditions(
@@ -35,6 +35,44 @@ public void Should_throw_unmapped_member_and_mismatched_type_exceptions()
             );
     }
 }
+
+public class When_testing_a_dto_with_mismatches_in_multiple_children : AutoMapperSpecBase
+{
+    public class Source
+    {
+        public Type Foo { get; set; }
+        public Type Bar { get; set; }
+    }
+
+    public class Destination
+    {
+        public int Foo { get; set; }
+        public int Bar { get; set; }
+    }
+
+    protected override MapperConfiguration CreateConfiguration() =>
+        new(cfg => { cfg.CreateMap<Source, Destination>(); });
+
+    [Fact]
+    public void Should_throw_for_both_mismatched_children()
+    {
+        new Action(AssertConfigurationIsValid)
+            .ShouldThrow<AggregateException>()
+            .ShouldSatisfyAllConditions(
+                aex => aex.InnerExceptions.ShouldBeOfLength(2),
+                aex => aex.InnerExceptions[0]
+                    .ShouldBeOfType<AutoMapperConfigurationException>()
+                    .ShouldSatisfyAllConditions(
+                        ex => ex.MemberMap.ShouldNotBeNull(),
+                        ex => ex.MemberMap.DestinationName.ShouldBe("Foo")),
+                aex => aex.InnerExceptions[1]
+                    .ShouldBeOfType<AutoMapperConfigurationException>()
+                    .ShouldSatisfyAllConditions(
+                        ex => ex.MemberMap.ShouldNotBeNull(),
+                        ex => ex.MemberMap.DestinationName.ShouldBe("Bar"))
+            );
+    }
+}
 public class ConstructorMappingValidation : NonValidatingSpecBase
 {
     public class Destination
@@ -63,7 +101,7 @@ private ComplexType(int someMember)
     });
 
     [Fact]
-    public void Should_fail_validation() => new Action(AssertConfigurationIsValid).ShouldThrowException<AutoMapperConfigurationException>(ex=>
+    public void Should_fail_validation() => new Action(AssertConfigurationIsValid).ShouldThrowException<AutoMapperConfigurationException>(ex=>
         ex.MemberMap.ToString().ShouldBe("Void .ctor(ComplexType), parameter myComplexMember"));
 }
 
@@ -80,7 +118,7 @@ public class B
     public class C { }
 
     protected override MapperConfiguration CreateConfiguration() => new(cfg => cfg.CreateMap<A, B>().ConvertUsing(x => new B { Foo = new C() }));
-    [Fact]
+    [Fact]
     public void Validate() => AssertConfigurationIsValid();
 }
 
@@ -102,7 +140,7 @@ class Converter : ITypeConverter<A, B>
     {
         public B Convert(A source, B dest, ResolutionContext context) => new B { Foo = new C() };
     }
-    [Fact]
+    [Fact]
     public void Validate() => AssertConfigurationIsValid();
 }
 
@@ -216,7 +254,7 @@ public Dest(int value, int blarg)
     }
 
     protected override MapperConfiguration CreateConfiguration() => new(cfg =>
-    {
+    {
         cfg.CreateMap<Source, Dest>()
             .ForCtorParam("blarg", opt => opt.MapFrom(src => src.Value));
     });
@@ -316,84 +354,84 @@ public void Should_fail_a_configuration_check()
     {
         typeof(AutoMapperConfigurationException).ShouldBeThrownBy(AssertConfigurationIsValid);
     }
-}
-
+}
+
 public class ResolversWithSourceValidation : AutoMapperSpecBase
 {
     class Source
     {
-        public int Resolved { get; set; }
-        public int TypedResolved { get; set; }
-        public int Converted { get; set; }
+        public int Resolved { get; set; }
+        public int TypedResolved { get; set; }
+        public int Converted { get; set; }
         public int TypedConverted { get; set; }
     }
     class Destination
     {
-        public int ResolvedDest { get; set; }
-        public int TypedResolvedDest { get; set; }
-        public int ConvertedDest { get; set; }
+        public int ResolvedDest { get; set; }
+        public int TypedResolvedDest { get; set; }
+        public int ConvertedDest { get; set; }
         public int TypedConvertedDest { get; set; }
     }
     class MemberResolver : IMemberValueResolver<Source, Destination, int, int>
     {
         public int Resolve(Source source, Destination destination, int sourceMember, int destinationMember, ResolutionContext context) => 5;
-    }
-    class ValueConverter : IValueConverter<int, int>
-    {
-        public int Convert(int sourceMember, ResolutionContext context) => 5;
+    }
+    class ValueConverter : IValueConverter<int, int>
+    {
+        public int Convert(int sourceMember, ResolutionContext context) => 5;
     }
     protected override MapperConfiguration CreateConfiguration() => new(cfg =>
     {
         cfg.CreateMap<Source, Destination>(MemberList.Source)
-            .ForMember(d => d.ResolvedDest, o => o.MapFrom<MemberResolver, int>("Resolved"))
-            .ForMember(d=>d.TypedResolvedDest, o => o.MapFrom<MemberResolver, int>(s => s.TypedResolved))
-            .ForMember(d => d.ConvertedDest, o => o.ConvertUsing<ValueConverter, int>("Converted"))
+            .ForMember(d => d.ResolvedDest, o => o.MapFrom<MemberResolver, int>("Resolved"))
+            .ForMember(d=>d.TypedResolvedDest, o => o.MapFrom<MemberResolver, int>(s => s.TypedResolved))
+            .ForMember(d => d.ConvertedDest, o => o.ConvertUsing<ValueConverter, int>("Converted"))
             .ForMember(d => d.TypedConvertedDest, o => o.ConvertUsing<ValueConverter, int>(s => s.TypedConverted));
     });
     [Fact]
     public void Should_work()
     {
         var result = Mapper.Map<Source, Destination>(new Source());
-        result.ResolvedDest.ShouldBe(5);
-        result.TypedResolvedDest.ShouldBe(5);
-        result.ConvertedDest.ShouldBe(5);
+        result.ResolvedDest.ShouldBe(5);
+        result.TypedResolvedDest.ShouldBe(5);
+        result.ConvertedDest.ShouldBe(5);
         result.TypedConvertedDest.ShouldBe(5);
     }
 }
-
-public class NonMemberExpressionWithSourceValidation : NonValidatingSpecBase
-{
-    class Source
-    {
-        public string Value { get; set; }
-    }
-    class Destination
-    {
-        public string OtherValue { get; set; }
-    }
-    protected override MapperConfiguration CreateConfiguration() => new(c=>c.CreateMap<Source, Destination>(MemberList.Source)
-        .ForMember(d=>d.OtherValue, o=>o.MapFrom(s=>s.Value ?? "")));
-    [Fact]
-    public void Should_be_ignored() => new Action(AssertConfigurationIsValid)
-        .ShouldThrow<AutoMapperConfigurationException>().Errors[0].UnmappedPropertyNames[0].ShouldBe(nameof(Source.Value));
-}
-
-public class MatchingNonMemberExpressionWithSourceValidation : NonValidatingSpecBase
-{
-    class Source
-    {
-        public string Value { get; set; }
-    }
-    class Destination
-    {
-        public string Value { get; set; }
-    }
-    protected override MapperConfiguration CreateConfiguration() => new(c => c.CreateMap<Source, Destination>(MemberList.Source)
-        .ForMember(d => d.Value, o => o.MapFrom(s => s.Value ?? "")));
-    [Fact]
-    public void Should_be_ignored() => new Action(AssertConfigurationIsValid)
-        .ShouldThrow<AutoMapperConfigurationException>().Errors[0].UnmappedPropertyNames[0].ShouldBe(nameof(Source.Value));
-}
+
+public class NonMemberExpressionWithSourceValidation : NonValidatingSpecBase
+{
+    class Source
+    {
+        public string Value { get; set; }
+    }
+    class Destination
+    {
+        public string OtherValue { get; set; }
+    }
+    protected override MapperConfiguration CreateConfiguration() => new(c=>c.CreateMap<Source, Destination>(MemberList.Source)
+        .ForMember(d=>d.OtherValue, o=>o.MapFrom(s=>s.Value ?? "")));
+    [Fact]
+    public void Should_be_ignored() => new Action(AssertConfigurationIsValid)
+        .ShouldThrow<AutoMapperConfigurationException>().Errors[0].UnmappedPropertyNames[0].ShouldBe(nameof(Source.Value));
+}
+
+public class MatchingNonMemberExpressionWithSourceValidation : NonValidatingSpecBase
+{
+    class Source
+    {
+        public string Value { get; set; }
+    }
+    class Destination
+    {
+        public string Value { get; set; }
+    }
+    protected override MapperConfiguration CreateConfiguration() => new(c => c.CreateMap<Source, Destination>(MemberList.Source)
+        .ForMember(d => d.Value, o => o.MapFrom(s => s.Value ?? "")));
+    [Fact]
+    public void Should_be_ignored() => new Action(AssertConfigurationIsValid)
+        .ShouldThrow<AutoMapperConfigurationException>().Errors[0].UnmappedPropertyNames[0].ShouldBe(nameof(Source.Value));
+}
 
 public class When_testing_a_dto_with_fully_mapped_and_custom_matchers : AutoMapperSpecBase
 {
@@ -414,7 +452,7 @@ public class ModelDto
         cfg.CreateMap<ModelObject, ModelDto>()
             .ForMember(dto => dto.Bar, opt => opt.MapFrom(m => m.Barr));
     });
-    [Fact]
+    [Fact]
     public void Validate() => AssertConfigurationIsValid();
 }
 
@@ -802,14 +840,14 @@ interface IAbstractDest
     {
         string DifferentName { get; set; }
     }
-}
-
+}
+
 public class When_configuring_a_resolver : AutoMapperSpecBase
-{
+{
     protected override MapperConfiguration CreateConfiguration() => new(cfg =>
     {
         cfg.CreateMap<Query, Command>().ForMember(d => d.Details, o => o.MapFrom<DetailsValueResolver>());
-    });
+    });
     public class DetailsValueResolver : IValueResolver<Query, Command, List<KeyValuePair<string, string>>>
     {
         public List<KeyValuePair<string, string>> Resolve(Query source, Command destination, List<KeyValuePair<string, string>> destMember, ResolutionContext context)
@@ -827,29 +865,29 @@ public class Query
     public class Command
     {
         public List<KeyValuePair<string, string>> Details { get; private set; }
-    }
-    [Fact]
-    public void Validate() => AssertConfigurationIsValid();
-}
+    }
+    [Fact]
+    public void Validate() => AssertConfigurationIsValid();
+}
 public class ObjectPropertyAndNestedTypes : AutoMapperSpecBase
-{
+{
     protected override MapperConfiguration CreateConfiguration() => new(cfg => cfg.CreateMap<RootLevel, RootLevelDto>());
-    public class RootLevel
-    {
-        public object ObjectProperty { get; set; }
-        public SecondLevel SecondLevel { get; set; }
-    }
-    public class RootLevelDto
-    {
-        public object ObjectProperty { get; set; }
-        public SecondLevelDto SecondLevel { get; set; }
-    }
-    public class SecondLevel
-    {
-    }
-    public class SecondLevelDto
-    {
-    }
+    public class RootLevel
+    {
+        public object ObjectProperty { get; set; }
+        public SecondLevel SecondLevel { get; set; }
+    }
+    public class RootLevelDto
+    {
+        public object ObjectProperty { get; set; }
+        public SecondLevelDto SecondLevel { get; set; }
+    }
+    public class SecondLevel
+    {
+    }
+    public class SecondLevelDto
+    {
+    }
     [Fact]
     public void Should_fail_validation() => new Action(AssertConfigurationIsValid).ShouldThrow<AutoMapperConfigurationException>().MemberMap.DestinationName.ShouldBe(nameof(RootLevelDto.SecondLevel));
 }