Skip to content

Commit

Permalink
Need a assert here in tests
Browse files Browse the repository at this point in the history
  • Loading branch information
arunagw committed Jun 12, 2012
1 parent a268d2d commit ddf58fa
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions activerecord/test/cases/mass_assignment_security_test.rb
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -95,11 +95,11 @@ def test_mass_assigning_invalid_attribute
end end


def test_mass_assigning_does_not_choke_on_nil def test_mass_assigning_does_not_choke_on_nil
Firm.new.assign_attributes(nil) assert_nil Firm.new.assign_attributes(nil)
end end


def test_mass_assigning_does_not_choke_on_empty_hash def test_mass_assigning_does_not_choke_on_empty_hash
Firm.new.assign_attributes({}) assert_nil Firm.new.assign_attributes({})
end end


def test_assign_attributes_uses_default_role_when_no_role_is_provided def test_assign_attributes_uses_default_role_when_no_role_is_provided
Expand Down

0 comments on commit ddf58fa

Please sign in to comment.