Fix inverting drop_table without options

Author: fatkodima

activerecord/lib/active_record/migration/command_recorder.rb

@@ -213,7 +213,9 @@ def invert_drop_table(args, &block)
             raise ActiveRecord::IrreversibleMigration, "To avoid mistakes, drop_table is only reversible if given options or a block (can be empty)."
           end
 
-          super(args.push(options), &block)
+          args << options unless options.empty?
+
+          super(args, &block)
         end
 
         def invert_rename_table(args)

activerecord/test/cases/invertible_migration_test.rb

@@ -154,6 +154,14 @@ def change
       end
     end
 
+    class DropTableMigration < SilentMigration
+      def change
+        drop_table("horses") do |t|
+          t.string :name
+        end
+      end
+    end
+
     class LegacyMigration < ActiveRecord::Migration::Current
       def self.up
         create_table("horses") do |t|
@@ -433,6 +441,20 @@ def test_migrate_enable_and_disable_extension
       end
     end
 
+    def test_migrate_revert_drop_table
+      connection = ActiveRecord::Base.lease_connection
+      migration1 = InvertibleMigration.new
+      migration1.migrate(:up)
+      assert connection.table_exists?("horses")
+
+      migration2 = DropTableMigration.new
+      migration2.migrate(:up)
+      assert_not connection.table_exists?("horses")
+
+      migration2.migrate(:down)
+      assert connection.table_exists?("horses")
+    end
+
     def test_revert_order
       block = Proc.new { |t| t.string :name }
       recorder = ActiveRecord::Migration::CommandRecorder.new(ActiveRecord::Base.lease_connection)