diff options
| author | Sean Griffin <sean@thoughtbot.com> | 2014-11-20 19:17:10 -0700 |
|---|---|---|
| committer | Sean Griffin <sean@thoughtbot.com> | 2014-11-20 19:17:10 -0700 |
| commit | 17efb3b919e8eff1ca5da49b12e3c1f607eb93e3 (patch) | |
| tree | 52fdbc0f20d053f11b1db3a37d3d8e064696d36f /activerecord/test | |
| parent | f767ac22fa213df754e160c59189d28ed4f95568 (diff) | |
| download | rails-17efb3b919e8eff1ca5da49b12e3c1f607eb93e3.tar.gz rails-17efb3b919e8eff1ca5da49b12e3c1f607eb93e3.tar.bz2 rails-17efb3b919e8eff1ca5da49b12e3c1f607eb93e3.zip | |
Remove is_a? check when ignoring tables
Technically changes the API, as it will allow any object which responds
to `===`. Personally, I think this is more flexible.
Diffstat (limited to 'activerecord/test')
| -rw-r--r-- | activerecord/test/cases/schema_dumper_test.rb | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/activerecord/test/cases/schema_dumper_test.rb b/activerecord/test/cases/schema_dumper_test.rb index 6303393c22..8731531f54 100644 --- a/activerecord/test/cases/schema_dumper_test.rb +++ b/activerecord/test/cases/schema_dumper_test.rb @@ -162,16 +162,6 @@ class SchemaDumperTest < ActiveRecord::TestCase assert_no_match %r{create_table "schema_migrations"}, output end - def test_schema_dump_illegal_ignored_table_value - stream = StringIO.new - old_ignore_tables, ActiveRecord::SchemaDumper.ignore_tables = ActiveRecord::SchemaDumper.ignore_tables, [5] - assert_raise(StandardError) do - ActiveRecord::SchemaDumper.dump(ActiveRecord::Base.connection, stream) - end - ensure - ActiveRecord::SchemaDumper.ignore_tables = old_ignore_tables - end - def test_schema_dumps_index_columns_in_right_order index_definition = standard_dump.split(/\n/).grep(/add_index.*companies/).first.strip if current_adapter?(:MysqlAdapter, :Mysql2Adapter, :PostgreSQLAdapter) |