record unsupported methods in CommandRecorder instead of letting the unsupported methods go through to the underlying db causing errors like duplicate columns to occur when rolling back migrations

activerecord/lib/active_record/migration/command_recorder.rb

@@ -93,11 +93,11 @@ module ActiveRecord
         [:remove_timestamps, args]
       end
 
-      # Forwards any missing method call to the \target.
+      # Record all the methods called in the +change+ method of a migration.
+      # This will ensure that IrreversibleMigration is raised when the corresponding
+      # invert_method does not exist while the migration is rolled back.
       def method_missing(method, *args, &block)
-        @delegate.send(method, *args, &block)
+        record(method, args)
-      rescue NoMethodError => e
-        raise e, e.message.sub(/ for #<.*$/, " via proxy for #{@delegate}")
       end
 
     end

activerecord/test/cases/migration/command_recorder_test.rb

@@ -14,9 +14,11 @@ module ActiveRecord
         assert recorder.respond_to?(:america)
       end
 
-      def test_send_calls_super
+      def test_non_existing_method_records_and_raises_on_inversion
-        assert_raises(NoMethodError) do
         @recorder.send(:non_existing_method, :horses)
+        assert_equal 1, @recorder.commands.length
+        assert_raises(ActiveRecord::IrreversibleMigration) do
+          @recorder.inverse
         end
       end