Remove ICollectionTypeFactory

This used to be a D.I. resolved service in pre-release versions of EF Core. The CollectionTypeFactory is now directly instantiated, so removing this interface. Issue #6616
dotnet · Oct 3, 2016 · 4af5c80 · 4af5c80
1 parent 9cc618f
commit 4af5c80
Show file tree

Hide file tree

Showing 5 changed files with 1 addition and 25 deletions.
diff --git a/...icrosoft.EntityFrameworkCore/Infrastructure/EntityFrameworkServiceCollectionExtensions.cs b/...icrosoft.EntityFrameworkCore/Infrastructure/EntityFrameworkServiceCollectionExtensions.cs
@@ -85,7 +85,6 @@ public static IServiceCollection AddEntityFramework(
                 .AddSingleton<IDbSetInitializer, DbSetInitializer>()
                 .AddSingleton<IDbSetSource, DbSetSource>()
                 .AddSingleton<IEntityFinderSource, EntityFinderSource>()
-                .AddSingleton<ICollectionTypeFactory, CollectionTypeFactory>()
                 .AddSingleton<IEntityMaterializerSource, EntityMaterializerSource>()
                 .AddSingleton<ICoreConventionSetBuilder, CoreConventionSetBuilder>()
                 .AddSingleton<IModelCustomizer, ModelCustomizer>()

diff --git a/src/Microsoft.EntityFrameworkCore/Metadata/Internal/CollectionTypeFactory.cs b/src/Microsoft.EntityFrameworkCore/Metadata/Internal/CollectionTypeFactory.cs
@@ -13,7 +13,7 @@ namespace Microsoft.EntityFrameworkCore.Metadata.Internal
     ///     This API supports the Entity Framework Core infrastructure and is not intended to be used
     ///     directly from your code. This API may change or be removed in future releases.
     /// </summary>
-    public class CollectionTypeFactory : ICollectionTypeFactory
+    public class CollectionTypeFactory
     {
         /// <summary>
         ///     This API supports the Entity Framework Core infrastructure and is not intended to be used

diff --git a/src/Microsoft.EntityFrameworkCore/Metadata/Internal/ICollectionTypeFactory.cs b/src/Microsoft.EntityFrameworkCore/Metadata/Internal/ICollectionTypeFactory.cs
diff --git a/src/Microsoft.EntityFrameworkCore/Microsoft.EntityFrameworkCore.csproj b/src/Microsoft.EntityFrameworkCore/Microsoft.EntityFrameworkCore.csproj
@@ -382,7 +382,6 @@
     <Compile Include="Metadata\Internal\IClrCollectionAccessor.cs" />
     <Compile Include="Metadata\Internal\IClrPropertyGetter.cs" />
     <Compile Include="Metadata\Internal\IClrPropertySetter.cs" />
-    <Compile Include="Metadata\Internal\ICollectionTypeFactory.cs" />
     <Compile Include="Metadata\Internal\IEntityMaterializer.cs" />
     <Compile Include="Metadata\Internal\IEntityMaterializerSource.cs" />
     <Compile Include="Metadata\Internal\Index.cs" />

diff --git a/test/Microsoft.EntityFrameworkCore.Tests/EntityFrameworkServiceCollectionExtensionsTest.cs b/test/Microsoft.EntityFrameworkCore.Tests/EntityFrameworkServiceCollectionExtensionsTest.cs
@@ -46,7 +46,6 @@ public virtual void Services_wire_up_correctly()
             VerifySingleton<IDbSetFinder>();
             VerifySingleton<IDbSetInitializer>();
             VerifySingleton<IDbSetSource>();
-            VerifySingleton<ICollectionTypeFactory>();
             VerifySingleton<IEntityMaterializerSource>();
             VerifySingleton<ILoggerFactory>();
             VerifySingleton<ICoreConventionSetBuilder>();