Skip to content

Commit 6ff78a1

Browse files
committed
Merge pull request rails#6715 from arunagw/add_asserts
Need a assert here in tests
2 parents a268d2d + ddf58fa commit 6ff78a1

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

activerecord/test/cases/mass_assignment_security_test.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,11 +95,11 @@ def test_mass_assigning_invalid_attribute
9595
end
9696

9797
def test_mass_assigning_does_not_choke_on_nil
98-
Firm.new.assign_attributes(nil)
98+
assert_nil Firm.new.assign_attributes(nil)
9999
end
100100

101101
def test_mass_assigning_does_not_choke_on_empty_hash
102-
Firm.new.assign_attributes({})
102+
assert_nil Firm.new.assign_attributes({})
103103
end
104104

105105
def test_assign_attributes_uses_default_role_when_no_role_is_provided

0 commit comments

Comments
 (0)