diff options
author | Eileen M. Uchitelle <eileencodes@users.noreply.github.com> | 2019-04-19 14:02:31 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-04-19 14:02:31 -0400 |
commit | 92c464373abe7c486e189b21a6376aca01173b06 (patch) | |
tree | e7279489d298ed7e84b304cb5ae8699eefe0e4d0 /railties/test | |
parent | 1a5381ff0cf04af68a50bd04f265b9b8199e37b4 (diff) | |
parent | f9244c65859c114d4a607823e31173ac4460a6e9 (diff) | |
download | rails-92c464373abe7c486e189b21a6376aca01173b06.tar.gz rails-92c464373abe7c486e189b21a6376aca01173b06.tar.bz2 rails-92c464373abe7c486e189b21a6376aca01173b06.zip |
Merge pull request #36039 from eileencodes/add-up-and-down-to-multi-db-rake-tasks
Handle up/down for multiple databases
Diffstat (limited to 'railties/test')
-rw-r--r-- | railties/test/application/rake/multi_dbs_test.rb | 59 |
1 files changed, 58 insertions, 1 deletions
diff --git a/railties/test/application/rake/multi_dbs_test.rb b/railties/test/application/rake/multi_dbs_test.rb index 147b8f94e1..31ea2246a9 100644 --- a/railties/test/application/rake/multi_dbs_test.rb +++ b/railties/test/application/rake/multi_dbs_test.rb @@ -24,7 +24,6 @@ module ApplicationTests assert_no_match(/already exists/, output) assert File.exist?(expected_database) - output = rails("db:drop") assert_match(/Dropped database/, output) assert_match_namespace(namespace, output) @@ -137,6 +136,36 @@ module ApplicationTests end end + def db_up_and_down(version, namespace = nil) + Dir.chdir(app_path) do + generate_models_for_animals + rails("db:migrate") + + if namespace + down_output = rails("db:migrate:down:#{namespace}", "VERSION=#{version}") + up_output = rails("db:migrate:up:#{namespace}", "VERSION=#{version}") + else + assert_raises RuntimeError, /You're using a multiple database application/ do + down_output = rails("db:migrate:down", "VERSION=#{version}") + end + + assert_raises RuntimeError, /You're using a multiple database application/ do + up_output = rails("db:migrate:up", "VERSION=#{version}") + end + end + + case namespace + when "primary" + assert_match(/OneMigration: reverting/, down_output) + assert_match(/OneMigration: migrated/, up_output) + when nil + else + assert_match(/TwoMigration: reverting/, down_output) + assert_match(/TwoMigration: migrated/, up_output) + end + end + end + def db_prepare Dir.chdir(app_path) do generate_models_for_animals @@ -219,6 +248,34 @@ module ApplicationTests end end + test "db:migrate:down and db:migrate:up without a namespace raises in a multi-db application" do + require "#{app_path}/config/environment" + + app_file "db/migrate/01_one_migration.rb", <<-MIGRATION + class OneMigration < ActiveRecord::Migration::Current + end + MIGRATION + + db_up_and_down "01" + end + + test "db:migrate:down:namespace and db:migrate:up:namespace works" do + require "#{app_path}/config/environment" + + app_file "db/migrate/01_one_migration.rb", <<-MIGRATION + class OneMigration < ActiveRecord::Migration::Current + end + MIGRATION + + app_file "db/animals_migrate/02_two_migration.rb", <<-MIGRATION + class TwoMigration < ActiveRecord::Migration::Current + end + MIGRATION + + db_up_and_down "01", "primary" + db_up_and_down "02", "animals" + end + test "db:migrate:status works on all databases" do require "#{app_path}/config/environment" db_migrate_and_migrate_status |