Add EntityFrameworkCore IMigrator mock (closes #264)

Author: ivaylokenov

src/MyTested.AspNetCore.Mvc.EntityFrameworkCore/Internal/EntityFrameworkCore/MigratorMock.cs

@@ -0,0 +1,26 @@
+namespace MyTested.AspNetCore.Mvc.Internal.EntityFrameworkCore
+{
+    using System.Threading;
+    using System.Threading.Tasks;
+    using Microsoft.EntityFrameworkCore.Migrations;
+    using Microsoft.AspNetCore.Mvc.Internal;
+
+    public class MigratorMock : IMigrator
+    {
+        public string GenerateScript(
+            string fromMigration = null,
+            string toMigration = null,
+            bool idempotent = false)
+            => string.Empty;
+
+        public void Migrate(string targetMigration = null)
+        {
+            // intentionally left empty
+        }
+
+        public Task MigrateAsync(
+            string targetMigration = null,
+            CancellationToken cancellationToken = default(CancellationToken))
+            => TaskCache.CompletedTask;
+    }
+}

src/MyTested.AspNetCore.Mvc.EntityFrameworkCore/Internal/EntityFrameworkCore/ScopedInMemoryOptionsExtension.cs

@@ -1,17 +1,17 @@
 namespace MyTested.AspNetCore.Mvc.Internal.EntityFrameworkCore
 {
     using Microsoft.EntityFrameworkCore.Infrastructure.Internal;
+    using Microsoft.EntityFrameworkCore.Migrations;
     using Microsoft.EntityFrameworkCore.Storage.Internal;
     using Microsoft.EntityFrameworkCore.ValueGeneration.Internal;
     using Microsoft.Extensions.DependencyInjection;
 
     public class ScopedInMemoryOptionsExtension : InMemoryOptionsExtension
     {
         public override void ApplyServices(IServiceCollection services)
-        {
-            services
+            => services
+                .AddScoped<IMigrator, MigratorMock>()
                 .ReplaceLifetime<IInMemoryStoreSource>(ServiceLifetime.Scoped)
                 .ReplaceLifetime<InMemoryValueGeneratorCache>(ServiceLifetime.Scoped);
-        }
     }
 }

test/MyTested.AspNetCore.Mvc.EntityFrameworkCore.Test/ServicesTests.cs

@@ -1,7 +1,7 @@
 namespace MyTested.AspNetCore.Mvc.Test
 {
     using System.Linq;
-    using Internal;
+    using Internal.EntityFrameworkCore;
     using Microsoft.EntityFrameworkCore;
     using Microsoft.EntityFrameworkCore.Internal;
     using Microsoft.EntityFrameworkCore.Infrastructure.Internal;
@@ -16,8 +16,7 @@ public void ReplaceDbContextShouldReplaceNonInMemoryDatabaseWithInMemoryScopedOn
         {
             var services = new ServiceCollection();
 
-            services.AddDbContext<CustomDbContext>(options =>
-                options.UseSqlServer("Server=(localdb)\\MSSQLLocalDB;Database=TestDb;Trusted_Connection=True;MultipleActiveResultSets=true;Connect Timeout=30;"));
+            this.AddDbContextWithSqlServer(services);
 
             services.ReplaceDbContext();
 
@@ -48,6 +47,26 @@ public void ReplaceDbContextShouldNotAddDbContextIfMissing()
             Assert.Null(serviceProvider.GetService<DbContext>());
         }
 
+        [Fact]
+        public void CallingMigrateShouldNotThrowExceptionWithInMemoryDatabase()
+        {
+            var services = new ServiceCollection();
+
+            this.AddDbContextWithSqlServer(services);
+
+            services.ReplaceDbContext();
+
+            var serviceProvider = services.BuildServiceProvider();
+
+            services.BuildServiceProvider().GetRequiredService<CustomDbContext>().Database.Migrate();
+        }
+
+        private void AddDbContextWithSqlServer(IServiceCollection services)
+        {
+            services.AddDbContext<CustomDbContext>(options =>
+                options.UseSqlServer("Server=(localdb)\\MSSQLLocalDB;Database=TestDb;Trusted_Connection=True;MultipleActiveResultSets=true;Connect Timeout=30;"));
+        }
+
         private void AssertCorrectDbContextAndOptions(IServiceCollection services)
         {
             var serviceProvider = services.BuildServiceProvider();