diff options
| author | Sean Griffin <sean@seantheprogrammer.com> | 2017-07-21 13:55:24 -0400 |
|---|---|---|
| committer | Sean Griffin <sean@seantheprogrammer.com> | 2017-07-21 14:01:08 -0400 |
| commit | 93c9a95e013ba7c77cf381bab9ead4a0de37e128 (patch) | |
| tree | 80d9c83c1be59f8227900369673607660d43b0c0 /activerecord/test/cases/adapters/postgresql/connection_test.rb | |
| parent | 84eb498f84ebc5d1be0b0db6f7bca9da3d679ca6 (diff) | |
| download | rails-93c9a95e013ba7c77cf381bab9ead4a0de37e128.tar.gz rails-93c9a95e013ba7c77cf381bab9ead4a0de37e128.tar.bz2 rails-93c9a95e013ba7c77cf381bab9ead4a0de37e128.zip | |
Revert "Extract `bind_param` and `bind_attribute` into `ActiveRecord::TestCase`"
This reverts commit b6ad4052d18e4b29b8a092526c2beef013e2bf4f.
This is not something that the majority of Active Record should be
testing or care about. We should look at having fewer places rely on
these details, not make it easier to rely on them.
Diffstat (limited to 'activerecord/test/cases/adapters/postgresql/connection_test.rb')
| -rw-r--r-- | activerecord/test/cases/adapters/postgresql/connection_test.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/test/cases/adapters/postgresql/connection_test.rb b/activerecord/test/cases/adapters/postgresql/connection_test.rb index 32afe331fa..ec90372545 100644 --- a/activerecord/test/cases/adapters/postgresql/connection_test.rb +++ b/activerecord/test/cases/adapters/postgresql/connection_test.rb @@ -133,8 +133,8 @@ module ActiveRecord if ActiveRecord::Base.connection.prepared_statements def test_statement_key_is_logged - binds = [bind_attribute(nil, 1)] - @connection.exec_query("SELECT $1::integer", "SQL", binds, prepare: true) + bind = Relation::QueryAttribute.new(nil, 1, Type::Value.new) + @connection.exec_query("SELECT $1::integer", "SQL", [bind], prepare: true) name = @subscriber.payloads.last[:statement_name] assert name res = @connection.exec_query("EXPLAIN (FORMAT JSON) EXECUTE #{name}(1)") |