RevEng: Add DatabaseModelFactoryOptions

src/EFCore.Design/Design/Internal/DatabaseOperations.cs

@@ -89,8 +89,7 @@ public class DatabaseOperations
 
             var scaffoldedModel = scaffolder.ScaffoldModel(
                 connectionString,
-                tables,
-                schemas,
+                new DatabaseModelFactoryOptions(tables, schemas),
                 _rootNamespace,
                 modelNamespace,
                 contextNamespace,

src/EFCore.Design/Scaffolding/IReverseEngineerScaffolder.cs

@@ -1,7 +1,6 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System.Collections.Generic;
 using JetBrains.Annotations;
 
 namespace Microsoft.EntityFrameworkCore.Scaffolding
@@ -15,8 +14,7 @@ public interface IReverseEngineerScaffolder
         ///     Scaffolds a model from a database schema.
         /// </summary>
         /// <param name="connectionString"> A connection string to the database. </param>
-        /// <param name="tables"> A list of tables to include. Empty to include all tables. </param>
-        /// <param name="schemas"> A list of schemas to include. Empty to include all schemas. </param>
+        /// <param name="options"> The options specifying which metadata to read from the database. </param>
         /// <param name="rootNamespace"> The namespace of the project. </param>
         /// <param name="modelNamespace"> The namespace for model classes. </param>
         /// <param name="contextNamespace"> The namespace for context class. </param>
@@ -28,8 +26,7 @@ public interface IReverseEngineerScaffolder
         /// <returns> The scaffolded model. </returns>
         ScaffoldedModel ScaffoldModel(
             [NotNull] string connectionString,
-            [NotNull] IEnumerable<string> tables,
-            [NotNull] IEnumerable<string> schemas,
+            [NotNull] DatabaseModelFactoryOptions options,
             [NotNull] string rootNamespace,
             [NotNull] string modelNamespace,
             [NotNull] string contextNamespace,

src/EFCore.Design/Scaffolding/Internal/ReverseEngineerScaffolder.cs

@@ -68,8 +68,7 @@ public class ReverseEngineerScaffolder : IReverseEngineerScaffolder
         /// </summary>
         public virtual ScaffoldedModel ScaffoldModel(
             string connectionString,
-            IEnumerable<string> tables,
-            IEnumerable<string> schemas,
+            DatabaseModelFactoryOptions options,
             string rootNamespace,
             string modelNamespace,
             string contextNamespace,
@@ -80,8 +79,7 @@ public class ReverseEngineerScaffolder : IReverseEngineerScaffolder
             ModelCodeGenerationOptions codeOptions)
         {
             Check.NotEmpty(connectionString, nameof(connectionString));
-            Check.NotNull(tables, nameof(tables));
-            Check.NotNull(schemas, nameof(schemas));
+            Check.NotNull(options, nameof(options));
             Check.NotEmpty(modelNamespace, nameof(modelNamespace));
             Check.NotEmpty(contextNamespace, nameof(contextNamespace));
             Check.NotNull(modelOptions, nameof(modelOptions));
@@ -101,7 +99,7 @@ public class ReverseEngineerScaffolder : IReverseEngineerScaffolder
                 codeOptions.SuppressConnectionStringWarning = true;
             }
 
-            var databaseModel = _databaseModelFactory.Create(resolvedConnectionString, tables, schemas);
+            var databaseModel = _databaseModelFactory.Create(resolvedConnectionString, options);
             var model = _factory.Create(databaseModel, modelOptions.UseDatabaseNames);
 
             if (model == null)

src/EFCore.Relational/Scaffolding/DatabaseModelFactoryOptions.cs

@@ -0,0 +1,39 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+using System.Collections.Generic;
+using JetBrains.Annotations;
+using Microsoft.EntityFrameworkCore.Utilities;
+
+namespace Microsoft.EntityFrameworkCore.Scaffolding
+{
+    /// <summary>
+    /// Specifies which metadata to read from the database.
+    /// </summary>
+    public class DatabaseModelFactoryOptions
+    {
+        /// <summary>
+        ///     Initializes a new instance of the <see cref="DatabaseModelFactoryOptions" /> class.
+        /// </summary>
+        /// <param name="tables"> A list of tables to include. Empty to include all tables. </param>
+        /// <param name="schemas"> A list of schemas to include. Empty to include all schemas. </param>
+        public DatabaseModelFactoryOptions([NotNull] IEnumerable<string> tables, [NotNull] IEnumerable<string> schemas)
+        {
+            Check.NotNull(tables, nameof(tables));
+            Check.NotNull(schemas, nameof(schemas));
+
+            Tables = tables;
+            Schemas = schemas;
+        }
+
+        /// <summary>
+        /// Gets the list of tables to include. If empty, include all tables.
+        /// </summary>
+        public IEnumerable<string> Tables { get; }
+
+        /// <summary>
+        /// Gets the list of schemas to include. If empty, include all schemas.
+        /// </summary>
+        public IEnumerable<string> Schemas { get; }
+    }
+}

src/EFCore.Relational/Scaffolding/IDatabaseModelFactory.cs

@@ -1,7 +1,6 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System.Collections.Generic;
 using System.Data.Common;
 using JetBrains.Annotations;
 using Microsoft.EntityFrameworkCore.Scaffolding.Metadata;
@@ -19,19 +18,17 @@ public interface IDatabaseModelFactory
         ///     for the database.
         /// </summary>
         /// <param name="connectionString"> The connection string for the database to reverse engineer. </param>
-        /// <param name="tables"> The tables to include in the model, or an empty enumerable to include all. </param>
-        /// <param name="schemas"> The schema to include in the model, or an empty enumerable to include all. </param>
+        /// <param name="options"> The options specifying which metadata to read. </param>
         /// <returns> The database model. </returns>
-        DatabaseModel Create([NotNull] string connectionString, [NotNull] IEnumerable<string> tables, [NotNull] IEnumerable<string> schemas);
+        DatabaseModel Create([NotNull] string connectionString, [NotNull] DatabaseModelFactoryOptions options);
 
         /// <summary>
         ///     Connects to the database using the given connection and creates a <see cref="DatabaseModel" />
         ///     for the database.
         /// </summary>
         /// <param name="connection"> The connection to the database to reverse engineer. </param>
-        /// <param name="tables"> The tables to include in the model, or an empty enumerable to include all. </param>
-        /// <param name="schemas"> The schema to include in the model, or an empty enumerable to include all. </param>
+        /// <param name="options"> The options specifying which metadata to read. </param>
         /// <returns> The database model. </returns>
-        DatabaseModel Create([NotNull] DbConnection connection, [NotNull] IEnumerable<string> tables, [NotNull] IEnumerable<string> schemas);
+        DatabaseModel Create([NotNull] DbConnection connection, [NotNull] DatabaseModelFactoryOptions options);
     }
 }

