Skip to content
Browse files

Merge pull request #6715 from arunagw/add_asserts

Need a assert here in tests
  • Loading branch information...
2 parents a268d2d + ddf58fa commit 6ff78a188a953fb06ebae469e17d3fa6917f08e2 @rafaelfranca rafaelfranca committed Jun 12, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 activerecord/test/cases/mass_assignment_security_test.rb
View
4 activerecord/test/cases/mass_assignment_security_test.rb
@@ -95,11 +95,11 @@ def test_mass_assigning_invalid_attribute
end
def test_mass_assigning_does_not_choke_on_nil
- Firm.new.assign_attributes(nil)
+ assert_nil Firm.new.assign_attributes(nil)
end
def test_mass_assigning_does_not_choke_on_empty_hash
- Firm.new.assign_attributes({})
+ assert_nil Firm.new.assign_attributes({})
end
def test_assign_attributes_uses_default_role_when_no_role_is_provided

0 comments on commit 6ff78a1

Please sign in to comment.
Something went wrong with that request. Please try again.