Extra test coverage

Author: SteveWilkes

AgileMapper.UnitTests/MapperCloning/WhenCloningMapperDataSources.cs

@@ -18,6 +18,9 @@ public void ShouldCloneADataSource()
                     .Map((s, t) => s.Value1 * 2)
                     .To(t => t.Value1);
 
+                // Populate the derived types cache so it can be cloned:
+                baseMapper.GetPlanFor<Customer>().ToANew<CustomerViewModel>();
+
                 using (var childMapper1 = baseMapper.CloneSelf())
                 using (var childMapper2 = baseMapper.CloneSelf())
                 {

AgileMapper.UnitTests/WhenMappingDerivedTypes.cs

@@ -270,6 +270,40 @@ public void ShouldCreateADerivedTypeInAnExistingMemberEnumerableUsingRuntimeType
             }
         }
 
+        [Fact]
+        public void ShouldMapADerivedTypeToAStruct()
+        {
+            using (var mapper = Mapper.CreateNew())
+            {
+                mapper.WhenMapping
+                    .From<MysteryCustomer>()
+                    .ToANew<PublicPropertyStruct<string>>()
+                    .Map((mc, pps) => mc.Name)
+                    .To(pps => pps.Value);
+
+                Customer customer = new MysteryCustomer { Id = Guid.NewGuid(), Name = "Mystery!" };
+                var customerResult = mapper.Map(customer).ToANew<PublicPropertyStruct<string>>();
+
+                customerResult.Value.ShouldBe("Mystery!");
+            }
+
+            var source = new MysteryCustomer { Discount = 0.2m };
+            var result = Mapper.Map(source).ToANew<PersonViewModel>();
+
+            result.ShouldBeOfType<MysteryCustomerViewModel>();
+            ((MysteryCustomerViewModel)result).Discount.ShouldBe(0.2);
+        }
+
+        [Fact]
+        public void ShouldMapDerivedTypesToTheSameTargetType()
+        {
+            var source = new MysteryCustomer { Id = Guid.NewGuid(), Name = "Customer!" };
+            var result = Mapper.Map(source).ToANew<PersonDto>();
+
+            result.Id.ShouldBe(source.Id.ToString());
+            result.Name.ShouldBe("Customer!");
+        }
+
         [Fact]
         public void ShouldCreateADerivedTypeForARequestedParentType()
         {
@@ -322,5 +356,19 @@ public void ShouldMapMultipleRuntimeTypedChildMembers()
             result.Value2.ShouldHaveSingleItem();
             result.Value2.First().Name.ShouldBe("Mr Faff");
         }
+
+        #region Helper Classes
+
+        // ReSharper disable once ClassNeverInstantiated.Local
+        // ReSharper disable UnusedAutoPropertyAccessor.Local
+        private class PersonDto
+        {
+            public string Id { get; set; }
+
+            public string Name { get; set; }
+        }
+        // ReSharper restore UnusedAutoPropertyAccessor.Local
+
+        #endregion
     }
 }

AgileMapper/Configuration/DerivedTypePairSet.cs

@@ -125,7 +125,6 @@ private void LookForDerivedTypePairs(IBasicMapperData mapperData, MapperContext
 
                 if (derivedTargetTypes.None())
                 {
-                    // TODO: Test coverage - derived source types but no derived target types
                     return;
                 }
 
@@ -238,6 +237,11 @@ private static Type GetRootType(Type type)
                 return type;
             }
 
+            if (type.IsValueType())
+            {
+                return typeof(ValueType);
+            }
+
             var parentType = type;
 
             while (parentType != typeof(object))