From 81bc621ed01637e8305fb53c9e504b1cc46c102f Mon Sep 17 00:00:00 2001
From: kamille-321 <yuji.kmjm@gmail.com>
Date: Tue, 19 Feb 2019 11:50:31 +0900
Subject: Fix query attribute method on user-defined attribute to be aware of
 typecasted value

change the line to check an attribute has user-defined type

ref: https://github.com/rails/rails/pull/35320#discussion_r257924552

check query attribute method is working when given value does not respond to to_i method
---
 activerecord/test/cases/attribute_methods_test.rb | 63 +++++++++++++++++++++++
 1 file changed, 63 insertions(+)

(limited to 'activerecord/test/cases')

diff --git a/activerecord/test/cases/attribute_methods_test.rb b/activerecord/test/cases/attribute_methods_test.rb
index 12ff6d4826..b7519bab81 100644
--- a/activerecord/test/cases/attribute_methods_test.rb
+++ b/activerecord/test/cases/attribute_methods_test.rb
@@ -439,6 +439,22 @@ class AttributeMethodsTest < ActiveRecord::TestCase
     end
   end
 
+  test "user defined text attribute predicate" do
+    klass = Class.new(ActiveRecord::Base) do
+      self.table_name = Topic.table_name
+
+      attribute :user_defined_text, :text
+    end
+
+    topic = klass.new(user_defined_text: "text")
+    assert_predicate topic, :user_defined_text?
+
+    ActiveModel::Type::Boolean::FALSE_VALUES.each do |value|
+      topic = klass.new(user_defined_text: value)
+      assert_predicate topic, :user_defined_text?
+    end
+  end
+
   test "number attribute predicate" do
     [nil, 0, "0"].each do |value|
       assert_equal false, Developer.new(salary: value).salary?
@@ -458,6 +474,53 @@ class AttributeMethodsTest < ActiveRecord::TestCase
     end
   end
 
+  test "user defined date attribute predicate" do
+    klass = Class.new(ActiveRecord::Base) do
+      self.table_name = Topic.table_name
+
+      attribute :user_defined_date, :date
+    end
+
+    topic = klass.new(user_defined_date: Date.current)
+    assert_predicate topic, :user_defined_date?
+  end
+
+  test "user defined datetime attribute predicate" do
+    klass = Class.new(ActiveRecord::Base) do
+      self.table_name = Topic.table_name
+
+      attribute :user_defined_datetime, :datetime
+    end
+
+    topic = klass.new(user_defined_datetime: Time.current)
+    assert_predicate topic, :user_defined_datetime?
+  end
+
+  test "user defined time attribute predicate" do
+    klass = Class.new(ActiveRecord::Base) do
+      self.table_name = Topic.table_name
+
+      attribute :user_defined_time, :time
+    end
+
+    topic = klass.new(user_defined_time: Time.current)
+    assert_predicate topic, :user_defined_time?
+  end
+
+  test "user defined json attribute predicate" do
+    klass = Class.new(ActiveRecord::Base) do
+      self.table_name = Topic.table_name
+
+      attribute :user_defined_json, :json
+    end
+
+    topic = klass.new(user_defined_json: { key: "value" })
+    assert_predicate topic, :user_defined_json?
+
+    topic = klass.new(user_defined_json: {})
+    assert_not_predicate topic, :user_defined_json?
+  end
+
   test "custom field attribute predicate" do
     object = Company.find_by_sql(<<~SQL).first
       SELECT c1.*, c2.type as string_value, c2.rating as int_value
-- 
cgit v1.2.3