From 79c8c4fed46775f30551f6d4b0c50b9ba397bc54 Mon Sep 17 00:00:00 2001 From: Aurelian Oancea Date: Fri, 8 Jun 2012 12:41:46 +0200 Subject: Don't assign the attributes if the list is empty. --- activerecord/test/cases/mass_assignment_security_test.rb | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'activerecord/test') diff --git a/activerecord/test/cases/mass_assignment_security_test.rb b/activerecord/test/cases/mass_assignment_security_test.rb index 2f98d3c646..fc89ced830 100644 --- a/activerecord/test/cases/mass_assignment_security_test.rb +++ b/activerecord/test/cases/mass_assignment_security_test.rb @@ -98,6 +98,10 @@ class MassAssignmentSecurityTest < ActiveRecord::TestCase Firm.new.assign_attributes(nil) end + def test_mass_assigning_does_not_choke_on_empty_hash + Firm.new.assign_attributes({}) + end + def test_assign_attributes_uses_default_role_when_no_role_is_provided p = LoosePerson.new p.assign_attributes(attributes_hash) -- cgit v1.2.3