Permalink
Browse files

switch to arrays for bind removal test

we can't make bind values for arrays yet
  • Loading branch information...
1 parent 2f88530 commit c65cfd7648349e3a41206b6e821b3412d4d3ccd8 @tenderlove tenderlove committed Jan 14, 2014
Showing with 2 additions and 4 deletions.
  1. +2 −4 activerecord/test/cases/relations_test.rb
@@ -1534,10 +1534,8 @@ def test_unscope_removes_binds
end
def test_merging_removes_rhs_bind_parameters
- left = Post.where(id: Arel::Nodes::BindParam.new('?'))
- column = Post.columns_hash['id']
- left.bind_values += [[column, 20]]
- right = Post.where(id: 10)
+ left = Post.where(id: 20)
+ right = Post.where(id: [1,2,3,4])
merged = left.merge(right)
assert_equal [], merged.bind_values

0 comments on commit c65cfd7

Please sign in to comment.