From 4ec583216551503702b2d467b9dc0b924e3d64c5 Mon Sep 17 00:00:00 2001 From: Matt Morrissette Date: Fri, 2 Dec 2022 16:45:06 -0800 Subject: [PATCH 1/3] Merge upstream/master with branch and fix conflicts Update to version 6.0.2 --- .github/workflows/ci.yml | 4 +- icon.png | Bin 0 -> 1672 bytes ...Mapper.Extensions.ExpressionMapping.csproj | 23 +- .../ExpressionMapper.cs | 6 +- .../FindMemberExpressionsVisitor.cs | 4 +- .../MapIncludesVisitor.cs | 8 +- .../MapperExtensions.cs | 12 +- .../PrependParentNameVisitor.cs | 112 +------- .../Resources.Designer.cs} | 22 +- .../Properties/Resources.resx | 146 ++++++++++ .../ReflectionExtensions.cs | 3 - .../Resource.resx | 161 ----------- .../TypeExtensions.cs | 15 +- .../XpressionMapperVisitor.cs | 52 ++-- ...ensions.ExpressionMapping.UnitTests.csproj | 3 +- ...mberExpressionsWithoutCustomExpressions.cs | 269 ++++++++++++++++++ ...omChildReferenceWithoutMemberExpression.cs | 53 ++++ ...MapParameterBodyWithoutMemberExpression.cs | 46 +++ .../EF.cs | 10 + ...erMappingsOfLiteralParentTypesMustMatch.cs | 114 ++++++++ ...dOperationExceptionForUnmatchedLiterals.cs | 140 +++++++++ .../XpressionMapperTests.cs | 37 ++- 22 files changed, 919 insertions(+), 321 deletions(-) create mode 100644 icon.png rename src/AutoMapper.Extensions.ExpressionMapping/{Resource.Designer.cs => Properties/Resources.Designer.cs} (88%) create mode 100644 src/AutoMapper.Extensions.ExpressionMapping/Properties/Resources.resx delete mode 100644 src/AutoMapper.Extensions.ExpressionMapping/Resource.resx create mode 100644 tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapMismatchedLiteralMemberExpressionsWithoutCustomExpressions.cs create mode 100644 tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyFromChildReferenceWithoutMemberExpression.cs create mode 100644 tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyWithoutMemberExpression.cs create mode 100644 tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/EF.cs create mode 100644 tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/MemberMappingsOfLiteralParentTypesMustMatch.cs create mode 100644 tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/ShouldThrowInvalidOperationExceptionForUnmatchedLiterals.cs diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 94ed54d..904c5fe 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -4,11 +4,11 @@ on: push: branches: - master - - v4.1.3 + - v6.0.2 pull_request: branches: - master - - v4.1.3 + - v6.0.2 jobs: build: diff --git a/icon.png b/icon.png new file mode 100644 index 0000000000000000000000000000000000000000..56b96cabfea86f6bed8f2392fbe4a3553d108c86 GIT binary patch literal 1672 zcmV;326y?1P)?ZyV}#yTcU9VCW@8K-~f6mOc@sprX^vnT#m-heSt|!~tq_0USYr0CO=$V}eW6#4KiEkcbNo7~9yo!RXev(Z0TS z{G;vbyZte`f{e2?d%gGEb3gZ-bI-kP84S|zC@fM;l8X__0F;6;3vQT}&)o#8GPe(1@cy%}I1-FCP{LL>uJh4(qzrNMErWqTN2>`BDQ7 z3Hf~R)NaG*w@1d+OIuihsdYObK%L!s|{0+I)f z5DTzWu0+=M_W&vA;n5?TWXj`@A?J&FKwxl}S3teoryDkv%!8x(D3>al$AkXW>)~ttD+VwE1kQ`b1;kIz zB*b^J1%uD7#FU51pkD5aWR}^C>|J#{q*q4YGta|!wnLvjS-`MP%Af$Cw4Z^}eugt@ z+prd<;(Iu~tF`aLceW$xFmnsI6`(N}2uXzX!ZYk-lhA zu#AO8RdHoU9W;ZN0R4pK7a0f6c!W+Ebi50MZS5*fSN`c2C|z9>4zm$3{G2psTn`vL zW14p#(jI((8*}ZehLplg6uW?=Koi~*4egoh)^KBdr_W&Q&(@prV$z`Fh1qCKuuq3& z*%EH-wT1(_$5Zv<4M7uamiUtprdGWUW-{sH)vH4oKG6*ACIBOC6o1zz6K3NA(kq@o z*6Noy8=l5P(0sm>!yGRP`mL3?7wM1`(--L%o%wKl@-YXgboby&{T|&D1i-d*2{Ime zI5M6{haAs-IKPPi!Kk1ayr@xhd-yqMz%=JxIGc__oR(t<2chq+8mL}b*JtVJ$lX?p zyAJL{){5mf0E~oMhYWjkqFyA%U|s$c3XUI!G$W7m=1{|acz^qyTUhq1ufUX-2WFB` zuxlqA)m4Dm%-PnA_6=S%>aZ?n5y3zon=hec&dWpkf^uY5zKFE>C5RT!u@mUs@-|4* z^s(afX(;}q1%j3Tpc@z%#NdHO4D4@&GB9As*~EFuhQDl};I0lXgF3o>y7RSJaWw@ys-L5J?Rk=k%5RhA21N+7e@dpw?>374|-2?xnOHfC=k+DW+1r#s} z*M}5D8ykcBgAZ|aeG{(KQ&OzD`U8+8H2k&hwsA0gn-89 zDYIuG_e~dao?gb)-2VmidfO=s9Xte2Q!_MqG^Ls-Iqa~nszmnUg-D-MoYd!f&F=>r zs^6=FM(2m-m7#QZL+S2;zv~i|&`o~4FvmU{77(>r1e+Ds`{%>#bV8h(1tBAY1B~Qo z0PNh1SzZc#|HPlo(5FZUF@ci>F}BCCPf=q*vOt730$<>bHH|O!5ie?n!X2wKeof^1 zjW1o7qTAa9BD6*fB`{zBoXF7^d%6Yx;x9gY>hTT=Nh|Ac6c%ZM&`u1N$qRE-E=K@C zAX$P^A{4v3IsvM_RbEUj%zVF57*Tb(+ubPuz}&uzdo_(*Dv_**!~hsKR3QC=qEDO* zAd&+lkjzMPEg0zC3xNOjzTHt+1O^v?G79}5cDuvO2yFmbMW)o3yWRQ^V*dez9rQ~~ Srxt<$0000true true AutoMapper.Extensions.ExpressionMapping - https://s3.amazonaws.com/automapper/icon.png + icon.png http://automapper.org https://github.com/AutoMapper/AutoMapper.Extensions.ExpressionMapping/blob/master/LICENSE git @@ -24,8 +24,12 @@ true + + + + - + all runtime; build; native; contentfiles; analyzers; buildtransitive @@ -33,4 +37,19 @@ + + + True + True + Resources.resx + + + + + + ResXFileCodeGenerator + Resources.Designer.cs + + + diff --git a/src/AutoMapper.Extensions.ExpressionMapping/ExpressionMapper.cs b/src/AutoMapper.Extensions.ExpressionMapping/ExpressionMapper.cs index b60a4de..8f51b56 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/ExpressionMapper.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/ExpressionMapper.cs @@ -95,7 +95,7 @@ protected override Expression VisitBinary(BinaryExpression node) var newLeft = Visit(node.Left); var newRight = Visit(node.Right); - // check if the non-string expression is a null constent + // check if the non-string expression is a null constant // as this would lead to a "null.ToString()" and thus an error when executing the expression if (newLeft.Type != newRight.Type && newRight.Type == typeof(string) && !IsNullConstant(newLeft)) newLeft = Call(newLeft, typeof(object).GetDeclaredMethod("ToString")); @@ -206,7 +206,7 @@ protected override Expression VisitMember(MemberExpression node) if (constantVisitor.IsConstant) return node; - SetSorceSubTypes(propertyMap); + SetSourceSubTypes(propertyMap); var replacedExpression = Visit(node.Expression); if (replacedExpression == node.Expression) @@ -282,7 +282,7 @@ private PropertyMap GetExistingPropertyMapFor(MemberInfo destinationProperty, Ty return typeMap.PropertyMaps.FirstOrDefault(pm => pm.DestinationName == destinationProperty.Name); } - private void SetSorceSubTypes(PropertyMap propertyMap) + private void SetSourceSubTypes(PropertyMap propertyMap) { if (propertyMap.SourceMember is PropertyInfo info) _destSubTypes = info.PropertyType.GetTypeInfo().GenericTypeArguments.Concat(new[] { info.PropertyType }).ToList(); diff --git a/src/AutoMapper.Extensions.ExpressionMapping/FindMemberExpressionsVisitor.cs b/src/AutoMapper.Extensions.ExpressionMapping/FindMemberExpressionsVisitor.cs index 24234bc..fe1543e 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/FindMemberExpressionsVisitor.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/FindMemberExpressionsVisitor.cs @@ -30,7 +30,7 @@ public MemberExpression Result if (string.IsNullOrEmpty(result) || next.Contains(result)) result = next; else throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, - Resource.includeExpressionTooComplex, + Properties.Resources.includeExpressionTooComplex, string.Concat(_newParentExpression.Type.Name, period, result), string.Concat(_newParentExpression.Type.Name, period, next))); @@ -50,7 +50,7 @@ protected override Expression VisitMember(MemberExpression node) if (node.Expression.NodeType == ExpressionType.MemberAccess && node.Type.IsLiteralType()) _memberExpressions.Add((MemberExpression)node.Expression); else if (node.Expression.NodeType == ExpressionType.Parameter && node.Type.IsLiteralType()) - throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, Resource.mappedMemberIsChildOfTheParameterFormat, node.GetPropertyFullName(), node.Type.FullName, sType.FullName)); + throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, Properties.Resources.mappedMemberIsChildOfTheParameterFormat, node.GetPropertyFullName(), node.Type.FullName, sType.FullName)); else _memberExpressions.Add(node); } diff --git a/src/AutoMapper.Extensions.ExpressionMapping/MapIncludesVisitor.cs b/src/AutoMapper.Extensions.ExpressionMapping/MapIncludesVisitor.cs index a224880..85898e3 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/MapIncludesVisitor.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/MapIncludesVisitor.cs @@ -2,8 +2,6 @@ using System.Collections.Generic; using System.Linq; using System.Linq.Expressions; -using System.Reflection; -using AutoMapper.Internal; using AutoMapper.Extensions.ExpressionMapping.Extensions; using AutoMapper.Extensions.ExpressionMapping.Structures; @@ -18,6 +16,9 @@ public MapIncludesVisitor(IMapper mapper, IConfigurationProvider configurationPr protected override Expression VisitLambda(Expression node) { + if (!node.Body.Type.IsLiteralType()) + return base.VisitLambda(node); + var ex = this.Visit(node.Body); var mapped = Expression.Lambda(ex, node.GetDestinationParameterExpressions(this.InfoDictionary, this.TypeMappings)); @@ -27,6 +28,9 @@ protected override Expression VisitLambda(Expression node) protected override Expression VisitMember(MemberExpression node) { + if (!node.Type.IsLiteralType()) + return base.VisitMember(node); + var parameterExpression = node.GetParameterExpression(); if (parameterExpression == null) return base.VisitMember(node); diff --git a/src/AutoMapper.Extensions.ExpressionMapping/MapperExtensions.cs b/src/AutoMapper.Extensions.ExpressionMapping/MapperExtensions.cs index c8df785..acf9c5a 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/MapperExtensions.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/MapperExtensions.cs @@ -109,7 +109,7 @@ TDestDelegate MapBody(Dictionary typeMappings, XpressionMapperVisito TDestDelegate GetLambda(Dictionary typeMappings, XpressionMapperVisitor visitor, Expression mappedBody) { if (mappedBody == null) - throw new InvalidOperationException(Resource.cantRemapExpression); + throw new InvalidOperationException(Properties.Resources.cantRemapExpression); return (TDestDelegate)Lambda ( @@ -255,7 +255,7 @@ public static List GetDestinationParameterExpressions(this /// public static Dictionary AddTypeMapping(this Dictionary typeMappings, IConfigurationProvider configurationProvider) => typeMappings == null - ? throw new ArgumentException(Resource.typeMappingsDictionaryIsNull) + ? throw new ArgumentException(Properties.Resources.typeMappingsDictionaryIsNull) : typeMappings.AddTypeMapping(configurationProvider, typeof(TSource), typeof(TDest)); private static bool HasUnderlyingType(this Type type) @@ -284,7 +284,7 @@ private static void AddUnderlyingTypes(this Dictionary typeMappings, public static Dictionary AddTypeMapping(this Dictionary typeMappings, IConfigurationProvider configurationProvider, Type sourceType, Type destType) { if (typeMappings == null) - throw new ArgumentException(Resource.typeMappingsDictionaryIsNull); + throw new ArgumentException(Properties.Resources.typeMappingsDictionaryIsNull); if (sourceType.GetTypeInfo().IsGenericType && sourceType.GetGenericTypeDefinition() == typeof(Expression<>)) { @@ -357,7 +357,7 @@ public static Type ReplaceType(this Dictionary typeMappings, Type so private static Dictionary AddTypeMappingsFromDelegates(this Dictionary typeMappings, IConfigurationProvider configurationProvider, Type sourceType, Type destType) { if (typeMappings == null) - throw new ArgumentException(Resource.typeMappingsDictionaryIsNull); + throw new ArgumentException(Properties.Resources.typeMappingsDictionaryIsNull); typeMappings.DoAddTypeMappingsFromDelegates ( @@ -372,7 +372,7 @@ private static Dictionary AddTypeMappingsFromDelegates(this Dictiona private static void DoAddTypeMappingsFromDelegates(this Dictionary typeMappings, IConfigurationProvider configurationProvider, List sourceArguments, List destArguments) { if (sourceArguments.Count != destArguments.Count) - throw new ArgumentException(Resource.invalidArgumentCount); + throw new ArgumentException(Properties.Resources.invalidArgumentCount); for (int i = 0; i < sourceArguments.Count; i++) { @@ -396,7 +396,7 @@ private static void DoAddTypeMappings(this Dictionary typeMappings, private static Type GetSourceMemberType(this PropertyMap propertyMap) => propertyMap.CustomMapExpression != null ? propertyMap.CustomMapExpression.ReturnType - : propertyMap.SourceMember.GetMemberType(); + : propertyMap.SourceMembers.Last().GetMemberType(); private static void FindChildPropertyTypeMaps(this Dictionary typeMappings, IConfigurationProvider ConfigurationProvider, Type source, Type dest) { diff --git a/src/AutoMapper.Extensions.ExpressionMapping/PrependParentNameVisitor.cs b/src/AutoMapper.Extensions.ExpressionMapping/PrependParentNameVisitor.cs index ae40e80..5397f18 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/PrependParentNameVisitor.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/PrependParentNameVisitor.cs @@ -1,7 +1,4 @@ -using AutoMapper.Extensions.ExpressionMapping.Extensions; -using System.Linq; -using System.Linq.Expressions; -using System.Runtime.CompilerServices; +using System.Linq.Expressions; namespace AutoMapper.Extensions.ExpressionMapping { @@ -18,111 +15,16 @@ public PrependParentNameVisitor(ParameterExpression currentParameter, string par public string ParentFullName { get; } public Expression NewParameter { get; } - protected override Expression VisitTypeBinary(TypeBinaryExpression node) + protected override Expression VisitParameter(ParameterExpression node) { - if (!(node.Expression is ParameterExpression)) - return base.VisitTypeBinary(node); - - if (!object.ReferenceEquals(CurrentParameter, node.GetParameterExpression())) - return base.VisitTypeBinary(node); - - return Expression.TypeIs - ( - string.IsNullOrEmpty(ParentFullName) - ? NewParameter - : ExpressionHelpers.MemberAccesses(ParentFullName, NewParameter), - node.TypeOperand - ); - } - - protected override Expression VisitMember(MemberExpression node) - { - if (node.NodeType == ExpressionType.Constant) - return base.VisitMember(node); - - if (!object.ReferenceEquals(CurrentParameter, node.GetParameterExpression()) || !node.IsMemberExpression()) - return base.VisitMember(node); - - return ExpressionHelpers.MemberAccesses - ( - string.IsNullOrEmpty(ParentFullName) - ? node.GetPropertyFullName() - : $"{ParentFullName}.{node.GetPropertyFullName()}", - NewParameter - ); - } - - protected override Expression VisitMethodCall(MethodCallExpression node) - { - if (!IsParentParameterExpression()) - return base.VisitMethodCall(node); - - if (!object.ReferenceEquals(CurrentParameter, node.GetParameterExpression())) - return base.VisitMethodCall(node); - - if (node.Method.IsStatic) + if (object.ReferenceEquals(CurrentParameter, node)) { - if (!IsExtentionMethod()) - return base.VisitMethodCall(node); - - if (node.Method.IsGenericMethod) - return Expression.Call - ( - node.Method.DeclaringType, - node.Method.Name, - node.Method.GetGenericArguments(), - GetNewArgumentsForExtensionMethod() - ); - else - return Expression.Call(node.Method, GetNewArgumentsForExtensionMethod()); - } - - //instance method - if (node.Method.IsGenericMethod) - { - return Expression.Call - ( - GetNewParent(), - node.Method.Name, - node.Method.GetGenericArguments(), - node.Arguments.ToArray() - ); - } - else - { - return Expression.Call - ( - GetNewParent(), - node.Method, - node.Arguments - ); - } - - Expression[] GetNewArgumentsForExtensionMethod() - { - Expression[] arguments = node.Arguments.ToArray(); - arguments[0] = GetNewParent(); - return arguments.ToArray(); - } - - Expression GetNewParent() - => string.IsNullOrEmpty(ParentFullName) - ? NewParameter - : ExpressionHelpers.MemberAccesses(ParentFullName, NewParameter); - - bool IsParentParameterExpression() - { - if (node.Method.IsStatic) - return node.Arguments[0] is ParameterExpression; - - if (!node.Method.IsStatic) - return node.Object is ParameterExpression; - - return false; + return string.IsNullOrEmpty(ParentFullName) + ? NewParameter + : ExpressionHelpers.MemberAccesses(ParentFullName, NewParameter); } - bool IsExtentionMethod() - => node.Method.IsDefined(typeof(ExtensionAttribute), true); + return base.VisitParameter(node); } } } diff --git a/src/AutoMapper.Extensions.ExpressionMapping/Resource.Designer.cs b/src/AutoMapper.Extensions.ExpressionMapping/Properties/Resources.Designer.cs similarity index 88% rename from src/AutoMapper.Extensions.ExpressionMapping/Resource.Designer.cs rename to src/AutoMapper.Extensions.ExpressionMapping/Properties/Resources.Designer.cs index 8eddfcd..461bc9c 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/Resource.Designer.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/Properties/Resources.Designer.cs @@ -8,9 +8,8 @@ // //------------------------------------------------------------------------------ -namespace AutoMapper.Extensions.ExpressionMapping { +namespace AutoMapper.Extensions.ExpressionMapping.Properties { using System; - using System.Reflection; /// @@ -20,17 +19,17 @@ namespace AutoMapper.Extensions.ExpressionMapping { // class via a tool like ResGen or Visual Studio. // To add or remove a member, edit your .ResX file then rerun ResGen // with the /str option, or rebuild your VS project. - [global::System.CodeDom.Compiler.GeneratedCodeAttribute("System.Resources.Tools.StronglyTypedResourceBuilder", "4.0.0.0")] + [global::System.CodeDom.Compiler.GeneratedCodeAttribute("System.Resources.Tools.StronglyTypedResourceBuilder", "17.0.0.0")] [global::System.Diagnostics.DebuggerNonUserCodeAttribute()] [global::System.Runtime.CompilerServices.CompilerGeneratedAttribute()] - internal class Resource { + internal class Resources { private static global::System.Resources.ResourceManager resourceMan; private static global::System.Globalization.CultureInfo resourceCulture; [global::System.Diagnostics.CodeAnalysis.SuppressMessageAttribute("Microsoft.Performance", "CA1811:AvoidUncalledPrivateCode")] - internal Resource() { + internal Resources() { } /// @@ -40,7 +39,7 @@ internal Resource() { internal static global::System.Resources.ResourceManager ResourceManager { get { if (object.ReferenceEquals(resourceMan, null)) { - global::System.Resources.ResourceManager temp = new global::System.Resources.ResourceManager("AutoMapper.Extensions.ExpressionMapping.Resource", typeof(Resource).GetTypeInfo().Assembly); + global::System.Resources.ResourceManager temp = new global::System.Resources.ResourceManager("AutoMapper.Extensions.ExpressionMapping.Properties.Resources", typeof(Resources).Assembly); resourceMan = temp; } return resourceMan; @@ -89,7 +88,7 @@ internal static string customResolversNotSupported { } /// - /// Looks up a localized string similar to The source and destination types must be the same for expression mapping between value types. Source Type: {0}, Source Description: {1}, Destination Type: {2}, Destination Property: {3}.. + /// Looks up a localized string similar to The source and destination types must be the same for expression mapping between literal types. Source Type: {0}, Source Description: {1}, Destination Type: {2}, Destination Property: {3}.. /// internal static string expressionMapValueTypeMustMatchFormat { get { @@ -124,6 +123,15 @@ internal static string invalidExpErr { } } + /// + /// Looks up a localized string similar to For members of literal types, use IMappingExpression.ForMember() to make the parent property types an exact match. Parent Source Type: {0}, Parent Destination Type: {1}, Full Member Name "{2}".. + /// + internal static string makeParentTypesMatchForMembersOfLiteralsFormat { + get { + return ResourceManager.GetString("makeParentTypesMatchForMembersOfLiteralsFormat", resourceCulture); + } + } + /// /// Looks up a localized string similar to The mapped member {0} is of type {1} and a child of the parameter type {2}. No reference type (parent of) {0} is available to map as an include.. /// diff --git a/src/AutoMapper.Extensions.ExpressionMapping/Properties/Resources.resx b/src/AutoMapper.Extensions.ExpressionMapping/Properties/Resources.resx new file mode 100644 index 0000000..da39c6c --- /dev/null +++ b/src/AutoMapper.Extensions.ExpressionMapping/Properties/Resources.resx @@ -0,0 +1,146 @@ + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + text/microsoft-resx + + + 1.3 + + + System.Resources.ResXResourceReader, System.Windows.Forms, Version=2.0.3500.0, Culture=neutral, PublicKeyToken=b77a5c561934e089 + + + System.Resources.ResXResourceWriter, System.Windows.Forms, Version=2.0.3500.0, Culture=neutral, PublicKeyToken=b77a5c561934e089 + + + Cannot create a binary expression for the following pair. Node: {0}, Type: {1} and Node: {2}, Type: {3}. + 0=leftNode; 1=leftNodeType; 2=rightNode; 3=rightNodeType + + + Can't rempa expression + + + The source and destination types must be the same for expression mapping between literal types. Source Type: {0}, Source Description: {1}, Destination Type: {2}, Destination Property: {3}. + 0=Source Type; 1=SourceDescription; 2=Destination Type; 3=Destination Property. + + + The Include value-type expression uses multiple sibling navigation objects "{0}", "{1}" and is too complex to translate. + 0=FirstNavigationProperty, 1=SecondNavigationProperty + + + Source and destination must have the same number of arguments. + + + Invalid expression type for this operation. + + + Mapper Info dictionary cannot be null. + + + SourceMember cannot be null. Source Type: {0}, Destination Type: {1}, Property: {2}. + 0=SorceType; 1=DestinationType; 2=Name of the source property + + + Type Mappings dictionary cannot be null. + + + Custom resolvers are not supported for expression mapping. + + + Arguments must be expressions. + + + The mapped member {0} is of type {1} and a child of the parameter type {2}. No reference type (parent of) {0} is available to map as an include. + 0=memberName, 1=memberType; 2=parameterType + + + For members of literal types, use IMappingExpression.ForMember() to make the parent property types an exact match. Parent Source Type: {0}, Parent Destination Type: {1}, Full Member Name "{2}". + 0=typeSource, 1=typeDestination; 2=sourceFullName + + \ No newline at end of file diff --git a/src/AutoMapper.Extensions.ExpressionMapping/ReflectionExtensions.cs b/src/AutoMapper.Extensions.ExpressionMapping/ReflectionExtensions.cs index 0e99879..22a3c02 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/ReflectionExtensions.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/ReflectionExtensions.cs @@ -11,9 +11,6 @@ namespace AutoMapper.Extensions.ExpressionMapping internal static class ReflectionExtensions { - public static object GetDefaultValue(this ParameterInfo parameter) - => ReflectionHelper.GetDefaultValue(parameter); - public static object MapMember(this ResolutionContext context, MemberInfo member, object value, object destination = null) => ReflectionHelper.MapMember(context, member, value, destination); diff --git a/src/AutoMapper.Extensions.ExpressionMapping/Resource.resx b/src/AutoMapper.Extensions.ExpressionMapping/Resource.resx deleted file mode 100644 index ddc224b..0000000 --- a/src/AutoMapper.Extensions.ExpressionMapping/Resource.resx +++ /dev/null @@ -1,161 +0,0 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - text/microsoft-resx - - - 2.0 - - - System.Resources.ResXResourceReader, System.Windows.Forms, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089 - - - System.Resources.ResXResourceWriter, System.Windows.Forms, Version=4.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089 - - - Cannot create a binary expression for the following pair. Node: {0}, Type: {1} and Node: {2}, Type: {3}. - 0=leftNode; 1=leftNodeType; 2=rightNode; 3=rightNodeType - - - Can't rempa expression - - - The source and destination types must be the same for expression mapping between literal types. Source Type: {0}, Source Description: {1}, Destination Type: {2}, Destination Property: {3}. - 0=Source Type; 1=SourceDescription; 2=Destination Type; 3=Destination Property. - - - The Include value-type expression uses multiple sibling navigation objects "{0}", "{1}" and is too complex to translate. - 0=FirstNavigationProperty, 1=SecondNavigationProperty - - - Source and destination must have the same number of arguments. - - - Invalid expression type for this operation. - - - Mapper Info dictionary cannot be null. - - - SourceMember cannot be null. Source Type: {0}, Destination Type: {1}, Property: {2}. - 0=SorceType; 1=DestinationType; 2=Name of the source property - - - Type Mappings dictionary cannot be null. - - - Custom resolvers are not supported for expression mapping. - - - Arguments must be expressions. - - - The mapped member {0} is of type {1} and a child of the parameter type {2}. No reference type (parent of) {0} is available to map as an include. - 0=memberName, 1=memberType; 2=parameterType - - \ No newline at end of file diff --git a/src/AutoMapper.Extensions.ExpressionMapping/TypeExtensions.cs b/src/AutoMapper.Extensions.ExpressionMapping/TypeExtensions.cs index f8bbc27..34a6f95 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/TypeExtensions.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/TypeExtensions.cs @@ -3,13 +3,9 @@ using System.Collections.Generic; using System.Linq; using System.Reflection; -using System.Reflection.Emit; namespace AutoMapper { -#if NET45 - using System.Reflection.Emit; -#endif internal static class TypeExtensions { @@ -127,14 +123,23 @@ public static bool IsLiteralType(this Type type) if (type.IsNullableType()) type = Nullable.GetUnderlyingType(type); - return LiteralTypes.Contains(type); + return LiteralTypes.Contains(type) || NonNetStandardLiteralTypes.Contains(type.FullName); } private static HashSet LiteralTypes => new HashSet(_literalTypes); + private static readonly HashSet NonNetStandardLiteralTypes = new() + { + "System.DateOnly", + "Microsoft.OData.Edm.Date", + "System.TimeOnly", + "Microsoft.OData.Edm.TimeOfDay" + }; + private static Type[] _literalTypes => new Type[] { typeof(bool), typeof(DateTime), + typeof(DateTimeOffset), typeof(TimeSpan), typeof(Guid), typeof(decimal), diff --git a/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs b/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs index a0fa6f5..5e5f35b 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs @@ -95,18 +95,24 @@ Expression GetMappedMemberExpression(Expression parentExpression, List ExpressionHelpers.MemberAccesses(fullName, visitedParentExpr); private Expression GetMemberExpressionFromCustomExpression(PropertyMapInfo lastWithCustExpression, @@ -134,7 +140,7 @@ Expression PrependParentMemberExpression(PrependParentNameVisitor visitor) ); } - protected Expression GetMemberExpressionFromCustomExpression(List propertyMapInfoList, PropertyMapInfo lastWithCustExpression, Expression mappedParentExpr) + protected Expression GetMemberExpressionFromCustomExpression(List propertyMapInfoList, PropertyMapInfo lastWithCustExpression, Expression mappedParentExpr) => GetMemberExpressionFromCustomExpression ( lastWithCustExpression, @@ -382,7 +388,7 @@ private MemberBinding DoBind(MemberInfo sourceMember, Expression initial, Expres private MemberInfo GetSourceMember(PropertyMap propertyMap) => propertyMap.CustomMapExpression != null ? propertyMap.CustomMapExpression.GetMemberExpression()?.Member - : propertyMap.SourceMember; + : propertyMap.SourceMembers.Last(); private MemberInfo GetParentMember(PropertyMap propertyMap) => propertyMap.IncludedMember?.ProjectToCustomSource != null @@ -614,20 +620,37 @@ private static void AddPropertyMapInfo(Type parentType, string name, List typeSource.IsGenericType() && typeDestination.IsGenericType() && typeSource.GetGenericTypeDefinition() == typeDestination.GetGenericTypeDefinition(); protected void FindDestinationFullName(Type typeSource, Type typeDestination, string sourceFullName, List propertyMapInfoList) { + if (typeSource.IsLiteralType() + && typeDestination.IsLiteralType() + && typeSource != typeDestination) + { + throw new InvalidOperationException + ( + string.Format + ( + CultureInfo.CurrentCulture, + Properties.Resources.makeParentTypesMatchForMembersOfLiteralsFormat, + typeSource, + typeDestination, + sourceFullName + ) + ); + } + const string period = "."; bool BothTypesAreAnonymous() => IsAnonymousType(typeSource) && IsAnonymousType(typeDestination); - TypeMap GetTypeMap() => BothTypesAreAnonymous() + TypeMap GetTypeMap() => BothTypesAreAnonymous() ? anonymousTypesConfigurationProvider.CheckIfTypeMapExists(sourceType: typeDestination, destinationType: typeSource) : ConfigurationProvider.CheckIfTypeMapExists(sourceType: typeDestination, destinationType: typeSource); //The destination becomes the source because to map a source expression to a destination expression, - //we need the expressions used to create the source from the destination + //we need the expressions used to create the source from the destination if (typeSource == typeDestination) { @@ -671,7 +694,7 @@ TypeMap GetTypeMap() => BothTypesAreAnonymous() var sourceType = typeSource.GetFieldOrProperty(propertyName).GetMemberType(); var destType = typeDestination.GetFieldOrProperty(propertyName).GetMemberType(); - + TypeMappings.AddTypeMapping(ConfigurationProvider, sourceType, destType); var childFullName = sourceFullName.Substring(sourceFullName.IndexOf(period, StringComparison.OrdinalIgnoreCase) + 1); @@ -694,13 +717,9 @@ TypeMap GetTypeMap() => BothTypesAreAnonymous() { var propertyMap = typeMap.GetMemberMapByDestinationProperty(sourceFullName); var sourceMemberInfo = typeSource.GetFieldOrProperty(propertyMap.GetDestinationName()); - if (propertyMap.ValueResolverConfig != null) - { - throw new InvalidOperationException(Resource.customResolversNotSupported); - } if (propertyMap.CustomMapExpression == null && !propertyMap.SourceMembers.Any()) - throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, Resource.srcMemberCannotBeNullFormat, typeSource.Name, typeDestination.Name, sourceFullName)); + throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, Properties.Resources.srcMemberCannotBeNullFormat, typeSource.Name, typeDestination.Name, sourceFullName)); // CompareSourceAndDestLiterals // ( @@ -714,7 +733,6 @@ TypeMap GetTypeMap() => BothTypesAreAnonymous() // //switch from IsValueType to IsLiteralType because we do not want to throw an exception for all structs // if ((mappedPropertyType.IsLiteralType() || sourceMemberType.IsLiteralType()) && sourceMemberType != mappedPropertyType) // throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, Resource.expressionMapValueTypeMustMatchFormat, mappedPropertyType.Name, mappedPropertyDescription, sourceMemberType.Name, propertyMap.GetDestinationName())); - // } if (propertyMap.IncludedMember?.ProjectToCustomSource != null) propertyMapInfoList.Add(new PropertyMapInfo(propertyMap.IncludedMember.ProjectToCustomSource, new List())); @@ -728,7 +746,7 @@ TypeMap GetTypeMap() => BothTypesAreAnonymous() var sourceMemberInfo = typeSource.GetFieldOrProperty(propertyMap.GetDestinationName()); if (propertyMap.CustomMapExpression == null && !propertyMap.SourceMembers.Any())//If sourceFullName has a period then the SourceMember cannot be null. The SourceMember is required to find the ProertyMap of its child object. - throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, Resource.srcMemberCannotBeNullFormat, typeSource.Name, typeDestination.Name, propertyName)); + throw new InvalidOperationException(string.Format(CultureInfo.CurrentCulture, Properties.Resources.srcMemberCannotBeNullFormat, typeSource.Name, typeDestination.Name, propertyName)); if (propertyMap.IncludedMember?.ProjectToCustomSource != null) propertyMapInfoList.Add(new PropertyMapInfo(propertyMap.IncludedMember.ProjectToCustomSource, new List())); @@ -737,9 +755,9 @@ TypeMap GetTypeMap() => BothTypesAreAnonymous() var childFullName = sourceFullName.Substring(sourceFullName.IndexOf(period, StringComparison.OrdinalIgnoreCase) + 1); FindDestinationFullName(sourceMemberInfo.GetMemberType(), propertyMap.CustomMapExpression == null - ? propertyMap.SourceMember.GetMemberType() + ? propertyMap.SourceMembers.Last().GetMemberType() : propertyMap.CustomMapExpression.ReturnType, childFullName, propertyMapInfoList); } } } -} \ No newline at end of file +} diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/AutoMapper.Extensions.ExpressionMapping.UnitTests.csproj b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/AutoMapper.Extensions.ExpressionMapping.UnitTests.csproj index 370fd9e..3ef441b 100644 --- a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/AutoMapper.Extensions.ExpressionMapping.UnitTests.csproj +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/AutoMapper.Extensions.ExpressionMapping.UnitTests.csproj @@ -1,13 +1,14 @@  - netcoreapp3.1 + net6.0 false + diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapMismatchedLiteralMemberExpressionsWithoutCustomExpressions.cs b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapMismatchedLiteralMemberExpressionsWithoutCustomExpressions.cs new file mode 100644 index 0000000..cfd5472 --- /dev/null +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapMismatchedLiteralMemberExpressionsWithoutCustomExpressions.cs @@ -0,0 +1,269 @@ +using Microsoft.OData.Edm; +using System; +using System.Linq.Expressions; +using Xunit; + +namespace AutoMapper.Extensions.ExpressionMapping.UnitTests +{ + public class CanMapMismatchedLiteralMemberExpressionsWithoutCustomExpressions + { + [Theory] + [InlineData(nameof(ProductModel.Bool), typeof(bool))] + [InlineData(nameof(ProductModel.DateTime), typeof(DateTime))] + [InlineData(nameof(ProductModel.DateTimeOffset), typeof(DateTimeOffset))] + [InlineData(nameof(ProductModel.Date), typeof(Date))] + [InlineData(nameof(ProductModel.DateOnly), typeof(DateOnly))] + [InlineData(nameof(ProductModel.TimeSpan), typeof(TimeSpan))] + [InlineData(nameof(ProductModel.TimeOfDay), typeof(TimeOfDay))] + [InlineData(nameof(ProductModel.TimeOnly), typeof(TimeOnly))] + [InlineData(nameof(ProductModel.Guid), typeof(Guid))] + [InlineData(nameof(ProductModel.Decimal), typeof(decimal))] + [InlineData(nameof(ProductModel.Byte), typeof(byte))] + [InlineData(nameof(ProductModel.Short), typeof(short))] + [InlineData(nameof(ProductModel.Int), typeof(int))] + [InlineData(nameof(ProductModel.Long), typeof(long))] + [InlineData(nameof(ProductModel.Float), typeof(float))] + [InlineData(nameof(ProductModel.Double), typeof(double))] + [InlineData(nameof(ProductModel.Char), typeof(char))] + [InlineData(nameof(ProductModel.SByte), typeof(sbyte))] + [InlineData(nameof(ProductModel.UShort), typeof(ushort))] + [InlineData(nameof(ProductModel.ULong), typeof(ulong))] + public void CanMapNonNullableToNullableWithoutCustomExpression(string memberName, Type constantType) + { + //arrange + var mapper = GetDataToModelMapper(); + ParameterExpression productParam = Expression.Parameter(typeof(ProductModel), "x"); + MemberExpression property = Expression.MakeMemberAccess(productParam, AutoMapper.Internal.TypeExtensions.GetFieldOrProperty(typeof(ProductModel), memberName)); + object constantValue = Activator.CreateInstance(constantType); + Expression> expression = Expression.Lambda> + ( + Expression.Equal + ( + property, + Expression.Constant(constantValue, constantType) + ), + productParam + ); + Product product = new(); + typeof(Product).GetProperty(memberName).SetValue(product, constantValue); + + //act + var mappedExpression = mapper.MapExpression>>(expression); + + //assert + Assert.True(mappedExpression.Compile()(product)); + } + + [Theory] + [InlineData(nameof(Product.Bool), typeof(bool?))] + [InlineData(nameof(Product.DateTime), typeof(DateTime?))] + [InlineData(nameof(Product.DateTimeOffset), typeof(DateTimeOffset?))] + [InlineData(nameof(Product.Date), typeof(Date?))] + [InlineData(nameof(Product.DateOnly), typeof(DateOnly?))] + [InlineData(nameof(Product.TimeSpan), typeof(TimeSpan?))] + [InlineData(nameof(Product.TimeOfDay), typeof(TimeOfDay?))] + [InlineData(nameof(Product.TimeOnly), typeof(TimeOnly?))] + [InlineData(nameof(Product.Guid), typeof(Guid?))] + [InlineData(nameof(Product.Decimal), typeof(decimal?))] + [InlineData(nameof(Product.Byte), typeof(byte?))] + [InlineData(nameof(Product.Short), typeof(short?))] + [InlineData(nameof(Product.Int), typeof(int?))] + [InlineData(nameof(Product.Long), typeof(long?))] + [InlineData(nameof(Product.Float), typeof(float?))] + [InlineData(nameof(Product.Double), typeof(double?))] + [InlineData(nameof(Product.Char), typeof(char?))] + [InlineData(nameof(Product.SByte), typeof(sbyte?))] + [InlineData(nameof(Product.UShort), typeof(ushort?))] + [InlineData(nameof(Product.ULong), typeof(ulong?))] + public void CanMapNullableToNonNullableWithoutCustomExpression(string memberName, Type constantType) + { + //arrange + var mapper = GetModelToDataMapper(); + ParameterExpression productParam = Expression.Parameter(typeof(Product), "x"); + MemberExpression property = Expression.MakeMemberAccess(productParam, AutoMapper.Internal.TypeExtensions.GetFieldOrProperty(typeof(Product), memberName)); + object constantValue = Activator.CreateInstance(Nullable.GetUnderlyingType(constantType)); + Expression> expression = Expression.Lambda> + ( + Expression.Equal + ( + property, + Expression.Constant(constantValue, constantType) + ), + productParam + ); + ProductModel product = new(); + typeof(ProductModel).GetProperty(memberName).SetValue(product, constantValue); + + //act + var mappedExpression = mapper.MapExpression>>(expression); + + //assert + Assert.True(mappedExpression.Compile()(product)); + } + + [Theory] + [InlineData(nameof(ProductModel.Bool), typeof(bool))] + [InlineData(nameof(ProductModel.DateTime), typeof(DateTime))] + [InlineData(nameof(ProductModel.DateTimeOffset), typeof(DateTimeOffset))] + [InlineData(nameof(ProductModel.Date), typeof(Date))] + [InlineData(nameof(ProductModel.DateOnly), typeof(DateOnly))] + [InlineData(nameof(ProductModel.TimeSpan), typeof(TimeSpan))] + [InlineData(nameof(ProductModel.TimeOfDay), typeof(TimeOfDay))] + [InlineData(nameof(ProductModel.TimeOnly), typeof(TimeOnly))] + [InlineData(nameof(ProductModel.Guid), typeof(Guid))] + [InlineData(nameof(ProductModel.Decimal), typeof(decimal))] + [InlineData(nameof(ProductModel.Byte), typeof(byte))] + [InlineData(nameof(ProductModel.Short), typeof(short))] + [InlineData(nameof(ProductModel.Int), typeof(int))] + [InlineData(nameof(ProductModel.Long), typeof(long))] + [InlineData(nameof(ProductModel.Float), typeof(float))] + [InlineData(nameof(ProductModel.Double), typeof(double))] + [InlineData(nameof(ProductModel.Char), typeof(char))] + [InlineData(nameof(ProductModel.SByte), typeof(sbyte))] + [InlineData(nameof(ProductModel.UShort), typeof(ushort))] + [InlineData(nameof(ProductModel.ULong), typeof(ulong))] + public void CanMapNonNullableSelectorToNullableelectorWithoutCustomExpression(string memberName, Type memberType) + { + var mapper = GetDataToModelMapper(); + ParameterExpression productParam = Expression.Parameter(typeof(ProductModel), "x"); + MemberExpression property = Expression.MakeMemberAccess(productParam, AutoMapper.Internal.TypeExtensions.GetFieldOrProperty(typeof(ProductModel), memberName)); + Type sourceType = typeof(Func<,>).MakeGenericType(typeof(ProductModel), memberType); + Type destType = typeof(Func<,>).MakeGenericType(typeof(Product), memberType); + Type sourceExpressionype = typeof(Expression<>).MakeGenericType(sourceType); + Type destExpressionType = typeof(Expression<>).MakeGenericType(destType); + var expression = Expression.Lambda + ( + sourceType, + property, + productParam + ); + object constantValue = Activator.CreateInstance(memberType); + Product product = new(); + typeof(Product).GetProperty(memberName).SetValue(product, constantValue); + + //act + var mappedExpression = mapper.MapExpression(expression, sourceExpressionype, destExpressionType); + + //assert + Assert.Equal(constantValue, mappedExpression.Compile().DynamicInvoke(product)); + } + + [Theory] + [InlineData(nameof(Product.Bool), typeof(bool?))] + [InlineData(nameof(Product.DateTime), typeof(DateTime?))] + [InlineData(nameof(Product.DateTimeOffset), typeof(DateTimeOffset?))] + [InlineData(nameof(Product.Date), typeof(Date?))] + [InlineData(nameof(Product.DateOnly), typeof(DateOnly?))] + [InlineData(nameof(Product.TimeSpan), typeof(TimeSpan?))] + [InlineData(nameof(Product.TimeOfDay), typeof(TimeOfDay?))] + [InlineData(nameof(Product.TimeOnly), typeof(TimeOnly?))] + [InlineData(nameof(Product.Guid), typeof(Guid?))] + [InlineData(nameof(Product.Decimal), typeof(decimal?))] + [InlineData(nameof(Product.Byte), typeof(byte?))] + [InlineData(nameof(Product.Short), typeof(short?))] + [InlineData(nameof(Product.Int), typeof(int?))] + [InlineData(nameof(Product.Long), typeof(long?))] + [InlineData(nameof(Product.Float), typeof(float?))] + [InlineData(nameof(Product.Double), typeof(double?))] + [InlineData(nameof(Product.Char), typeof(char?))] + [InlineData(nameof(Product.SByte), typeof(sbyte?))] + [InlineData(nameof(Product.UShort), typeof(ushort?))] + [InlineData(nameof(Product.ULong), typeof(ulong?))] + public void CanMapNullableSelectorToNonNullableelectorWithoutCustomExpression(string memberName, Type memberType) + { + var mapper = GetModelToDataMapper(); + ParameterExpression productParam = Expression.Parameter(typeof(Product), "x"); + MemberExpression property = Expression.MakeMemberAccess(productParam, AutoMapper.Internal.TypeExtensions.GetFieldOrProperty(typeof(Product), memberName)); + Type sourceType = typeof(Func<,>).MakeGenericType(typeof(Product), memberType); + Type destType = typeof(Func<,>).MakeGenericType(typeof(ProductModel), memberType); + Type sourceExpressionype = typeof(Expression<>).MakeGenericType(sourceType); + Type destExpressionType = typeof(Expression<>).MakeGenericType(destType); + var expression = Expression.Lambda + ( + sourceType, + property, + productParam + ); + + object constantValue = Activator.CreateInstance(Nullable.GetUnderlyingType(memberType)); + ProductModel product = new(); + typeof(ProductModel).GetProperty(memberName).SetValue(product, constantValue); + + //act + var mappedExpression = mapper.MapExpression(expression, sourceExpressionype, destExpressionType); + + //assert + Assert.Equal(constantValue, mappedExpression.Compile().DynamicInvoke(product)); + } + + private static IMapper GetModelToDataMapper() + { + var config = new MapperConfiguration(c => + { + c.CreateMap(); + }); + config.AssertConfigurationIsValid(); + return config.CreateMapper(); + } + + private static IMapper GetDataToModelMapper() + { + var config = new MapperConfiguration(c => + { + c.CreateMap(); + }); + config.AssertConfigurationIsValid(); + return config.CreateMapper(); + } + + class Product + { + public bool? Bool { get; set; } + public DateTimeOffset? DateTimeOffset { get; set; } + public DateTime? DateTime { get; set; } + public Date? Date { get; set; } + public DateOnly? DateOnly { get; set; } + public TimeSpan? TimeSpan { get; set; } + public TimeOfDay? TimeOfDay { get; set; } + public TimeOnly? TimeOnly { get; set; } + public Guid? Guid { get; set; } + public decimal? Decimal { get; set; } + public byte? Byte { get; set; } + public short? Short { get; set; } + public int? Int { get; set; } + public long? Long { get; set; } + public float? Float { get; set; } + public double? Double { get; set; } + public char? Char { get; set; } + public sbyte? SByte { get; set; } + public ushort? UShort { get; set; } + public uint? UInt { get; set; } + public ulong? ULong { get; set; } + } + + class ProductModel + { + public bool Bool { get; set; } + public DateTimeOffset DateTimeOffset { get; set; } + public DateTime DateTime { get; set; } + public Date Date { get; set; } + public DateOnly DateOnly { get; set; } + public TimeSpan TimeSpan { get; set; } + public TimeOfDay TimeOfDay { get; set; } + public TimeOnly TimeOnly { get; set; } + public Guid Guid { get; set; } + public decimal Decimal { get; set; } + public byte Byte { get; set; } + public short Short { get; set; } + public int Int { get; set; } + public long Long { get; set; } + public float Float { get; set; } + public double Double { get; set; } + public char Char { get; set; } + public sbyte SByte { get; set; } + public ushort UShort { get; set; } + public uint UInt { get; set; } + public ulong ULong { get; set; } + } + } +} diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyFromChildReferenceWithoutMemberExpression.cs b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyFromChildReferenceWithoutMemberExpression.cs new file mode 100644 index 0000000..0c26713 --- /dev/null +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyFromChildReferenceWithoutMemberExpression.cs @@ -0,0 +1,53 @@ +using System; +using System.Collections.Generic; +using System.Linq; +using System.Linq.Expressions; +using Xunit; + +namespace AutoMapper.Extensions.ExpressionMapping.UnitTests +{ + public class CanMapParameterBodyFromChildReferenceWithoutMemberExpression + { + [Fact] + public void Can_map_parameter_body_from_child_reference_without_member_expression() + { + // Arrange + var config = new MapperConfiguration(c => + { + c.CreateMap() + .ForMember(p => p.Brand, c => c.MapFrom(p => EF.Property(p, "BrandId"))); ; + + c.CreateMap() + .IncludeMembers(p => p.Category); + }); + + config.AssertConfigurationIsValid(); + var mapper = config.CreateMapper(); + + var products = new List() { + new TestProduct { } + }.AsQueryable(); + + //Act + Expression> expr = x => x.Brand == 2; + var mappedExpression = mapper.MapExpression>>(expr); + + //Assert + Assert.Equal("x => (Property(x.Category, \"BrandId\") == 2)", mappedExpression.ToString()); + } + + public class TestCategory + { + // Has FK BrandId + } + public class TestProduct + { + public TestCategory? Category { get; set; } + } + + public class TestProductDTO + { + public int Brand { get; set; } + } + } +} diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyWithoutMemberExpression.cs b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyWithoutMemberExpression.cs new file mode 100644 index 0000000..f0e8db3 --- /dev/null +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/CanMapParameterBodyWithoutMemberExpression.cs @@ -0,0 +1,46 @@ +using System; +using System.Collections.Generic; +using System.Linq; +using System.Linq.Expressions; +using Xunit; + +namespace AutoMapper.Extensions.ExpressionMapping.UnitTests +{ + public class CanMapParameterBodyWithoutMemberExpression + { + [Fact] + public void Can_map_parameter_body_without_member_expression() + { + // Arrange + var config = new MapperConfiguration(c => + { + c.CreateMap() + .ForMember(p => p.Brand, c => c.MapFrom(p => EF.Property(p, "BrandId"))); + }); + + config.AssertConfigurationIsValid(); + var mapper = config.CreateMapper(); + + var products = new List() { + new TestProduct { } + }.AsQueryable(); + + //Act + Expression> expr = x => x.Brand == 2; + var mappedExpression = mapper.MapExpression>>(expr); + + //Assert + Assert.Equal("x => (Property(x, \"BrandId\") == 2)", mappedExpression.ToString()); + } + + public class TestProduct + { + // Empty, has shadow key named BrandId + } + + public class TestProductDTO + { + public int Brand { get; set; } + } + } +} diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/EF.cs b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/EF.cs new file mode 100644 index 0000000..6836154 --- /dev/null +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/EF.cs @@ -0,0 +1,10 @@ +namespace AutoMapper.Extensions.ExpressionMapping.UnitTests +{ + internal class EF + { + internal static T Property(object p, string v) + { + return default; + } + } +} diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/MemberMappingsOfLiteralParentTypesMustMatch.cs b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/MemberMappingsOfLiteralParentTypesMustMatch.cs new file mode 100644 index 0000000..27b7a4d --- /dev/null +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/MemberMappingsOfLiteralParentTypesMustMatch.cs @@ -0,0 +1,114 @@ +using System; +using System.Linq.Expressions; +using Xunit; + +namespace AutoMapper.Extensions.ExpressionMapping.UnitTests +{ + public class MemberMappingsOfLiteralParentTypesMustMatch + { + [Fact] + public void MappingMemberOfNullableParentToMemberOfNonNullableParentWithoutCustomExpressionsThrowsException() + { + //arrange + var mapper = GetMapper(); + Expression> expression = x => x.DateTimeOffset.Value.Day.ToString() == "2"; + + //act + var exception = Assert.Throws(() => mapper.MapExpression>>(expression)); + + //assert + Assert.StartsWith + ( + "For members of literal types, use IMappingExpression.ForMember() to make the parent property types an exact match.", + exception.Message + ); + } + + [Fact] + public void MappingMemberOfNonNullableParentToMemberOfNullableParentWithoutCustomExpressionsThrowsException() + { + //arrange + var mapper = GetMapper(); + Expression> expression = x => x.DateTime.Day.ToString() == "2"; + + //act + var exception = Assert.Throws(() => mapper.MapExpression>>(expression)); + + //assert + Assert.StartsWith + ( + "For members of literal types, use IMappingExpression.ForMember() to make the parent property types an exact match.", + exception.Message + ); + } + + [Fact] + public void MappingMemberOfNullableParentToMemberOfNonNullableParentWorksUsingCustomExpressions() + { + //arrange + var mapper = GetMapperWithCustomExpressions(); + Expression> expression = x => x.DateTimeOffset.Value.Day.ToString() == "2"; + + //act + var mappedExpression = mapper.MapExpression>>(expression); + + //assert + Assert.NotNull(mappedExpression); + Func func = mappedExpression.Compile(); + Assert.False(func(new Product { DateTimeOffset = new DateTimeOffset(new DateTime(2000, 3, 3), TimeSpan.Zero) })); + Assert.True(func(new Product { DateTimeOffset = new DateTimeOffset(new DateTime(2000, 2, 2), TimeSpan.Zero) })); + } + + [Fact] + public void MappingMemberOfNonNullableParentToMemberOfNullableParentWorksUsingCustomExpressions() + { + //arrange + var mapper = GetMapperWithCustomExpressions(); + Expression> expression = x => x.DateTime.Day.ToString() == "2"; + + //act + var mappedExpression = mapper.MapExpression>>(expression); + + //assert + Assert.NotNull(mappedExpression); + Func func = mappedExpression.Compile(); + Assert.False(func(new Product { DateTime = new DateTime(2000, 3, 3) })); + Assert.True(func(new Product { DateTime = new DateTime(2000, 2, 2) })); + } + + + private static IMapper GetMapper() + { + var config = new MapperConfiguration(c => + { + c.CreateMap(); + }); + config.AssertConfigurationIsValid(); + return config.CreateMapper(); + } + + private static IMapper GetMapperWithCustomExpressions() + { + var config = new MapperConfiguration(c => + { + c.CreateMap() + .ForMember(d => d.DateTime, o => o.MapFrom(s => s.DateTime.Value)) + .ForMember(d => d.DateTimeOffset, o => o.MapFrom(s => (DateTimeOffset?)s.DateTimeOffset)); + }); + config.AssertConfigurationIsValid(); + return config.CreateMapper(); + } + + class Product + { + public DateTime? DateTime { get; set; } + public DateTimeOffset DateTimeOffset { get; set; } + } + + class ProductModel + { + public DateTime DateTime { get; set; } + public DateTimeOffset? DateTimeOffset { get; set; } + } + } +} diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/ShouldThrowInvalidOperationExceptionForUnmatchedLiterals.cs b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/ShouldThrowInvalidOperationExceptionForUnmatchedLiterals.cs new file mode 100644 index 0000000..6fb88f9 --- /dev/null +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/ShouldThrowInvalidOperationExceptionForUnmatchedLiterals.cs @@ -0,0 +1,140 @@ +using Microsoft.OData.Edm; +using System; +using System.Linq.Expressions; +using Xunit; + +namespace AutoMapper.Extensions.ExpressionMapping.UnitTests +{ + public class ShouldThrowInvalidOperationExceptionForUnmatchedLiterals + { + [Theory] + [InlineData(nameof(ProductModel.Bool), typeof(bool?))] + [InlineData(nameof(ProductModel.DateTime), typeof(DateTime?))] + [InlineData(nameof(ProductModel.DateTimeOffset), typeof(DateTimeOffset?))] + [InlineData(nameof(ProductModel.Date), typeof(Date?))] + [InlineData(nameof(ProductModel.DateOnly), typeof(DateOnly?))] + [InlineData(nameof(ProductModel.TimeSpan), typeof(TimeSpan?))] + [InlineData(nameof(ProductModel.TimeOfDay), typeof(TimeOfDay?))] + [InlineData(nameof(ProductModel.TimeOnly), typeof(TimeOnly?))] + [InlineData(nameof(ProductModel.Guid), typeof(Guid?))] + [InlineData(nameof(ProductModel.Decimal), typeof(decimal?))] + [InlineData(nameof(ProductModel.Byte), typeof(byte?))] + [InlineData(nameof(ProductModel.Short), typeof(short?))] + [InlineData(nameof(ProductModel.Int), typeof(int?))] + [InlineData(nameof(ProductModel.Long), typeof(long?))] + [InlineData(nameof(ProductModel.Float), typeof(float?))] + [InlineData(nameof(ProductModel.Double), typeof(double?))] + [InlineData(nameof(ProductModel.Char), typeof(char?))] + [InlineData(nameof(ProductModel.SByte), typeof(sbyte?))] + [InlineData(nameof(ProductModel.UShort), typeof(ushort?))] + [InlineData(nameof(ProductModel.ULong), typeof(ulong?))] + public void ThrowsCreatingBinaryExpressionCombiningNonNullableParameterWithNullableConstant(string memberName, Type constantType) + { + ParameterExpression productParam = Expression.Parameter(typeof(ProductModel), "x"); + MemberExpression property = Expression.MakeMemberAccess(productParam, AutoMapper.Internal.TypeExtensions.GetFieldOrProperty(typeof(ProductModel), memberName)); + + Assert.Throws + ( + () => Expression.Lambda> + ( + Expression.Equal + ( + property, + Expression.Constant(Activator.CreateInstance(constantType), constantType) + ), + productParam + ) + ); + } + + [Theory] + [InlineData(nameof(Product.Bool), typeof(bool))] + [InlineData(nameof(Product.DateTime), typeof(DateTime))] + [InlineData(nameof(Product.DateTimeOffset), typeof(DateTimeOffset))] + [InlineData(nameof(Product.Date), typeof(Date))] + [InlineData(nameof(Product.DateOnly), typeof(DateOnly))] + [InlineData(nameof(Product.TimeSpan), typeof(TimeSpan))] + [InlineData(nameof(Product.TimeOfDay), typeof(TimeOfDay))] + [InlineData(nameof(Product.TimeOnly), typeof(TimeOnly))] + [InlineData(nameof(Product.Guid), typeof(Guid))] + [InlineData(nameof(Product.Decimal), typeof(decimal))] + [InlineData(nameof(Product.Byte), typeof(byte))] + [InlineData(nameof(Product.Short), typeof(short))] + [InlineData(nameof(Product.Int), typeof(int))] + [InlineData(nameof(Product.Long), typeof(long))] + [InlineData(nameof(Product.Float), typeof(float))] + [InlineData(nameof(Product.Double), typeof(double))] + [InlineData(nameof(Product.Char), typeof(char))] + [InlineData(nameof(Product.SByte), typeof(sbyte))] + [InlineData(nameof(Product.UShort), typeof(ushort))] + [InlineData(nameof(Product.ULong), typeof(ulong))] + public void ThrowsCreatingBinaryExpressionCombiningNullableParameterWithNonNullableConstant(string memberName, Type constantType) + { + ParameterExpression productParam = Expression.Parameter(typeof(Product), "x"); + MemberExpression property = Expression.MakeMemberAccess(productParam, AutoMapper.Internal.TypeExtensions.GetFieldOrProperty(typeof(Product), memberName)); + + var ex = Assert.Throws + ( + () => Expression.Lambda> + ( + Expression.Equal + ( + property, + Expression.Constant(Activator.CreateInstance(constantType), constantType) + ), + productParam + ) + ); + } + + class Product + { + public bool? Bool { get; set; } + public DateTimeOffset? DateTimeOffset { get; set; } + public DateTime? DateTime { get; set; } + public Date? Date { get; set; } + public DateOnly? DateOnly { get; set; } + public TimeSpan? TimeSpan { get; set; } + public TimeOfDay? TimeOfDay { get; set; } + public TimeOnly? TimeOnly { get; set; } + public Guid? Guid { get; set; } + public decimal? Decimal { get; set; } + public byte? Byte { get; set; } + public short? Short { get; set; } + public int? Int { get; set; } + public long? Long { get; set; } + public float? Float { get; set; } + public double? Double { get; set; } + public char? Char { get; set; } + public sbyte? SByte { get; set; } + public ushort? UShort { get; set; } + public uint? UInt { get; set; } + public ulong? ULong { get; set; } + } + + class ProductModel + { + public bool Bool { get; set; } + public DateTimeOffset DateTimeOffset { get; set; } + public DateTime DateTime { get; set; } + public Date Date { get; set; } + public DateOnly DateOnly { get; set; } + public TimeSpan TimeSpan { get; set; } + public TimeOfDay TimeOfDay { get; set; } + public TimeOnly TimeOnly { get; set; } + public Guid Guid { get; set; } + public decimal Decimal { get; set; } + public byte Byte { get; set; } + public short Short { get; set; } + public int Int { get; set; } + public long Long { get; set; } + public float Float { get; set; } + public double Double { get; set; } + public char Char { get; set; } + public sbyte SByte { get; set; } + public ushort UShort { get; set; } + public uint UInt { get; set; } + public ulong ULong { get; set; } + } + } +} diff --git a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/XpressionMapperTests.cs b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/XpressionMapperTests.cs index b923e47..0346583 100644 --- a/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/XpressionMapperTests.cs +++ b/tests/AutoMapper.Extensions.ExpressionMapping.UnitTests/XpressionMapperTests.cs @@ -147,6 +147,20 @@ public void Map_includes_trim_string_nested_in_select_using_explicit_types() Assert.True(cars.Count == 4); } + private static bool IncludeTest(IQueryable model, Expression> navigationPropertyPath) + { + return !model.Equals(navigationPropertyPath); + } + + [Fact] + public void Map_includes_with_include_method_call() + { + Expression, bool>> selection = s => IncludeTest(s, s => s.ValidLines); + var expression = mapper.MapExpressionAsInclude, bool>>>(selection); + + Assert.True(expression.Compile()(new List() { new PurchaseOrder() }.AsQueryable())); + } + [Fact] public void Map_object_type_change() { @@ -625,8 +639,8 @@ public void Map_orderBy_thenBy_To_Dictionary_Select_expression_without_generic_t //Act Expression, IEnumerable>> expMapped = (Expression, IEnumerable>>)mapper.MapExpression ( - grouped, - typeof(Expression, IEnumerable>>), + grouped, + typeof(Expression, IEnumerable>>), typeof(Expression, IEnumerable>>) ); @@ -1019,7 +1033,7 @@ public void Can_map_expression_with_multiple_destination_parameters_of_the_same_ Assert.Equal (//parameters ch and c are of the same type - "c => ((c.Finished == null) AndAlso Not(c.Part.History.Any(ch => (ch.Started > c.Started))))", + "c => ((c.Finished == null) AndAlso Not(c.Part.History.Any(ch => (ch.Started > c.Started))))", mappedExpression.ToString() ); } @@ -1094,7 +1108,7 @@ private static void SetupQueryableCollection() new Thing { Bar = "Bar4", Car = new Car { Color = "White", Year = 2015 } } }, Type = "Business", - Users = new User[] + Users = new User[] { new User { @@ -1265,6 +1279,16 @@ public interface IUserModel : IIdentifiable bool IsActive { get; set; } } + public class PurchaseOrder + { + public IList Lines { get; set; } + } + + public class PurchaseOrderDTO + { + public IEnumerable ValidLines { get; set; } + } + public class OrderLine { public int Id { get; set; } @@ -1415,7 +1439,7 @@ class ItemEntity public string Name { get; set; } } - class ListParentExtension + class ListParentExtension { public ListExtension List { get; set; } } @@ -1588,6 +1612,9 @@ public OrganizationProfile() CreateMap() .ForMember(dest => dest.CreateDate, opts => opts.MapFrom(x => x.Date)); + CreateMap() + .ForMember(d => d.ValidLines, opt => opt.MapFrom(s => s.Lines.Where(l => l.Quantity > 0))); + CreateMap() .ForMember(dto => dto.Name, conf => conf.MapFrom(src => src.ItemName)); CreateMap() From fe58d96a3a1a0ffed86f7d7d72a3c00dc44c4de9 Mon Sep 17 00:00:00 2001 From: Matt Morrissette Date: Fri, 2 Dec 2022 18:19:43 -0800 Subject: [PATCH 2/3] Ignore errors from new code as it breaks RH360s expression mapping --- .../XpressionMapperVisitor.cs | 20 +++++++++++++++++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs b/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs index 5e5f35b..ee7efad 100644 --- a/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs +++ b/src/AutoMapper.Extensions.ExpressionMapping/XpressionMapperVisitor.cs @@ -95,8 +95,17 @@ Expression GetMappedMemberExpression(Expression parentExpression, List Date: Fri, 2 Dec 2022 18:39:24 -0800 Subject: [PATCH 3/3] Update tag version --- .github/workflows/ci.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 904c5fe..8e75f4c 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -4,11 +4,11 @@ on: push: branches: - master - - v6.0.2 + - v6.0.3 pull_request: branches: - master - - v6.0.2 + - v6.0.3 jobs: build: