Should test test_change_column_default in PostgresqlUUIDTest

Follow up to #25395.
rails · Nov 24, 2016 · e683950 · e683950
1 parent 1b795f5
commit e683950
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/activerecord/test/cases/adapters/postgresql/uuid_test.rb b/activerecord/test/cases/adapters/postgresql/uuid_test.rb
@@ -43,20 +43,20 @@ def test_uuid_column_default
       column = UUIDType.columns_hash["thingy"]
       assert_equal "gen_random_uuid()", column.default_function
     end
-  else
-    def test_change_column_default
-      connection.add_column :uuid_data_type, :thingy, :uuid, null: false, default: "uuid_generate_v1()"
-      UUIDType.reset_column_information
-      column = UUIDType.columns_hash["thingy"]
-      assert_equal "uuid_generate_v1()", column.default_function
+  end
 
-      connection.change_column :uuid_data_type, :thingy, :uuid, null: false, default: "uuid_generate_v4()"
-      UUIDType.reset_column_information
-      column = UUIDType.columns_hash["thingy"]
-      assert_equal "uuid_generate_v4()", column.default_function
-    ensure
-      UUIDType.reset_column_information
-    end
+  def test_change_column_default
+    connection.add_column :uuid_data_type, :thingy, :uuid, null: false, default: "uuid_generate_v1()"
+    UUIDType.reset_column_information
+    column = UUIDType.columns_hash["thingy"]
+    assert_equal "uuid_generate_v1()", column.default_function
+
+    connection.change_column :uuid_data_type, :thingy, :uuid, null: false, default: "uuid_generate_v4()"
+    UUIDType.reset_column_information
+    column = UUIDType.columns_hash["thingy"]
+    assert_equal "uuid_generate_v4()", column.default_function
+  ensure
+    UUIDType.reset_column_information
   end
 
   def test_data_type_of_uuid_types