From 656720583ff0ee382e608fa8b074649d8b5a7132 Mon Sep 17 00:00:00 2001 From: Lucas Mazza Date: Mon, 27 Jun 2016 16:48:12 -0300 Subject: [PATCH] Remove `assert_nothing_raised` deprecated usage. --- test/controllers/sessions_controller_test.rb | 2 +- test/devise_test.rb | 6 +++--- test/models_test.rb | 2 +- 3 files changed, 5 insertions(+), 5 deletions(-) diff --git a/test/controllers/sessions_controller_test.rb b/test/controllers/sessions_controller_test.rb index 82d066b2..69b65f37 100644 --- a/test/controllers/sessions_controller_test.rb +++ b/test/controllers/sessions_controller_test.rb @@ -94,7 +94,7 @@ class SessionsControllerTest < Devise::ControllerTestCase User.class_eval { attr_protected :email } begin - assert_nothing_raised ActiveModel::MassAssignmentSecurity::Error do + assert_nothing_raised do get :new, user: { email: "allez viens!" } end ensure diff --git a/test/devise_test.rb b/test/devise_test.rb index 1281a518..07377cc6 100755 --- a/test/devise_test.rb +++ b/test/devise_test.rb @@ -67,18 +67,18 @@ class DeviseTest < ActiveSupport::TestCase end test 'add new module using the helper method' do - assert_nothing_raised(Exception) { Devise.add_module(:coconut) } + Devise.add_module(:coconut) assert_equal 1, Devise::ALL.select { |v| v == :coconut }.size refute Devise::STRATEGIES.include?(:coconut) refute defined?(Devise::Models::Coconut) Devise::ALL.delete(:coconut) - assert_nothing_raised(Exception) { Devise.add_module(:banana, strategy: :fruits) } + Devise.add_module(:banana, strategy: :fruits) assert_equal :fruits, Devise::STRATEGIES[:banana] Devise::ALL.delete(:banana) Devise::STRATEGIES.delete(:banana) - assert_nothing_raised(Exception) { Devise.add_module(:kivi, controller: :fruits) } + Devise.add_module(:kivi, controller: :fruits) assert_equal :fruits, Devise::CONTROLLERS[:kivi] Devise::ALL.delete(:kivi) Devise::CONTROLLERS.delete(:kivi) diff --git a/test/models_test.rb b/test/models_test.rb index 3b462afc..2dc3d9d8 100644 --- a/test/models_test.rb +++ b/test/models_test.rb @@ -112,7 +112,7 @@ class CheckFieldsTest < ActiveSupport::TestCase attr_accessor :encrypted_password, :email end - assert_nothing_raised Devise::Models::MissingAttribute do + assert_nothing_raised do Devise::Models.check_fields!(Player) end end