Permalink
Browse files

no need to `return skip` in tests. `skip` is enough.

  • Loading branch information...
1 parent 0167765 commit a6512cbd6ca0e120bb2f314ceb88a44724e15f22 @senny senny committed Jan 15, 2014
@@ -21,7 +21,7 @@ def setup
end
end
rescue ActiveRecord::StatementInvalid
- return skip "do not test on PG without json"
+ skip "do not test on PG without json"
end
@column = JsonDataType.columns.find { |c| c.name == 'payload' }
end
@@ -26,7 +26,7 @@ def setup
end
end
rescue ActiveRecord::StatementInvalid
- return skip "do not test on PG without range"
+ skip "do not test on PG without range"
end
insert_range(id: 101,
@@ -18,7 +18,7 @@ def setup
end
end
rescue ActiveRecord::StatementInvalid
- return skip "do not test on PG without xml"
+ skip "do not test on PG without xml"
end
@column = XmlDataType.columns.find { |c| c.name == 'payload' }
end
@@ -1129,7 +1129,7 @@ def test_sequence_name_with_abstract_class
k = Class.new(ak)
k.table_name = "projects"
orig_name = k.sequence_name
- return skip "sequences not supported by db" unless orig_name
+ skip "sequences not supported by db" unless orig_name
assert_equal k.reset_sequence_name, orig_name
end

0 comments on commit a6512cb

Please sign in to comment.