Refactor access modifiers to internal and add more documentation for public members

fabien.menager · fabien.menager · commit 5157d9daf7a8 · 2025-05-13T18:37:59.000+02:00
diff --git a/README.md b/README.md
@@ -74,7 +74,7 @@ await dbContext.ExecuteInsertWithIdentityAsync(entities, options => {});
 
 ## Benchmarks
 Benchmark projects are available in the [`tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark`](tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark/LibComparator.cs) directory.
-Run them to compare performance with other libraries (https://github.com/videokojot/EFCore.BulkExtensions.MIT and https://entityframework-extensions.net/bulk-extensions),
+Run them to compare performance with other libraries (https://github.com/borisdj/EFCore.BulkExtensions and https://entityframework-extensions.net/bulk-extensions),
 using optimized configuration (local Docker is required).
 
 There is no need to compare with basic EF Core SaveChangesAsync, as it is significantly slower.
diff --git a/src/Directory.Build.props b/src/Directory.Build.props
@@ -8,7 +8,7 @@
         <NeutralLanguage>en</NeutralLanguage>
         <EnableNETAnalyzers>true</EnableNETAnalyzers>
 <!--        <AnalysisMode>AllEnabledByDefault</AnalysisMode>-->
-        <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
+        <TreatWarningsAsErrors Condition="$(Configuration) != 'Debug'">true</TreatWarningsAsErrors>
     </PropertyGroup>
 
     <PropertyGroup>
@@ -32,7 +32,7 @@
     </ItemGroup>
 
     <PropertyGroup Label="Build">
-<!--        <GenerateDocumentationFile>true</GenerateDocumentationFile>-->
+        <GenerateDocumentationFile>true</GenerateDocumentationFile>
         <SymbolPackageFormat>snupkg</SymbolPackageFormat>
     </PropertyGroup>
 
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql.csproj b/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql.csproj
@@ -5,8 +5,8 @@
     </ItemGroup>
 
     <ItemGroup>
-      <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Condition="'$(TargetFramework)' == 'net8.0'" Version="8.0.11" />
-      <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Condition="'$(TargetFramework)' == 'net9.0'" Version="9.0.4" />
+      <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Condition="'$(TargetFramework)' == 'net8.0'" Version="8.*" />
+      <PackageReference Include="Npgsql.EntityFrameworkCore.PostgreSQL" Condition="'$(TargetFramework)' == 'net9.0'" Version="9.*" />
     </ItemGroup>
 
 </Project>
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PostgreSqlBulkInsertProvider.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PostgreSqlBulkInsertProvider.cs
@@ -9,7 +9,7 @@
 namespace PhenX.EntityFrameworkCore.BulkInsert.PostgreSql;
 
 [UsedImplicitly]
-public class PostgreSqlBulkInsertProvider : BulkInsertProviderBase<PostgreSqlDialectBuilder>
+internal class PostgreSqlBulkInsertProvider : BulkInsertProviderBase<PostgreSqlDialectBuilder>
 {
     //language=sql
     protected override string CreateTableCopySql => "CREATE TEMPORARY TABLE {0} AS TABLE {1} WITH NO DATA;";
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PostgreSqlDbContextOptionsExtensions.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PostgreSqlDbContextOptionsExtensions.cs
@@ -3,8 +3,14 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.PostgreSql;
 
+/// <summary>
+/// DbContext options extension for PostgreSQL.
+/// </summary>
 public static class PostgreSqlDbContextOptionsExtensions
 {
+    /// <summary>
+    /// Configures the DbContext to use the PostgreSQL bulk insert provider.
+    /// </summary>
     public static DbContextOptionsBuilder UseExecuteInsertPostgres(this DbContextOptionsBuilder optionsBuilder)
     {
         var extension = optionsBuilder.Options.FindExtension<ExecuteInsertOptionsExtension<PostgreSqlBulkInsertProvider>>() ?? new ExecuteInsertOptionsExtension<PostgreSqlBulkInsertProvider>();
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PostgreSqlDialectBuilder.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.PostgreSql/PostgreSqlDialectBuilder.cs
@@ -2,7 +2,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.PostgreSql;
 
-public class PostgreSqlDialectBuilder : SqlDialectBuilder
+internal class PostgreSqlDialectBuilder : SqlDialectBuilder
 {
     protected override string OpenDelimiter => "\"";
     protected override string CloseDelimiter => "\"";
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/PhenX.EntityFrameworkCore.BulkInsert.SqlServer.csproj b/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/PhenX.EntityFrameworkCore.BulkInsert.SqlServer.csproj
@@ -5,8 +5,8 @@
     </ItemGroup>
 
     <ItemGroup>
-      <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Condition="'$(TargetFramework)' == 'net8.0'" Version="8.0.15" />
-      <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Condition="'$(TargetFramework)' == 'net9.0'" Version="9.0.4" />
+      <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Condition="'$(TargetFramework)' == 'net8.0'" Version="8.*" />
+      <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Condition="'$(TargetFramework)' == 'net9.0'" Version="9.*" />
     </ItemGroup>
 
 </Project>
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/SqlServerBulkInsertProvider.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/SqlServerBulkInsertProvider.cs
@@ -8,7 +8,7 @@
 namespace PhenX.EntityFrameworkCore.BulkInsert.SqlServer;
 
 [UsedImplicitly]
-public class SqlServerBulkInsertProvider : BulkInsertProviderBase<SqlServerDialectBuilder>
+internal class SqlServerBulkInsertProvider : BulkInsertProviderBase<SqlServerDialectBuilder>
 {
     //language=sql
     protected override string CreateTableCopySql => "SELECT {2} INTO {0} FROM {1} WHERE 1 = 0;";
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/SqlServerDbContextOptionsExtensions.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/SqlServerDbContextOptionsExtensions.cs
@@ -3,8 +3,14 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.SqlServer;
 
+/// <summary>
+/// DbContext options extension for SQL Server.
+/// </summary>
 public static class SqlServerDbContextOptionsExtensions
 {
+    /// <summary>
+    /// Configures the DbContext to use the SQL Server bulk insert provider.
+    /// </summary>
     public static DbContextOptionsBuilder UseExecuteInsertSqlServer(this DbContextOptionsBuilder optionsBuilder)
     {
         var extension = optionsBuilder.Options.FindExtension<ExecuteInsertOptionsExtension<SqlServerBulkInsertProvider>>() ?? new ExecuteInsertOptionsExtension<SqlServerBulkInsertProvider>();
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/SqlServerDialectBuilder.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.SqlServer/SqlServerDialectBuilder.cs
@@ -9,7 +9,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.SqlServer;
 
-public class SqlServerDialectBuilder : SqlDialectBuilder
+internal class SqlServerDialectBuilder : SqlDialectBuilder
 {
     protected override string OpenDelimiter => "[";
     protected override string CloseDelimiter => "]";
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/PhenX.EntityFrameworkCore.BulkInsert.Sqlite.csproj b/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/PhenX.EntityFrameworkCore.BulkInsert.Sqlite.csproj
@@ -5,8 +5,8 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Condition="'$(TargetFramework)' == 'net8.0'" Version="8.0.11" />
-    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Condition="'$(TargetFramework)' == 'net9.0'" Version="9.0.4" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Condition="'$(TargetFramework)' == 'net8.0'" Version="8.*" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Condition="'$(TargetFramework)' == 'net9.0'" Version="9.*" />
   </ItemGroup>
 
 </Project>
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/SqliteBulkInsertProvider.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/SqliteBulkInsertProvider.cs
@@ -11,7 +11,7 @@
 namespace PhenX.EntityFrameworkCore.BulkInsert.Sqlite;
 
 [UsedImplicitly]
-public class SqliteBulkInsertProvider : BulkInsertProviderBase<SqliteDialectBuilder>
+internal class SqliteBulkInsertProvider : BulkInsertProviderBase<SqliteDialectBuilder>
 {
     protected override string BulkInsertId => "rowid";
 
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/SqliteDbContextOptionsExtensions.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/SqliteDbContextOptionsExtensions.cs
@@ -3,8 +3,14 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.Sqlite;
 
+/// <summary>
+/// DbContext options extension for SQLite.
+/// </summary>
 public static class SqliteDbContextOptionsExtensions
 {
+    /// <summary>
+    /// Configures the DbContext to use the SQLite bulk insert provider.
+    /// </summary>
     public static DbContextOptionsBuilder UseExecuteInsertSqlite(this DbContextOptionsBuilder optionsBuilder)
     {
         var extension = optionsBuilder.Options.FindExtension<ExecuteInsertOptionsExtension<SqliteBulkInsertProvider>>() ?? new ExecuteInsertOptionsExtension<SqliteBulkInsertProvider>();
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/SqliteDialectBuilder.cs b/src/PhenX.EntityFrameworkCore.BulkInsert.Sqlite/SqliteDialectBuilder.cs
@@ -2,7 +2,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.Sqlite;
 
-public class SqliteDialectBuilder : SqlDialectBuilder
+internal class SqliteDialectBuilder : SqlDialectBuilder
 {
     protected override string OpenDelimiter => "\"";
     protected override string CloseDelimiter => "\"";
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/Abstractions/IBulkInsertProvider.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/Abstractions/IBulkInsertProvider.cs
@@ -4,8 +4,14 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.Abstractions;
 
-public interface IBulkInsertProvider
+/// <summary>
+/// Internal bulk insert provider interface.
+/// </summary>
+internal interface IBulkInsertProvider
 {
+    /// <summary>
+    /// Calls the provider to perform a bulk insert operation.
+    /// </summary>
     internal Task<List<T>> BulkInsertWithIdentityAsync<T>(
         DbContext context,
         IEnumerable<T> entities,
@@ -14,6 +20,9 @@ internal Task<List<T>> BulkInsertWithIdentityAsync<T>(
         CancellationToken ctk = default
     ) where T : class;
 
+    /// <summary>
+    /// Calls the provider to perform a bulk insert operation without returning the inserted entities.
+    /// </summary>
     internal Task BulkInsertWithoutReturnAsync<T>(
         DbContext context,
         IEnumerable<T> entities,
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/BulkInsertProviderBase.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/BulkInsertProviderBase.cs
@@ -10,7 +10,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert;
 
-public abstract class BulkInsertProviderBase<TDialect> : IBulkInsertProvider
+internal abstract class BulkInsertProviderBase<TDialect> : IBulkInsertProvider
     where TDialect : SqlDialectBuilder, new()
 {
     protected readonly TDialect SqlDialect = new();
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/Dialect/RawSqlValue.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/Dialect/RawSqlValue.cs
@@ -6,5 +6,8 @@
 /// <param name="sql"></param>
 public class RawSqlValue(string sql)
 {
+    /// <summary>
+    /// The raw SQL value.
+    /// </summary>
     public string Sql { get; } = sql;
 }
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/Dialect/SqlDialectBuilder.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/Dialect/SqlDialectBuilder.cs
@@ -8,7 +8,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.Dialect;
 
-public abstract class SqlDialectBuilder
+internal abstract class SqlDialectBuilder
 {
     protected abstract string OpenDelimiter { get; }
     protected abstract string CloseDelimiter { get; }
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/EnumerableDataReader.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/EnumerableDataReader.cs
@@ -2,7 +2,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert;
 
-public class EnumerableDataReader<T> : IDataReader
+internal class EnumerableDataReader<T> : IDataReader
 {
     private readonly IEnumerator<T> _enumerator;
     private readonly PropertyAccessor[] _properties;
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/ExecuteInsertOptionsExtension.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/ExecuteInsertOptionsExtension.cs
@@ -5,7 +5,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert;
 
-public class ExecuteInsertOptionsExtension<TProvider> : IDbContextOptionsExtension
+internal class ExecuteInsertOptionsExtension<TProvider> : IDbContextOptionsExtension
     where TProvider : class, IBulkInsertProvider
 {
     public DbContextOptionsExtensionInfo Info
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/Extensions/DbContextExtensions.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/Extensions/DbContextExtensions.cs
@@ -6,7 +6,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.Extensions;
 
-public static class DbContextExtensions
+internal static class DbContextExtensions
 {
     /// <summary>
     /// Gets cached properties for an entity type, using reflection if not already cached.
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/Extensions/DbSetExtensions.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/Extensions/DbSetExtensions.cs
@@ -6,9 +6,15 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.Extensions;
 
+/// <summary>
+/// DbSet extensions for bulk insert operations.
+/// </summary>
 public static class DbSetExtensions
 {
-    public static async Task<List<T>> ExecuteInsertWithIdentityAsync<T>(
+    /// <summary>
+    /// Executes a bulk insert operation returning the inserted/updated entities, from the DbSet.
+    /// </summary>
+    public static async Task<List<T>> ExecuteInsertReturnEntitiesAsync<T>(
         this DbSet<T> dbSet,
         IEnumerable<T> entities,
         Action<BulkInsertOptions>? configure = null,
@@ -21,40 +27,23 @@ public static async Task<List<T>> ExecuteInsertWithIdentityAsync<T>(
         return await provider.BulkInsertWithIdentityAsync(context, entities, options, onConflict, ctk);
     }
 
-    public static async Task<List<T>> ExecuteInsertWithIdentityAsync<T>(this DbContext dbContext, IEnumerable<T> entities, Action<BulkInsertOptions>? configure = null, OnConflictOptions? onConflict = null, CancellationToken cancellationToken = default) where T : class
+    /// <summary>
+    /// Executes a bulk insert operation returning the inserted/updated entities, from the DbContext.
+    /// </summary>
+    public static async Task<List<T>> ExecuteInsertReturnEntitiesAsync<T>(this DbContext dbContext, IEnumerable<T> entities, Action<BulkInsertOptions>? configure = null, OnConflictOptions? onConflict = null, CancellationToken cancellationToken = default) where T : class
     {
         var dbSet = dbContext.Set<T>();
         if (dbSet == null)
         {
             throw new InvalidOperationException($"DbSet of type {typeof(T).Name} not found in DbContext.");
         }
 
-        return await dbSet.ExecuteInsertWithIdentityAsync(entities, configure, onConflict, cancellationToken);
+        return await dbSet.ExecuteInsertReturnEntitiesAsync(entities, configure, onConflict, cancellationToken);
     }
 
-    // public static async Task<List<object>> ExecuteInsertWithPrimaryKeyAsync<T>(
-    //     this DbSet<T> dbSet,
-    //     IEnumerable<T> entities,
-    //     Action<BulkInsertOptions>? configure = null,
-    //     CancellationToken ctk = default
-    // ) where T : class
-    // {
-    //     var provider = InitProvider(dbSet, configure, out var context, out var options);
-    //
-    //     return await provider.BulkInsertWithPrimaryKeyAsync(context, entities, options, ctk);
-    // }
-    //
-    // public static async Task<List<object>> ExecuteInsertWithPrimaryKeyAsync<T>(this DbContext dbContext, IEnumerable<T> entities, Action<BulkInsertOptions>? configure = null, CancellationToken cancellationToken = default) where T : class
-    // {
-    //     var dbSet = dbContext.Set<T>();
-    //     if (dbSet == null)
-    //     {
-    //         throw new InvalidOperationException($"DbSet of type {typeof(T).Name} not found in DbContext.");
-    //     }
-    //
-    //     return await dbSet.ExecuteInsertWithPrimaryKeyAsync(entities, configure, cancellationToken);
-    // }
-
+    /// <summary>
+    /// Executes a bulk insert operation without returning the inserted/updated entities, from the DbSet.
+    /// </summary>
     public static async Task ExecuteInsertAsync<T>(
         this DbSet<T> dbSet,
         IEnumerable<T> entities,
@@ -68,6 +57,9 @@ public static async Task ExecuteInsertAsync<T>(
         await provider.BulkInsertWithoutReturnAsync(context, entities, options, onConflict, ctk);
     }
 
+    /// <summary>
+    /// Executes a bulk insert operation without returning the inserted/updated entities, from the DbContext.
+    /// </summary>
     public static async Task ExecuteInsertAsync<T>(this DbContext dbContext, IEnumerable<T> entities, Action<BulkInsertOptions>? configure = null, OnConflictOptions? onConflict = null, CancellationToken cancellationToken = default) where T : class
     {
         var dbSet = dbContext.Set<T>();
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/Options/BulkInsertOptions.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/Options/BulkInsertOptions.cs
@@ -1,5 +1,8 @@
 ﻿namespace PhenX.EntityFrameworkCore.BulkInsert.Options;
 
+/// <summary>
+/// Bulk insert general options.
+/// </summary>
 public class BulkInsertOptions
 {
     /// <summary>
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/Options/OnConflictOptions.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/Options/OnConflictOptions.cs
@@ -2,6 +2,9 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert.Options;
 
+/// <summary>
+/// Conflict options for bulk insert.
+/// </summary>
 public abstract class OnConflictOptions
 {
     /// <summary>
@@ -10,6 +13,10 @@ public abstract class OnConflictOptions
     public string? Condition {get; set; }
 }
 
+/// <summary>
+/// Conflict options for bulk insert, for a specific entity type.
+/// </summary>
+/// <typeparam name="T"></typeparam>
 public class OnConflictOptions<T> : OnConflictOptions
 {
     /// <summary>
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/PhenX.EntityFrameworkCore.BulkInsert.csproj b/src/PhenX.EntityFrameworkCore.BulkInsert/PhenX.EntityFrameworkCore.BulkInsert.csproj
@@ -1,7 +1,14 @@
 ﻿<Project Sdk="Microsoft.NET.Sdk">
 
-    <ItemGroup>
-      <PackageReference Include="Microsoft.EntityFrameworkCore.Relational" Version="8.0.15" />
-    </ItemGroup>
+  <ItemGroup>
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Relational" Condition="'$(TargetFramework)' == 'net8.0'" Version="8.*" />
+    <PackageReference Include="Microsoft.EntityFrameworkCore.Relational" Condition="'$(TargetFramework)' == 'net9.0'" Version="9.*" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <InternalsVisibleTo Include="PhenX.EntityFrameworkCore.BulkInsert.PostgreSql"/>
+    <InternalsVisibleTo Include="PhenX.EntityFrameworkCore.BulkInsert.SqlServer"/>
+    <InternalsVisibleTo Include="PhenX.EntityFrameworkCore.BulkInsert.Sqlite"/>
+  </ItemGroup>
 
 </Project>
diff --git a/src/PhenX.EntityFrameworkCore.BulkInsert/PropertyAccessor.cs b/src/PhenX.EntityFrameworkCore.BulkInsert/PropertyAccessor.cs
@@ -3,7 +3,7 @@
 
 namespace PhenX.EntityFrameworkCore.BulkInsert;
 
-public readonly struct PropertyAccessor
+internal readonly struct PropertyAccessor
 {
     private Func<object, object?> ValueGetter { get; }
 
diff --git a/tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark/LibComparator.cs b/tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark/LibComparator.cs
diff --git a/tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark/PhenX.EntityFrameworkCore.BulkInsert.Benchmark.csproj b/tests/PhenX.EntityFrameworkCore.BulkInsert.Benchmark/PhenX.EntityFrameworkCore.BulkInsert.Benchmark.csproj
diff --git a/tests/PhenX.EntityFrameworkCore.BulkInsert.Tests/Tests/Basic/BasicTestsBase.cs b/tests/PhenX.EntityFrameworkCore.BulkInsert.Tests/Tests/Basic/BasicTestsBase.cs

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`namespace PhenX.EntityFrameworkCore.BulkInsert.PostgreSql;`
`10`	`10`
`11`	`11`	`[UsedImplicitly]`
`12`		`-public class PostgreSqlBulkInsertProvider : BulkInsertProviderBase<PostgreSqlDialectBuilder>`
	`12`	`+internal class PostgreSqlBulkInsertProvider : BulkInsertProviderBase<PostgreSqlDialectBuilder>`
`13`	`13`	`{`
`14`	`14`	`//language=sql`
`15`	`15`	`protected override string CreateTableCopySql => "CREATE TEMPORARY TABLE {0} AS TABLE {1} WITH NO DATA;";`
Original file line number	Diff line number	Diff line change
`@@ -3,8 +3,14 @@`
`3`	`3`
`4`	`4`	`namespace PhenX.EntityFrameworkCore.BulkInsert.PostgreSql;`
`5`	`5`
	`6`	`+/// <summary>`
	`7`	`+/// DbContext options extension for PostgreSQL.`
	`8`	`+/// </summary>`
`6`	`9`	`public static class PostgreSqlDbContextOptionsExtensions`
`7`	`10`	`{`
	`11`	`+ /// <summary>`
	`12`	`+ /// Configures the DbContext to use the PostgreSQL bulk insert provider.`
	`13`	`+ /// </summary>`
`8`	`14`	`public static DbContextOptionsBuilder UseExecuteInsertPostgres(this DbContextOptionsBuilder optionsBuilder)`
`9`	`15`	`{`
`10`	`16`	`var extension = optionsBuilder.Options.FindExtension<ExecuteInsertOptionsExtension<PostgreSqlBulkInsertProvider>>() ?? new ExecuteInsertOptionsExtension<PostgreSqlBulkInsertProvider>();`
Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,7 @@`
`2`	`2`
`3`	`3`	`namespace PhenX.EntityFrameworkCore.BulkInsert.PostgreSql;`
`4`	`4`
`5`		`-public class PostgreSqlDialectBuilder : SqlDialectBuilder`
	`5`	`+internal class PostgreSqlDialectBuilder : SqlDialectBuilder`
`6`	`6`	`{`
`7`	`7`	`protected override string OpenDelimiter => "\"";`
`8`	`8`	`protected override string CloseDelimiter => "\"";`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`namespace PhenX.EntityFrameworkCore.BulkInsert.SqlServer;`
`9`	`9`
`10`	`10`	`[UsedImplicitly]`
`11`		`-public class SqlServerBulkInsertProvider : BulkInsertProviderBase<SqlServerDialectBuilder>`
	`11`	`+internal class SqlServerBulkInsertProvider : BulkInsertProviderBase<SqlServerDialectBuilder>`
`12`	`12`	`{`
`13`	`13`	`//language=sql`
`14`	`14`	`protected override string CreateTableCopySql => "SELECT {2} INTO {0} FROM {1} WHERE 1 = 0;";`