Merge pull request #68 from LuccaSA/Test-ExpressionReducer

FIX : reducer on concrete classe using abstract property

Author: Poltuu

.gitignore

@@ -1,4 +1,5 @@
-*/bin/*
+
+*/bin/*
 */obj/*
 
 *.suo
@@ -18,4 +19,4 @@ packages/# NuGet Packages
 /OpenCover.4.6.519
 /ComputeVersion.ps1
 /.versionCounter
-/NExtends/*.user
+/NExtends/*.user

NExtends.Tests/Expressions/ExpressionReducerTests.cs

@@ -1,6 +1,7 @@
 using NExtends.Expressions;
 using System;
 using System.Collections.Generic;
+using System.Linq.Expressions;
 using Xunit;
 
 namespace NExtends.Tests.Expressions
@@ -20,9 +21,9 @@ public UserDate(User owner, DateTime date)
 
 	public class ExpressionReducerTests
 	{
-		DateTime today = DateTime.Today;
-		List<int> elements = new List<int> { 0, 1, 2 };
-		UserDate fakeUd = new UserDate(new User(), DateTime.Today);
+        readonly DateTime today = DateTime.Today;
+        readonly List<int> elements = new List<int> { 0, 1, 2 };
+        readonly UserDate fakeUd = new UserDate(new User(), DateTime.Today);
 
 		[Fact]
 		public void ExpressionReducerEmptyLogic()
@@ -111,5 +112,22 @@ public void ExpressionReducerInterfaceMember()
 
 			Assert.Equal("u => (u.Content != null)", result.ToString());
 		}
-	}
+
+        abstract class AbstractTruc
+        {
+            public User User { get; set; }
+        }
+
+        class ConcreteBiduleContainer : AbstractTruc
+        {
+            public User User2 { get; set; }
+        }
+
+        [Fact]
+        public void ExpressionReducerAbstractClassMember()
+        {
+            var result = ExpressionReducer<ConcreteBiduleContainer, User>.Reduce(bc => bc.User.Id != 0 && bc.User2.Id != 23, u => u.User, true);
+            Assert.Equal("u => (u.Id != 0)", result.ToString());
+        }
+    }
 }

NExtends/Expressions/ExpressionReducer.cs

@@ -120,7 +120,7 @@ protected override Expression VisitMember(MemberExpression node)
 			var result = base.VisitMember(node);
 
 			//Même membre, branché sur le paramètre => on remplace
-			if (CurrentNodeDependsOnPreviousParameter && node.Member == Reducer.Member)
+			if (CurrentNodeDependsOnPreviousParameter && AreEquals(node.Member, Reducer.Member))
 			{
 				CurrentNodeDependsOnPreviousParameter = false;
 				result = Parameter;
@@ -131,7 +131,14 @@ protected override Expression VisitMember(MemberExpression node)
 			return result;
 		}
 
-		protected override Expression VisitParameter(ParameterExpression node)
+        private bool AreEquals(MemberInfo member1, MemberInfo member2)
+        {
+            return member1.DeclaringType == member2.DeclaringType
+                && member1.Name == member2.Name;
+        }
+
+
+        protected override Expression VisitParameter(ParameterExpression node)
 		{
 			//on sauvegarde la dépendance du noeud courant au paramètre d'entrée
 			CurrentNodeDependsOnPreviousParameter = true;