From 4775d3d051af1a506b63a7e4069c94e49c82177d Mon Sep 17 00:00:00 2001 From: Gannon McGibbon Date: Fri, 28 Sep 2018 13:36:06 -0400 Subject: Refactor migrations_path command option to database --- railties/test/generators/generators_test_helper.rb | 14 ++++++++++++++ railties/test/generators/migration_generator_test.rb | 12 +++++++----- railties/test/generators/model_generator_test.rb | 12 +++++++----- railties/test/generators/scaffold_generator_test.rb | 8 +++++--- 4 files changed, 33 insertions(+), 13 deletions(-) (limited to 'railties/test') diff --git a/railties/test/generators/generators_test_helper.rb b/railties/test/generators/generators_test_helper.rb index ad2a55f496..355d4b09a8 100644 --- a/railties/test/generators/generators_test_helper.rb +++ b/railties/test/generators/generators_test_helper.rb @@ -42,6 +42,20 @@ module GeneratorsTestHelper end end + def with_secondary_database_configuration + ActiveRecord::Base.configurations = { + test: { + secondary: { + database: "db/secondary.sqlite3", + migrations_paths: "db/secondary_migrate", + }, + }, + } + yield + ensure + ActiveRecord::Base.configurations = {} + end + def copy_routes routes = File.expand_path("../../lib/rails/generators/rails/app/templates/config/routes.rb.tt", __dir__) destination = File.join(destination_root, "config") diff --git a/railties/test/generators/migration_generator_test.rb b/railties/test/generators/migration_generator_test.rb index 5c57d607fc..5812cbdfc9 100644 --- a/railties/test/generators/migration_generator_test.rb +++ b/railties/test/generators/migration_generator_test.rb @@ -254,11 +254,13 @@ class MigrationGeneratorTest < Rails::Generators::TestCase end end - def test_migrations_paths_puts_migrations_in_that_folder - run_generator ["create_books", "--migrations_paths=db/test_migrate"] - assert_migration "db/test_migrate/create_books.rb" do |content| - assert_method :change, content do |change| - assert_match(/create_table :books/, change) + def test_database_puts_migrations_in_configured_folder + with_secondary_database_configuration do + run_generator ["create_books", "--database=secondary"] + assert_migration "db/secondary_migrate/create_books.rb" do |content| + assert_method :change, content do |change| + assert_match(/create_table :books/, change) + end end end end diff --git a/railties/test/generators/model_generator_test.rb b/railties/test/generators/model_generator_test.rb index 5a0c2f74c7..b06db6dd8a 100644 --- a/railties/test/generators/model_generator_test.rb +++ b/railties/test/generators/model_generator_test.rb @@ -392,11 +392,13 @@ class ModelGeneratorTest < Rails::Generators::TestCase end end - def test_migrations_paths_puts_migrations_in_that_folder - run_generator ["account", "--migrations_paths=db/test_migrate"] - assert_migration "db/test_migrate/create_accounts.rb" do |content| - assert_method :change, content do |change| - assert_match(/create_table :accounts/, change) + def test_database_puts_migrations_in_configured_folder + with_secondary_database_configuration do + run_generator ["account", "--database=secondary"] + assert_migration "db/secondary_migrate/create_accounts.rb" do |content| + assert_method :change, content do |change| + assert_match(/create_table :accounts/, change) + end end end end diff --git a/railties/test/generators/scaffold_generator_test.rb b/railties/test/generators/scaffold_generator_test.rb index dbcf49290e..94aab15aa9 100644 --- a/railties/test/generators/scaffold_generator_test.rb +++ b/railties/test/generators/scaffold_generator_test.rb @@ -476,10 +476,12 @@ class ScaffoldGeneratorTest < Rails::Generators::TestCase end end - def test_scaffold_generator_migrations_paths - run_generator ["posts", "--migrations-paths=db/kingston_migrate"] + def test_scaffold_generator_database + with_secondary_database_configuration do + run_generator ["posts", "--database=secondary"] - assert_migration "db/kingston_migrate/create_posts.rb" + assert_migration "db/secondary_migrate/create_posts.rb" + end end def test_scaffold_generator_password_digest -- cgit v1.2.3