move column ordering tests to it's own class

2 files changed, 58 insertions, 46 deletions

diff --git a/activerecord/test/cases/migration/change_schema_test.rb b/activerecord/test/cases/migration/change_schema_test.rb
index b8b27ded17..f8eedc712f 100644
--- a/activerecord/test/cases/migration/change_schema_test.rb
+++ b/activerecord/test/cases/migration/change_schema_test.rb
@@ -244,38 +244,6 @@ module ActiveRecord
         end
       end
 
-      def test_column_positioning
-        testing_table_for_positioning do |conn|
-          assert_equal %w(first second third), conn.columns(:testings).map {|c| c.name }
-        end
-      end
-
-      def test_add_column_with_positioning
-        testing_table_for_positioning do |conn|
-          conn.add_column :testings, :new_col, :integer
-          assert_equal %w(first second third new_col), conn.columns(:testings).map {|c| c.name }
-        end
-        testing_table_for_positioning do |conn|
-          conn.add_column :testings, :new_col, :integer, :first => true
-          assert_equal %w(new_col first second third), conn.columns(:testings).map {|c| c.name }
-        end
-        testing_table_for_positioning do |conn|
-          conn.add_column :testings, :new_col, :integer, :after => :first
-          assert_equal %w(first new_col second third), conn.columns(:testings).map {|c| c.name }
-        end
-      end
-
-      def test_change_column_with_positioning
-        testing_table_for_positioning do |conn|
-          conn.change_column :testings, :second, :integer, :first => true
-          assert_equal %w(second first third), conn.columns(:testings).map {|c| c.name }
-        end
-        testing_table_for_positioning do |conn|
-          conn.change_column :testings, :second, :integer, :after => :third
-          assert_equal %w(first third second), conn.columns(:testings).map {|c| c.name }
-        end
-      end
-
       def test_change_column_quotes_column_names
         connection.create_table :testings do |t|
           t.column :select, :string
@@ -395,20 +363,6 @@ module ActiveRecord
 
         yield
       end
-
-      def testing_table_for_positioning
-        unless current_adapter?(:MysqlAdapter, :Mysql2Adapter)
-          skip "not supported on #{connection.class}"
-        end
-
-        connection.create_table :testings, :id => false do |t|
-          t.column :first, :integer
-          t.column :second, :integer
-          t.column :third, :integer
-        end
-
-        yield connection
-      end
     end
   end
 end
diff --git a/activerecord/test/cases/migration/column_positioning_test.rb b/activerecord/test/cases/migration/column_positioning_test.rb
new file mode 100644
index 0000000000..9a7e1e1f60
--- /dev/null
+++ b/activerecord/test/cases/migration/column_positioning_test.rb
@@ -0,0 +1,58 @@
+module ActiveRecord
+  class Migration
+    class ColumnPositioningTest < ActiveRecord::TestCase
+      attr_reader :connection, :table_name
+      alias :conn :connection
+
+      def setup
+        super
+
+        unless current_adapter?(:MysqlAdapter, :Mysql2Adapter)
+          skip "not supported on #{connection.class}"
+        end
+
+        @connection = ActiveRecord::Base.connection
+
+        connection.create_table :testings, :id => false do |t|
+          t.column :first, :integer
+          t.column :second, :integer
+          t.column :third, :integer
+        end
+      end
+
+      def teardown
+        super
+        connection.drop_table :testings rescue nil
+        ActiveRecord::Base.primary_key_prefix_type = nil
+      end
+
+      def test_column_positioning
+        assert_equal %w(first second third), conn.columns(:testings).map {|c| c.name }
+      end
+
+      def test_add_column_with_positioning
+        conn.add_column :testings, :new_col, :integer
+        assert_equal %w(first second third new_col), conn.columns(:testings).map {|c| c.name }
+      end
+
+      def test_add_column_with_positioning_first
+        conn.add_column :testings, :new_col, :integer, :first => true
+        assert_equal %w(new_col first second third), conn.columns(:testings).map {|c| c.name }
+      end
+
+      def test_add_column_with_positioning_after
+        conn.add_column :testings, :new_col, :integer, :after => :first
+        assert_equal %w(first new_col second third), conn.columns(:testings).map {|c| c.name }
+      end
+
+      def test_change_column_with_positioning
+        conn.change_column :testings, :second, :integer, :first => true
+        assert_equal %w(second first third), conn.columns(:testings).map {|c| c.name }
+
+        conn.change_column :testings, :second, :integer, :after => :third
+        assert_equal %w(first third second), conn.columns(:testings).map {|c| c.name }
+      end
+
+    end
+  end
+end