diff options
author | Vijay Dev <vijaydev.cse@gmail.com> | 2011-07-02 23:43:13 +0530 |
---|---|---|
committer | Vijay Dev <vijaydev.cse@gmail.com> | 2011-07-02 23:43:13 +0530 |
commit | 15f35c0ac0adb5cdec8dfdba407e2ad42172121e (patch) | |
tree | 043b8fa46e4d9705d7093384f86f80ccbfad3744 /activerecord/test/cases/migration | |
parent | 908be47781de892779639d5ae83b3471d46e6e48 (diff) | |
download | rails-15f35c0ac0adb5cdec8dfdba407e2ad42172121e.tar.gz rails-15f35c0ac0adb5cdec8dfdba407e2ad42172121e.tar.bz2 rails-15f35c0ac0adb5cdec8dfdba407e2ad42172121e.zip |
Reversing the changes done in c278a2c while still resolving #1857.
The changes broke bulk migration tests and were fixed in 4d256bc6;
however that brought back the issue of #1857 and so this commit goes
back to the original scenario and just adds change_table to the list
of methods which are to be recorded in the CommandRecorder. The
method_missing now delegates all calls to the underlying connection as
before.
Diffstat (limited to 'activerecord/test/cases/migration')
-rw-r--r-- | activerecord/test/cases/migration/command_recorder_test.rb | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/activerecord/test/cases/migration/command_recorder_test.rb b/activerecord/test/cases/migration/command_recorder_test.rb index 85d38fac25..36007255fa 100644 --- a/activerecord/test/cases/migration/command_recorder_test.rb +++ b/activerecord/test/cases/migration/command_recorder_test.rb @@ -14,11 +14,9 @@ module ActiveRecord assert recorder.respond_to?(:america) end - def test_non_existing_method_records_and_raises_on_inversion - @recorder.send(:non_existing_method, :horses) - assert_equal 1, @recorder.commands.length - assert_raises(ActiveRecord::IrreversibleMigration) do - @recorder.inverse + def test_send_calls_super + assert_raises(NoMethodError) do + @recorder.send(:non_existing_method, :horses) end end |