test/EFCore.Design.Tests/Scaffolding/Internal/ReverseEngineeringConfigurationTests.cs

@@ -39,8 +39,9 @@ private void ValidateContextNameInReverseEngineerGenerator(string contextName)
                 Assert.Throws<ArgumentException>(
                         () => reverseEngineer.ScaffoldModel(
                             "connectionstring",
-                            /* tables: */ Enumerable.Empty<string>(),
-                            /* schemas: */ Enumerable.Empty<string>(),
+                            new DatabaseModelFactoryOptions(
+                                /* tables: */ Enumerable.Empty<string>(),
+                                /* schemas: */ Enumerable.Empty<string>()),
                             "FakeNamespace",
                             "FakeNamespace",
                             "FakeNamespace",

test/EFCore.Relational.Specification.Tests/TestUtilities/RelationalDatabaseCleaner.cs

@@ -50,7 +50,8 @@ public virtual void Clean(DatabaseFacade facade)
             {
                 var databaseModelFactory = CreateDatabaseModelFactory(loggerFactory);
                 var databaseModel = databaseModelFactory.Create(
-                    connection.DbConnection, Enumerable.Empty<string>(), Enumerable.Empty<string>());
+                    connection.DbConnection,
+                    new DatabaseModelFactoryOptions(Enumerable.Empty<string>(), Enumerable.Empty<string>()));
 
                 var operations = new List<MigrationOperation>();
 

test/EFCore.Sqlite.FunctionalTests/Scaffolding/SqliteDatabaseModelFactoryTest.cs

@@ -55,7 +55,9 @@ public SqliteDatabaseModelFactoryTest(SqliteDatabaseModelFixture fixture)
                     .BuildServiceProvider()
                     .GetRequiredService<IDatabaseModelFactory>();
 
-                var databaseModel = databaseModelFactory.Create(Fixture.TestStore.ConnectionString, tables, schemas);
+                var databaseModel = databaseModelFactory.Create(
+                    Fixture.TestStore.ConnectionString,
+                    new DatabaseModelFactoryOptions(tables, schemas));
                 Assert.NotNull(databaseModel);
                 asserter(databaseModel);
             }