Don't use assert_nothing_raised when assert_equal is used

1 files changed, 2 insertions, 8 deletions

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