Feat: EntityFramework Snapshots (#108)

* bugfix: don't publish transaction if it wasn't committed to the database

* feat: EntityFramework snapshots

* test: EntityFramework snapshots

* fix: remove test code

* refactor: provide a dbContext which handles the boilerplate

* fix(test): update tests to match new structure

* refactor: use ValueConverter for the ValueObjects

* refactor: make choosing table names mandatory

* refactor: use ValueObjects instead of Raw Values

to keep usage consistent

* refactor: don't require a named DbSet property

prevents using one DbContext for multiple snapshots (and therefore prevents related snapshots)

* refactor: use real transactions for entity framework snapshots in test mode

* refactor: update snapshot reference if one already exists

* chore: coverage for TimeStamp converter

* chore: exclude ToString overrides from coverage

* chore: exclude functioning CommitTransaction from tests

Tests should be running in TestMode, and should not actually commit data.

* chore: remove unused method

* fix: swap order of methods

* fix: working delete method

* Update global.json

* chore: include debug logs

* refactor: SnapshotReference _has one_ Snapshot instead of _owns_ snapshot

* nit: explicit OnDelete behavior

* fix: IEntityFrameworkSnapshot belongs in its own file, and in the Snapshots namespace

* refactor: better snapshot cleanup

1. delete snapshot when all references deleted
2. delete snapshot when all references are changed to point to a different snapshot
3. provide an option to opt-out of cleanup behavior

* refactor: force a constructor instead of using IDbContextFactory

this allows for a single DbContext to have multiple connection configurations (e.g., read-only connection string)

* fix: somehow this wasn't on this branch originally

* refactor: rename bits and bobs

IEntityDbContext, EntityDbContextBase

does not explicitly depend on involving snapshots

* refactor: IEntityDbContextFactory

allow the end user to create DbContexts using just session option names

* Empty Commit

Author: the-avid-engineer

EntityDb.sln

@@ -47,7 +47,9 @@ Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "EntityDb.SqlDb", "src\Entit
 EndProject
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "EntityDb.Json", "src\EntityDb.Json\EntityDb.Json.csproj", "{4936FFE0-98E5-43A2-89C9-0415A13CAA9B}"
 EndProject
-Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "EntityDb.Provisioner", "src\EntityDb.Provisioner\EntityDb.Provisioner.csproj", "{26FCDB9D-0DE3-4BB9-858D-3E2C3EF763E3}"
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "EntityDb.Provisioner", "src\EntityDb.Provisioner\EntityDb.Provisioner.csproj", "{26FCDB9D-0DE3-4BB9-858D-3E2C3EF763E3}"
+EndProject
+Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "EntityDb.EntityFramework", "src\EntityDb.EntityFramework\EntityDb.EntityFramework.csproj", "{199606BF-6283-4684-A224-4DA7E80D8F45}"
 EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
@@ -115,6 +117,10 @@ Global
 		{26FCDB9D-0DE3-4BB9-858D-3E2C3EF763E3}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{26FCDB9D-0DE3-4BB9-858D-3E2C3EF763E3}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{26FCDB9D-0DE3-4BB9-858D-3E2C3EF763E3}.Release|Any CPU.Build.0 = Release|Any CPU
+		{199606BF-6283-4684-A224-4DA7E80D8F45}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{199606BF-6283-4684-A224-4DA7E80D8F45}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{199606BF-6283-4684-A224-4DA7E80D8F45}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{199606BF-6283-4684-A224-4DA7E80D8F45}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE
@@ -135,6 +141,7 @@ Global
 		{F2491666-31D1-47B5-A493-F25E167D1FDF} = {ABACFBCC-B59F-4616-B6CC-99C37AEC8960}
 		{4936FFE0-98E5-43A2-89C9-0415A13CAA9B} = {ABACFBCC-B59F-4616-B6CC-99C37AEC8960}
 		{26FCDB9D-0DE3-4BB9-858D-3E2C3EF763E3} = {ABACFBCC-B59F-4616-B6CC-99C37AEC8960}
+		{199606BF-6283-4684-A224-4DA7E80D8F45} = {ABACFBCC-B59F-4616-B6CC-99C37AEC8960}
 	EndGlobalSection
 	GlobalSection(ExtensibilityGlobals) = postSolution
 		SolutionGuid = {E9D288EE-9351-4018-ABE8-B0968AEB0465}

global.json

@@ -1,6 +1,6 @@
 {
   "sdk": {
-    "version": "7.0.100",
+    "version": "7.0.201",
     "allowPrerelease": false,
     "rollForward": "disable"
   }

src/EntityDb.Common/Properties/AssemblyInfo.cs

@@ -3,6 +3,7 @@
 // src
 [assembly: InternalsVisibleTo("EntityDb.SqlDb")]
 [assembly: InternalsVisibleTo("EntityDb.Npgsql")]
+[assembly: InternalsVisibleTo("EntityDb.EntityFramework")]
 [assembly: InternalsVisibleTo("EntityDb.InMemory")]
 [assembly: InternalsVisibleTo("EntityDb.MongoDb")]
 [assembly: InternalsVisibleTo("EntityDb.Provisioner")]

src/EntityDb.EntityFramework/Converters/IdConverter.cs

@@ -0,0 +1,15 @@
+using EntityDb.Abstractions.ValueObjects;
+using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
+using System.Linq.Expressions;
+
+namespace EntityDb.EntityFramework.Converters;
+
+internal class IdConverter : ValueConverter<Id, Guid>
+{
+    private static readonly Expression<Func<Id, Guid>> IdToGuid = (id) => id.Value;
+    private static readonly Expression<Func<Guid, Id>> GuidToId = (guid) => new Id(guid);
+
+    public IdConverter() : base(IdToGuid, GuidToId, null)
+    {
+    }
+}

src/EntityDb.EntityFramework/Converters/TimeStampConverter.cs

@@ -0,0 +1,15 @@
+using EntityDb.Abstractions.ValueObjects;
+using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
+using System.Linq.Expressions;
+
+namespace EntityDb.EntityFramework.Converters;
+
+internal class TimeStampConverter : ValueConverter<TimeStamp, DateTime>
+{
+    private static readonly Expression<Func<TimeStamp, DateTime>> TimeStampToDateTime = (timeStamp) => timeStamp.Value;
+    private static readonly Expression<Func<DateTime, TimeStamp>> DateTimeToTimeStamp = (dateTime) => new TimeStamp(dateTime);
+
+    public TimeStampConverter() : base(TimeStampToDateTime, DateTimeToTimeStamp, null)
+    {
+    }
+}

src/EntityDb.EntityFramework/Converters/VersionNumberConverter.cs

@@ -0,0 +1,15 @@
+using EntityDb.Abstractions.ValueObjects;
+using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
+using System.Linq.Expressions;
+
+namespace EntityDb.EntityFramework.Converters;
+
+internal class VersionNumberConverter : ValueConverter<VersionNumber, ulong>
+{
+    private static readonly Expression<Func<VersionNumber, ulong>> VersionNumberToUlong = (versionNumber) => versionNumber.Value;
+    private static readonly Expression<Func<ulong, VersionNumber>> UlongToVersionNumber = (@ulong) => new VersionNumber(@ulong);
+
+    public VersionNumberConverter() : base(VersionNumberToUlong, UlongToVersionNumber, null)
+    {
+    }
+}

src/EntityDb.EntityFramework/DbContexts/EntityDbContextBase.cs

@@ -0,0 +1,27 @@
+using EntityDb.Abstractions.ValueObjects;
+using EntityDb.EntityFramework.Converters;
+using Microsoft.EntityFrameworkCore;
+
+namespace EntityDb.EntityFramework.DbContexts;
+
+/// <summary>
+///     A DbContext that adds basic converters for types defined in <see cref="Abstractions.ValueObjects"/>
+/// </summary>
+public abstract class EntityDbContextBase : DbContext
+{
+    /// <inheritdoc />
+    protected override void ConfigureConventions(ModelConfigurationBuilder configurationBuilder)
+    {
+        configurationBuilder
+            .Properties<Id>()
+            .HaveConversion<IdConverter>();
+
+        configurationBuilder
+            .Properties<VersionNumber>()
+            .HaveConversion<VersionNumberConverter>();
+
+        configurationBuilder
+            .Properties<TimeStamp>()
+            .HaveConversion<TimeStampConverter>();
+    }
+}

src/EntityDb.EntityFramework/DbContexts/EntityDbContextFactory.cs

@@ -0,0 +1,26 @@
+using EntityDb.EntityFramework.Sessions;
+using Microsoft.EntityFrameworkCore;
+using Microsoft.Extensions.Options;
+
+namespace EntityDb.EntityFramework.DbContexts;
+
+internal class EntityDbContextFactory<TDbContext> : IEntityDbContextFactory<TDbContext>
+    where TDbContext : DbContext, IEntityDbContext<TDbContext>
+{
+    private readonly IOptionsFactory<EntityFrameworkSnapshotSessionOptions> _optionsFactory;
+
+    public EntityDbContextFactory(IOptionsFactory<EntityFrameworkSnapshotSessionOptions> optionsFactory)
+    {
+        _optionsFactory = optionsFactory;
+    }
+
+    public TDbContext Create(string snapshotSessionOptionsName)
+    {
+        return TDbContext.Construct(_optionsFactory.Create(snapshotSessionOptionsName));
+    }
+
+    TDbContext IEntityDbContextFactory<TDbContext>.Create(EntityFrameworkSnapshotSessionOptions snapshotSessionOptions)
+    {
+        return TDbContext.Construct(snapshotSessionOptions);
+    }
+}

src/EntityDb.EntityFramework/DbContexts/IEntityDbContext.cs

@@ -0,0 +1,19 @@
+using EntityDb.EntityFramework.Sessions;
+using Microsoft.EntityFrameworkCore;
+
+namespace EntityDb.EntityFramework.DbContexts;
+
+/// <summary>
+///     A type of a <see cref="DbContext"/> that can be used for EntityDb purposes.
+/// </summary>
+/// <typeparam name="TDbContext">The type of the <see cref="DbContext"/></typeparam>
+public interface IEntityDbContext<TDbContext>
+    where TDbContext : DbContext, IEntityDbContext<TDbContext>
+{
+    /// <summary>
+    ///     Returns a new <typeparamref name="TDbContext"/> that will be configured using <paramref name="entityFrameworkSnapshotSessionOptions"/>.
+    /// </summary>
+    /// <param name="entityFrameworkSnapshotSessionOptions">The options for the database</param>
+    /// <returns>A new <typeparamref name="TDbContext"/> that will be configured using <paramref name="entityFrameworkSnapshotSessionOptions"/>.</returns>
+    static abstract TDbContext Construct(EntityFrameworkSnapshotSessionOptions entityFrameworkSnapshotSessionOptions);
+}

src/EntityDb.EntityFramework/DbContexts/IEntityDbContextFactory.cs

@@ -0,0 +1,20 @@
+using EntityDb.EntityFramework.Sessions;
+using Microsoft.EntityFrameworkCore;
+
+namespace EntityDb.EntityFramework.DbContexts;
+
+/// <summary>
+///     Represents a type used to create instances of <typeparamref name="TDbContext"/>.
+/// </summary>
+/// <typeparam name="TDbContext">The type of the <see cref="DbContext"/>.</typeparam>
+public interface IEntityDbContextFactory<TDbContext>
+{
+    internal TDbContext Create(EntityFrameworkSnapshotSessionOptions snapshotSessionOptions);
+
+    /// <summary>
+    ///     Create a new instance of <typeparamref name="TDbContext"/>.
+    /// </summary>
+    /// <param name="snapshotSessionOptionsName">The agent's use case for the <see cref="DbContext"/>.</param>
+    /// <returns>A new instance of <typeparamref name="TDbContext"/>.</returns>
+    TDbContext Create(string snapshotSessionOptionsName);
+}