Skip to content
Browse files

Removing FIXME

As we did in master.
  • Loading branch information...
1 parent a5013bb commit c8918c6972d122531a5ae750ae1b22ff7d539f10 @arunagw arunagw committed Mar 8, 2013
Showing with 8 additions and 9 deletions.
  1. +8 −9 activerecord/test/cases/bind_parameter_test.rb
View
17 activerecord/test/cases/bind_parameter_test.rb
@@ -23,16 +23,15 @@ def setup
@listener = LogListener.new
@pk = Topic.columns.find { |c| c.primary }
ActiveSupport::Notifications.subscribe('sql.active_record', @listener)
+
+ skip_if_prepared_statement_caching_is_not_supported
end
def teardown
ActiveSupport::Notifications.unsubscribe(@listener)
end
def test_binds_are_logged
- # FIXME: use skip with minitest
- return unless @connection.supports_statement_cache?
-
sub = @connection.substitute_at(@pk, 0)
binds = [[@pk, 1]]
sql = "select * from topics where id = #{sub}"
@@ -44,19 +43,13 @@ def test_binds_are_logged
end
def test_find_one_uses_binds
- # FIXME: use skip with minitest
- return unless @connection.supports_statement_cache?
-
Topic.find(1)
binds = [[@pk, 1]]
message = @listener.calls.find { |args| args[4][:binds] == binds }
assert message, 'expected a message with binds'
end
def test_logs_bind_vars
- # FIXME: use skip with minitest
- return unless @connection.supports_statement_cache?
-
pk = Topic.columns.find { |x| x.primary }
payload = {
@@ -86,5 +79,11 @@ def debug str
logger.sql event
assert_match([[pk.name, 10]].inspect, logger.debugs.first)
end
+
+ private
+
+ def skip_if_prepared_statement_caching_is_not_supported
+ return skip('prepared statement caching is not supported') unless @connection.supports_statement_cache?
+ end
end
end

0 comments on commit c8918c6

Please sign in to comment.
Something went wrong with that request. Please try again.