Test coverage for callbacks based on derived source types / Removing unused code

Author: SteveWilkes

AgileMapper.UnitTests/Configuration/WhenConfiguringMappingCallbacks.cs

@@ -318,5 +318,25 @@ public void ShouldRestrictAPostMappingCallbackBySourceType()
                 matchingResult.Value.ShouldBe("SetByCallback");
             }
         }
+
+        [Fact]
+        public void ShouldExecuteAPostMappingCallbackForADerivedType()
+        {
+            using (var mapper = Mapper.CreateNew())
+            {
+                var derivedSource = default(object);
+
+                mapper.WhenMapping
+                    .From<CustomerViewModel>()
+                    .To<Person>()
+                    .After.MappingEnds
+                    .Call(ctx => derivedSource = ctx.Source);
+
+                PersonViewModel source = new CustomerViewModel { Name = "?!?!?" };
+                mapper.Map(source).ToANew<Person>();
+
+                derivedSource.ShouldBeSameAs(source);
+            }
+        }
     }
 }

AgileMapper/ObjectPopulation/MappingDataFactory.cs

@@ -15,22 +15,6 @@ internal static class MappingDataFactory
         public static readonly MethodInfo ForElementMethod = typeof(MappingDataFactory)
             .GetPublicStaticMethod("ForElement");
 
-        public static ObjectMappingData<TSource, TTarget> ForRoot<TSource, TTarget>(
-            TSource source,
-            TTarget target,
-            IMappingContext mappingContext)
-        {
-            var mapperKey = new RootObjectMapperKey(MappingTypes.Fixed<TSource, TTarget>(), mappingContext);
-
-            return new ObjectMappingData<TSource, TTarget>(
-                source,
-                target,
-                null,
-                mapperKey,
-                mappingContext,
-                parent: null);
-        }
-
         public static ObjectMappingData<TSource, TTarget> ForChild<TSource, TTarget>(
             TSource source,
             TTarget target,