Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Remove more skip

  • Loading branch information...
commit 78bac468e6ac0c97ff81c15fe81c5a5a99c1c177 1 parent bd09afb
@rafaelfranca rafaelfranca authored
View
14 activerecord/test/cases/migration/index_test.rb
@@ -185,14 +185,14 @@ def test_add_index
end
end
- def test_add_partial_index
- skip 'only on pg' unless current_adapter?(:PostgreSQLAdapter)
+ if current_adapter?(:PostgreSQLAdapter)
+ def test_add_partial_index
+ connection.add_index("testings", "last_name", :where => "first_name = 'john doe'")
+ assert connection.index_exists?("testings", "last_name")
- connection.add_index("testings", "last_name", :where => "first_name = 'john doe'")
- assert connection.index_exists?("testings", "last_name")
-
- connection.remove_index("testings", "last_name")
- assert !connection.index_exists?("testings", "last_name")
+ connection.remove_index("testings", "last_name")
+ assert !connection.index_exists?("testings", "last_name")
+ end
end
private
View
2  activerecord/test/cases/migration/rename_table_test.rb
@@ -78,8 +78,6 @@ def test_rename_table_does_not_rename_custom_named_index
if current_adapter?(:PostgreSQLAdapter)
def test_rename_table_for_postgresql_should_also_rename_default_sequence
- skip 'not supported'
-
rename_table :test_models, :octopi
pk, seq = connection.pk_and_sequence_for('octopi')
View
2  activerecord/test/cases/migration_test.rb
@@ -442,8 +442,6 @@ def test_create_table_with_binary_column
if current_adapter? :OracleAdapter
def test_create_table_with_custom_sequence_name
- skip "not supported"
-
# table name is 29 chars, the standard sequence name will
# be 33 chars and should be shortened
assert_nothing_raised do
View
2  activerecord/test/cases/primary_keys_test.rb
@@ -187,8 +187,6 @@ class PrimaryKeyWithNoConnectionTest < ActiveRecord::TestCase
unless in_memory_db?
def test_set_primary_key_with_no_connection
- return skip("disconnect wipes in-memory db")
-
connection = ActiveRecord::Base.remove_connection
model = Class.new(ActiveRecord::Base)
Please sign in to comment.
Something went wrong with that request. Please try again.