Eagerly cast array values passed to the predicate builder

Part of a larger refactoring to remove type casting from Arel.

/cc @mrgilman

[Sean Griffin & Melanie Gilman]

1 files changed, 6 insertions, 2 deletions

diff --git a/activerecord/test/cases/relation/where_chain_test.rb b/activerecord/test/cases/relation/where_chain_test.rb
index 3a02e8230d..f9d3badceb 100644
--- a/activerecord/test/cases/relation/where_chain_test.rb
+++ b/activerecord/test/cases/relation/where_chain_test.rb
@@ -36,7 +36,8 @@ module ActiveRecord
     end
 
     def test_not_in
-      expected = Post.arel_table[@name].not_in(%w[hello goodbye])
+      values = %w[hello goodbye].map { |v| Arel::Nodes::Quoted.new(v) }
+      expected = Post.arel_table[@name].not_in(values)
       relation = Post.where.not(title: %w[hello goodbye])
       assert_equal([expected], relation.where_values)
     end
@@ -90,7 +91,10 @@ module ActiveRecord
     def test_chaining_multiple
       relation = Post.where.not(author_id: [1, 2]).where.not(title: 'ruby on rails')
 
-      expected = Post.arel_table['author_id'].not_in([1, 2])
+      expected = Post.arel_table['author_id'].not_in([
+        Arel::Nodes::Quoted.new(1),
+        Arel::Nodes::Quoted.new(2),
+      ])
       assert_equal(expected, relation.where_values[0])
 
       value = relation.where_values[1]