Merge pull request #148 from linq2db/version3

Release 3.11.0

Author: sdanyliv

Build/linq2db.Default.props

@@ -1,6 +1,6 @@
 <Project>
 	<PropertyGroup>
-		<Version>3.10.1</Version>
+		<Version>3.11.0</Version>
 
 		<Authors>Svyatoslav Danyliv, Igor Tkachev, Dmitry Lukashenko, Ilya Chudin</Authors>
 		<Product>Linq to DB</Product>

NuGet/linq2db.EntityFrameworkCore.nuspec

@@ -16,7 +16,7 @@
 		<dependencies>
 			<group targetFramework=".NETStandard2.0">
 				<dependency id="Microsoft.EntityFrameworkCore.Relational" version="3.1.10" />
-				<dependency id="linq2db" version="3.3.0" />
+				<dependency id="linq2db" version="3.4.0" />
 			</group>
 		</dependencies>
 	</metadata>

Source/LinqToDB.EntityFrameworkCore/EFCoreMetadataReader.cs

@@ -150,12 +150,25 @@ public T[] GetAttributes<T>(Type type, MemberInfo memberInfo, bool inherit = tru
 						}
 
 						var isIdentity = prop.GetAnnotations()
-							.Any(a => a.Name.EndsWith(":ValueGenerationStrategy") && a.Value?.ToString() == "IdentityColumn");
-
-						if (!isIdentity && isPrimaryKey)
-						{
-							isIdentity = prop.ValueGenerated == ValueGenerated.OnAdd;
-						}
+							.Any(a =>
+							{
+								if (a.Name.EndsWith(":ValueGenerationStrategy"))
+									return a.Value?.ToString().Contains("Identity") == true;
+
+								if (a.Name.EndsWith(":Autoincrement"))
+									return a.Value is bool b && b;
+
+								// for postgres
+								if (a.Name == "Relational:DefaultValueSql")
+								{
+									if (a.Value is string str)
+									{
+										return str.ToLower().Contains("nextval");
+									}
+								}
+
+								return false;
+							});
 
 						return new T[]{(T)(Attribute) new ColumnAttribute
 						{

Source/LinqToDB.EntityFrameworkCore/LinqToDBForEFTools.ContextExtensions.cs

@@ -233,7 +233,7 @@ public static async Task<BulkCopyRowsCopied> BulkCopyAsync<T>(
 		[LinqTunnel]
 		[Pure]
 		public static IValueInsertable<T> Into<T>(this DbContext context, ITable<T> target)
-		where T: notnull
+			where T: notnull
 		{
 			if (context == null) throw new ArgumentNullException(nameof(context));
 			if (target == null)  throw new ArgumentNullException(nameof(target));

Source/LinqToDB.EntityFrameworkCore/LinqToDBForEFToolsImplDefault.cs

@@ -17,6 +17,7 @@
 
 using JetBrains.Annotations;
 using Microsoft.EntityFrameworkCore.Diagnostics;
+using Microsoft.Extensions.Caching.Memory;
 
 namespace LinqToDB.EntityFrameworkCore
 {
@@ -84,10 +85,10 @@ public override int GetHashCode()
 			#endregion
 		}
 
-		readonly ConcurrentDictionary<ProviderKey, IDataProvider> _knownProviders = new ConcurrentDictionary<ProviderKey, IDataProvider>();
+		readonly ConcurrentDictionary<ProviderKey, IDataProvider> _knownProviders = new();
 
-		private readonly MemoryCache _schemaCache = new MemoryCache(
-			new MemoryCacheOptions
+		private readonly MemoryCache _schemaCache = new(
+			new Microsoft.Extensions.Caching.Memory.MemoryCacheOptions
 			{
 				ExpirationScanFrequency = TimeSpan.FromHours(1.0)
 			});

Source/LinqToDB.EntityFrameworkCore/linq2db.EntityFrameworkCore.csproj

@@ -23,7 +23,7 @@
 	</PropertyGroup>
 
 	<ItemGroup>
-		<PackageReference Include="linq2db" Version="3.3.0" />
+		<PackageReference Include="linq2db" Version="3.4.0" />
 		<PackageReference Include="Microsoft.EntityFrameworkCore.Relational" Version="3.1.11" />
 		<PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0">
 			<PrivateAssets>all</PrivateAssets>

Tests/LinqToDB.EntityFrameworkCore.BaseTests/ForMappingTestsBase.cs

@@ -0,0 +1,91 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using FluentAssertions;
+using LinqToDB.Data;
+using LinqToDB.EntityFrameworkCore.BaseTests.Models.ForMapping;
+using NUnit.Framework;
+
+namespace LinqToDB.EntityFrameworkCore.BaseTests
+{
+	public abstract class ForMappingTestsBase : TestsBase
+	{
+		public abstract ForMappingContextBase CreateContext();
+
+		[Test]
+		public virtual void TestIdentityMapping()
+		{
+			using var context = CreateContext();
+			using var connection = context.CreateLinqToDbConnection();
+
+			var ed = connection.MappingSchema.GetEntityDescriptor(typeof(WithIdentity));
+			var pk = ed.Columns.Where(c => c.IsPrimaryKey).Single();
+
+			pk.IsIdentity.Should().BeTrue();
+		}
+
+		[Test]
+		public virtual void TestNoIdentityMapping()
+		{
+			using var context = CreateContext();
+			using var connection = context.CreateLinqToDbConnection();
+
+			var ed = connection.MappingSchema.GetEntityDescriptor(typeof(NoIdentity));
+			var pk = ed.Columns.Where(c => c.IsPrimaryKey).Single();
+
+			pk.IsIdentity.Should().BeFalse();
+		}
+
+		[Test]
+		public virtual void TestTableCreation()
+		{
+			using var context = CreateContext();
+			using var connection = context.CreateLinqToDbConnection();
+
+			using var t1 = connection.CreateTempTable<WithIdentity>();
+			using var t2 = connection.CreateTempTable<NoIdentity>();
+		}
+
+
+		[Test]
+		public virtual void TestBulkCopyNoIdentity()
+		{
+			using var context = CreateContext();
+			using var connection = context.CreateLinqToDbConnection();
+
+			using var t = connection.CreateTempTable<NoIdentity>();
+
+			var items = new List<NoIdentity>()
+			{
+				new() {Id = Guid.NewGuid(), Name = "John Doe"},
+				new() {Id = Guid.NewGuid(), Name = "Jane Doe"}
+			};
+
+			t.BulkCopy(items);
+
+
+			items.Should().BeEquivalentTo(t);
+		}
+
+		[Test]
+		public virtual void TestBulkCopyWithIdentity()
+		{
+			using var context = CreateContext();
+			using var connection = context.CreateLinqToDbConnection();
+
+			using var t = connection.CreateTempTable<WithIdentity>();
+
+			var items = new List<WithIdentity>()
+			{
+				new() {Id = 1, Name = "John Doe"},
+				new() {Id = 2, Name = "Jane Doe"}
+			};
+
+			t.BulkCopy(items);
+
+
+			t.Should().HaveCount(items.Count);
+		}
+
+	}
+}

Tests/LinqToDB.EntityFrameworkCore.BaseTests/LinqToDB.EntityFrameworkCore.BaseTests.csproj

@@ -9,6 +9,7 @@
 	</ItemGroup>
 
 	<ItemGroup>
+		<PackageReference Include="FluentAssertions" Version="5.10.3" />
 		<PackageReference Include="linq2db.Tools" Version="3.2.3" />
 		<PackageReference Include="Microsoft.Bcl.AsyncInterfaces" Version="5.0.0" />
 		<PackageReference Include="Microsoft.EntityFrameworkCore.InMemory" Version="3.1.11" />

Tests/LinqToDB.EntityFrameworkCore.BaseTests/Models/ForMapping/ForMappingContextBase.cs

@@ -0,0 +1,15 @@
+using Microsoft.EntityFrameworkCore;
+
+namespace LinqToDB.EntityFrameworkCore.BaseTests.Models.ForMapping
+{
+	public abstract class ForMappingContextBase : DbContext
+	{
+		protected ForMappingContextBase(DbContextOptions options) : base(options)
+		{
+		}
+
+		public DbSet<WithIdentity> WithIdentity { get; set; } = null!;
+		public DbSet<NoIdentity> NoIdentity { get; set; } = null!;
+
+	}
+}

Tests/LinqToDB.EntityFrameworkCore.BaseTests/Models/ForMapping/NoIdentity.cs

@@ -0,0 +1,10 @@
+using System;
+
+namespace LinqToDB.EntityFrameworkCore.BaseTests.Models.ForMapping
+{
+	public class NoIdentity
+	{
+		public Guid Id { get; set; }
+		public string Name { get; set; } = null!;
+	}
+}