Ticket #6999: preserve_options_rename_column_mysql.diff

test/migration_test.rb

@@ -362 +362 @@
-      end
-    end
-
+    if current_adapter?(:MysqlAdapter)
+      def test_rename_column_preserve_auto_increment
+        begin
+          ActiveRecord::Base.connection.create_table(:legacy) {} # We just need the id column
+          legacy = Class.new(ActiveRecord::Base)
+          legacy.send(:define_attr_method, :table_name, "legacy")
+          options = legacy.connection.select_one("SHOW COLUMNS FROM legacy LIKE 'id'")
+          assert_equal 'auto_increment', options['Extra'] 
+
+          legacy.connection.rename_column("legacy", "id", "orig_id")
+          options = legacy.connection.select_one("SHOW COLUMNS FROM legacy LIKE 'orig_id'")
+          assert_equal 'auto_increment', options['Extra'] 
+        ensure
+          ActiveRecord::Base.connection.drop_table :legacy rescue nil
+        end
+      end
+    end
+
+    def test_rename_column_preserve_default
+      begin
+        Person.connection.add_column "people", "contibutions", :integer, :default => 10, :null => false
+        Person.reset_column_information
+        columns = Person.connection.columns(:people)
+        column = columns.detect { |c| c.name == "contibutions" }
+        assert_equal 10, column.default
+        assert !column.null
+
+        Person.connection.rename_column("people", "contibutions", "contributions")
+        Person.reset_column_information
+        columns = Person.connection.columns(:people)
+        column = columns.detect { |c| c.name == "contributions" }
+        assert_equal 10, column.default
+        assert !column.null
+      ensure
+        Person.connection.remove_column("people", "contibutions") rescue ActiveRecord::StatementInvalid
+        Person.connection.remove_column("people", "contributions") rescue ActiveRecord::StatementInvalid
+      end
+    end
+
-    def test_rename_table
-      begin
-        ActiveRecord::Base.connection.create_table :octopuses do |t|

lib/active_record/connection_adapters/mysql_adapter.rb

@@ -380 +380 @@
-      end
-
-      def rename_column(table_name, column_name, new_column_name) #:nodoc:
-
-
+
+
+
+
+
-      end
-
-
-      private
-        def connect
-          encoding = @config[:encoding]