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 --- activemodel/test/cases/forbidden_attributes_protection_test.rb | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) (limited to 'activemodel') diff --git a/activemodel/test/cases/forbidden_attributes_protection_test.rb b/activemodel/test/cases/forbidden_attributes_protection_test.rb index f6437333b0..3cb204a2c5 100644 --- a/activemodel/test/cases/forbidden_attributes_protection_test.rb +++ b/activemodel/test/cases/forbidden_attributes_protection_test.rb @@ -27,16 +27,10 @@ class ActiveModelMassUpdateProtectionTest < ActiveSupport::TestCase test "permitted attributes can be used for mass updating" do params = ProtectedParams.new({ "a" => "b" }).permit! - assert_nothing_raised do - assert_equal({ "a" => "b" }, - Account.new.sanitize_for_mass_assignment(params)) - end + assert_equal({ "a" => "b" }, Account.new.sanitize_for_mass_assignment(params)) end test "regular attributes should still be allowed" do - assert_nothing_raised do - assert_equal({ a: "b" }, - Account.new.sanitize_for_mass_assignment(a: "b")) - end + assert_equal({ a: "b" }, Account.new.sanitize_for_mass_assignment(a: "b")) end end -- cgit v1.2.3