Address code review feedback: explicit private modifiers, helper method, remove duplicate test

Agent-Logs-Url: https://github.com/EFNext/EntityFrameworkCore.Projectables/sessions/a64490fa-46e4-49f0-8bf9-d507f70ef85d

Co-authored-by: PhenX <42170+PhenX@users.noreply.github.com>

Author: Copilot

tests/EntityFrameworkCore.Projectables.VendorTests/EFCoreBulkExtensionsCompatibilityTests.cs

@@ -19,7 +19,7 @@ namespace EntityFrameworkCore.Projectables.VendorTests;
 /// </summary>
 public class EFCoreBulkExtensionsCompatibilityTests : IDisposable
 {
-    readonly TestDbContext _context;
+    private readonly TestDbContext _context;
 
     public EFCoreBulkExtensionsCompatibilityTests()
     {
@@ -59,20 +59,6 @@ public void GetDbContext_WithProjectablesEnabled_ReturnsCorrectContext()
         Assert.Same(_context, dbContext);
     }
 
-    [Fact]
-    public void GetDbContext_WithProjectableProperty_DoesNotThrow()
-    {
-        // Arrange – entity with a [Projectable] property so that CustomQueryCompiler is
-        // exercised with actual projectable expression expansion.
-        var query = _context.Set<Order>().Where(o => o.IsCompleted);
-
-        // Act
-        var exception = Record.Exception(() => BatchUtil.GetDbContext(query));
-
-        // Assert
-        Assert.Null(exception);
-    }
-
     [Fact]
     public async Task BatchDeleteAsync_WithProjectablesEnabled_DoesNotThrowTargetException()
     {
@@ -87,16 +73,11 @@ public async Task BatchDeleteAsync_WithProjectablesEnabled_DoesNotThrowTargetExc
             () => query.BatchDeleteAsync(TestContext.Current.CancellationToken));
 #pragma warning restore CS0618
 
-        // Assert – a TargetException means the reflection-based DbContext discovery
-        // inside EFCore.BulkExtensions failed.  All other exceptions (e.g. SQL syntax
-        // differences on SQLite) are acceptable because they come from actual SQL
-        // execution, not from the broken reflection chain.
-        Assert.False(
-            exception is System.Reflection.TargetException,
-            $"BatchDeleteAsync threw TargetException: {exception?.Message}");
-        Assert.False(
-            exception?.Message?.Contains("Non-static method requires a target") == true,
-            $"BatchDeleteAsync threw 'Non-static method requires a target': {exception?.Message}");
+        // A TargetException means the reflection-based DbContext discovery inside
+        // EFCore.BulkExtensions failed. Other exceptions (e.g. SQL syntax differences
+        // on SQLite) are acceptable because they come from actual SQL execution, not
+        // from the broken reflection chain.
+        AssertNoTargetException(exception, "BatchDeleteAsync");
     }
 
     [Fact]
@@ -113,12 +94,12 @@ public async Task BatchUpdateAsync_WithProjectablesEnabled_DoesNotThrowTargetExc
                 cancellationToken: TestContext.Current.CancellationToken));
 #pragma warning restore CS0618
 
-        // Assert – same as above: only TargetException is a regression.
-        Assert.False(
-            exception is System.Reflection.TargetException,
-            $"BatchUpdateAsync threw TargetException: {exception?.Message}");
-        Assert.False(
-            exception?.Message?.Contains("Non-static method requires a target") == true,
-            $"BatchUpdateAsync threw 'Non-static method requires a target': {exception?.Message}");
+        AssertNoTargetException(exception, "BatchUpdateAsync");
     }
+
+    private static void AssertNoTargetException(Exception? exception, string operationName)
+        => Assert.False(
+            exception is System.Reflection.TargetException,
+            $"{operationName} threw TargetException (\"Non-static method requires a target\"). " +
+            $"This indicates that CustomQueryCompiler's _queryContextFactory shadow field is missing.");
 }

tests/EntityFrameworkCore.Projectables.VendorTests/TestContext.cs

@@ -24,7 +24,7 @@ public class TestDbContext : DbContext
 {
     // Keep the connection open for the lifetime of the context so the in-memory
     // SQLite database is not destroyed between operations.
-    readonly SqliteConnection _connection;
+    private readonly SqliteConnection _connection;
 
     public TestDbContext()
     {