From 213ff7ca0c27739715bda83a9c6237c03a577c2b Mon Sep 17 00:00:00 2001 From: Jay Hayes Date: Sun, 26 Apr 2015 22:17:05 -0500 Subject: [PATCH] Add tests to verify exit status for create/drop failures * Running the db:create task when the database already exists isn't really an error case. That is processing may proceed in this case because the database exists as requested. So let's validate that behavior with a test. * Likewise, if the database doesn't exist when running the db:drop task processing may continue as the requested condition is already met. Thus a test. --- railties/test/application/rake/dbs_test.rb | 28 ++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/railties/test/application/rake/dbs_test.rb b/railties/test/application/rake/dbs_test.rb index e7beab8b5e..8731b7d86b 100644 --- a/railties/test/application/rake/dbs_test.rb +++ b/railties/test/application/rake/dbs_test.rb @@ -49,6 +49,34 @@ module ApplicationTests db_create_and_drop database_url_db_name end + def with_database_existing + Dir.chdir(app_path) do + set_database_url + `bin/rake db:create` + yield + `bin/rake db:drop` + end + end + + test 'db:create failure because database exists' do + with_database_existing do + output = `bin/rake db:create 2>&1` + assert_match /already exists/, output + assert_equal 0, $?.exitstatus + end + end + + test 'db:drop failure because database does not exist' do + Dir.chdir(app_path) do + output = `bin/rake db:drop 2>&1` + # This assertion should work, but it does not. The SQLite3 adapter + # does not raise an error when nothing exists to drop. + # https://github.com/rails/rails/blob/f00554a8226b9529c38be1f3e61b6b1888682fb4/activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rb#L34-L37 + # assert_match /does not exist/, output + assert_equal 0, $?.exitstatus + end + end + def db_migrate_and_status(expected_database) Dir.chdir(app_path) do `bin/rails generate model book title:string;