From 8cfe95d7194a1118f9950fe1d81a4e915cb929d6 Mon Sep 17 00:00:00 2001 From: Guillermo Iguaran Date: Wed, 29 Aug 2012 09:54:27 -0500 Subject: Don't use assert_nothing_raised when assert_equal is used --- .../test/cases/forbidden_attributes_protection_test.rb | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) (limited to 'activerecord/test/cases') diff --git a/activerecord/test/cases/forbidden_attributes_protection_test.rb b/activerecord/test/cases/forbidden_attributes_protection_test.rb index 89e9de261a..4706eaec2b 100644 --- a/activerecord/test/cases/forbidden_attributes_protection_test.rb +++ b/activerecord/test/cases/forbidden_attributes_protection_test.rb @@ -34,21 +34,17 @@ class ForbiddenAttributesProtectionTest < ActiveRecord::TestCase def test_permitted_attributes_can_be_used_for_mass_assignment params = ProtectedParams.new(first_name: 'Guille', gender: 'm') params.permit! - assert_nothing_raised do - person = Person.new(params) + person = Person.new(params) - assert_equal 'Guille', person.first_name - assert_equal 'm', person.gender - end + assert_equal 'Guille', person.first_name + assert_equal 'm', person.gender end def test_regular_hash_should_still_be_used_for_mass_assignment - assert_nothing_raised do - person = Person.new(first_name: 'Guille', gender: 'm') + person = Person.new(first_name: 'Guille', gender: 'm') - assert_equal 'Guille', person.first_name - assert_equal 'm', person.gender - end + assert_equal 'Guille', person.first_name + assert_equal 'm', person.gender end def test_protected_attributes_cannot_be_used_for_mass_assignment -- cgit v1.2.3