Merge pull request #64 from koenbeuk/issue-63

Don't throw on private and protected properties

Author: koenbeuk

src/EntityFrameworkCore.Projectables/Extensions/TypeExtensions.cs

@@ -89,7 +89,7 @@ public static MethodInfo GetOverridingMethod(this Type derivedType, MethodInfo m
 
         public static PropertyInfo GetOverridingProperty(this Type derivedType, PropertyInfo propertyInfo)
         {
-            var accessor = propertyInfo.GetAccessors()[0];
+            var accessor = propertyInfo.GetAccessors(true)[0];
 
             if (!derivedType.CanHaveOverridingMethod(accessor))
             {

tests/EntityFrameworkCore.Projectables.FunctionalTests/PrivateProjectables.Issue63Repro.verified.txt

@@ -0,0 +1,2 @@
+SELECT [e].[Id]
+FROM [Entity] AS [e]

tests/EntityFrameworkCore.Projectables.FunctionalTests/PrivateProjectables.cs

@@ -0,0 +1,34 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using EntityFrameworkCore.Projectables.FunctionalTests.Helpers;
+using Microsoft.EntityFrameworkCore;
+using VerifyXunit;
+using Xunit;
+
+namespace EntityFrameworkCore.Projectables.FunctionalTests
+{
+    [UsesVerify]
+    public class PrivateProjectables
+    {
+        public record Entity
+        {
+            public int Id { get; set; }
+        }
+
+        bool IsAdmin => true;
+
+        [Fact]
+        public Task Issue63Repro()
+        {
+            using var dbContext = new SampleDbContext<Entity>();
+
+            var query = dbContext.Set<Entity>()
+                    .Where(product => IsAdmin || product.Id == 1);
+
+            return Verifier.Verify(query.ToQueryString());
+        }
+    }
+}