Add cases to test combining validation conditions
- Test condition that is defined by array of conditions - Test condition that is defined by combining :if and :unless - Test local condition that is defined by :if - Test local condition that is defined by :unless See http://edgeguides.rubyonrails.org/active_record_validations.html#combining-validation-conditions
This commit is contained in:
parent
e617fb57f5
commit
3ff2c15837
|
@ -18,6 +18,22 @@ class ConditionalValidationTest < ActiveModel::TestCase
|
||||||
assert_equal ["hoo 5"], t.errors["title"]
|
assert_equal ["hoo 5"], t.errors["title"]
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_if_validation_using_array_of_true_methods
|
||||||
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", if: [:condition_is_true, :condition_is_true])
|
||||||
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
|
assert t.invalid?
|
||||||
|
assert t.errors[:title].any?
|
||||||
|
assert_equal ["hoo 5"], t.errors["title"]
|
||||||
|
end
|
||||||
|
|
||||||
|
def test_unless_validation_using_array_of_false_methods
|
||||||
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", unless: [:condition_is_false, :condition_is_false])
|
||||||
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
|
assert t.invalid?
|
||||||
|
assert t.errors[:title].any?
|
||||||
|
assert_equal ["hoo 5"], t.errors["title"]
|
||||||
|
end
|
||||||
|
|
||||||
def test_unless_validation_using_method_true
|
def test_unless_validation_using_method_true
|
||||||
# When the method returns true
|
# When the method returns true
|
||||||
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", unless: :condition_is_true)
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", unless: :condition_is_true)
|
||||||
|
@ -26,9 +42,23 @@ class ConditionalValidationTest < ActiveModel::TestCase
|
||||||
assert_empty t.errors[:title]
|
assert_empty t.errors[:title]
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_if_validation_using_array_of_true_and_false_methods
|
||||||
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", if: [:condition_is_true, :condition_is_false])
|
||||||
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
|
assert t.valid?
|
||||||
|
assert_empty t.errors[:title]
|
||||||
|
end
|
||||||
|
|
||||||
|
def test_unless_validation_using_array_of_true_and_felse_methods
|
||||||
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", unless: [:condition_is_true, :condition_is_false])
|
||||||
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
|
assert t.valid?
|
||||||
|
assert_empty t.errors[:title]
|
||||||
|
end
|
||||||
|
|
||||||
def test_if_validation_using_method_false
|
def test_if_validation_using_method_false
|
||||||
# When the method returns false
|
# When the method returns false
|
||||||
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", if: :condition_is_true_but_its_not)
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", if: :condition_is_false)
|
||||||
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
assert t.valid?
|
assert t.valid?
|
||||||
assert_empty t.errors[:title]
|
assert_empty t.errors[:title]
|
||||||
|
@ -36,7 +66,7 @@ class ConditionalValidationTest < ActiveModel::TestCase
|
||||||
|
|
||||||
def test_unless_validation_using_method_false
|
def test_unless_validation_using_method_false
|
||||||
# When the method returns false
|
# When the method returns false
|
||||||
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", unless: :condition_is_true_but_its_not)
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", unless: :condition_is_false)
|
||||||
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
assert t.invalid?
|
assert t.invalid?
|
||||||
assert t.errors[:title].any?
|
assert t.errors[:title].any?
|
||||||
|
@ -80,4 +110,19 @@ class ConditionalValidationTest < ActiveModel::TestCase
|
||||||
assert t.errors[:title].any?
|
assert t.errors[:title].any?
|
||||||
assert_equal ["hoo 5"], t.errors["title"]
|
assert_equal ["hoo 5"], t.errors["title"]
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def test_validation_using_conbining_if_true_and_unless_true_conditions
|
||||||
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", if: :condition_is_true, unless: :condition_is_true)
|
||||||
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
|
assert t.valid?
|
||||||
|
assert_empty t.errors[:title]
|
||||||
|
end
|
||||||
|
|
||||||
|
def test_validation_using_conbining_if_true_and_unless_false_conditions
|
||||||
|
Topic.validates_length_of(:title, maximum: 5, too_long: "hoo %{count}", if: :condition_is_true, unless: :condition_is_false)
|
||||||
|
t = Topic.new("title" => "uhohuhoh", "content" => "whatever")
|
||||||
|
assert t.invalid?
|
||||||
|
assert t.errors[:title].any?
|
||||||
|
assert_equal ["hoo 5"], t.errors["title"]
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -59,7 +59,7 @@ class NumericalityValidationTest < ActiveModel::TestCase
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_validates_numericality_of_with_integer_only_and_symbol_as_value
|
def test_validates_numericality_of_with_integer_only_and_symbol_as_value
|
||||||
Topic.validates_numericality_of :approved, only_integer: :condition_is_true_but_its_not
|
Topic.validates_numericality_of :approved, only_integer: :condition_is_false
|
||||||
|
|
||||||
invalid!(NIL + BLANK + JUNK)
|
invalid!(NIL + BLANK + JUNK)
|
||||||
valid!(FLOATS + INTEGERS + BIGDECIMAL + INFINITY)
|
valid!(FLOATS + INTEGERS + BIGDECIMAL + INFINITY)
|
||||||
|
|
|
@ -62,17 +62,23 @@ class ValidatesTest < ActiveModel::TestCase
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_validates_with_if_as_local_conditions
|
def test_validates_with_if_as_local_conditions
|
||||||
Person.validates :karma, presence: true, email: { unless: :condition_is_true }
|
Person.validates :karma, presence: true, email: { if: :condition_is_false }
|
||||||
person = Person.new
|
person = Person.new
|
||||||
person.valid?
|
person.valid?
|
||||||
assert_equal ["can't be blank"], person.errors[:karma]
|
assert_equal ["can't be blank"], person.errors[:karma]
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_validates_with_if_as_shared_conditions
|
def test_validates_with_if_as_shared_conditions
|
||||||
Person.validates :karma, presence: true, email: true, if: :condition_is_true
|
Person.validates :karma, presence: true, email: true, if: :condition_is_false
|
||||||
|
person = Person.new
|
||||||
|
assert person.valid?
|
||||||
|
end
|
||||||
|
|
||||||
|
def test_validates_with_unless_as_local_conditions
|
||||||
|
Person.validates :karma, presence: true, email: { unless: :condition_is_true }
|
||||||
person = Person.new
|
person = Person.new
|
||||||
person.valid?
|
person.valid?
|
||||||
assert_equal ["can't be blank", "is not an email"], person.errors[:karma].sort
|
assert_equal ["can't be blank"], person.errors[:karma]
|
||||||
end
|
end
|
||||||
|
|
||||||
def test_validates_with_unless_shared_conditions
|
def test_validates_with_unless_shared_conditions
|
||||||
|
|
|
@ -9,6 +9,10 @@ class Person
|
||||||
def condition_is_true
|
def condition_is_true
|
||||||
true
|
true
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def condition_is_false
|
||||||
|
false
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
class Person::Gender
|
class Person::Gender
|
||||||
|
|
|
@ -23,7 +23,7 @@ class Topic
|
||||||
true
|
true
|
||||||
end
|
end
|
||||||
|
|
||||||
def condition_is_true_but_its_not
|
def condition_is_false
|
||||||
false
|
false
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue