Merge pull request #8958 from balexand/strong_parameters_exception_handling

Strong parameters exception handling

1 files changed, 6 insertions, 9 deletions

diff --git a/actionpack/test/controller/required_params_test.rb b/actionpack/test/controller/required_params_test.rb
index 661bcb3945..2898adee72 100644
--- a/actionpack/test/controller/required_params_test.rb
+++ b/actionpack/test/controller/required_params_test.rb
@@ -11,20 +11,17 @@ class ActionControllerRequiredParamsTest < ActionController::TestCase
   tests BooksController
 
   test "missing required parameters will raise exception" do
-    post :create, { magazine: { name: "Mjallo!" } }
-    assert_response :bad_request
+    assert_raise (ActionController::ParameterMissing) do
+      post :create, { magazine: { name: "Mjallo!" } }
+    end
 
-    post :create, { book: { title: "Mjallo!" } }
-    assert_response :bad_request
+    assert_raise (ActionController::ParameterMissing) do
+      post :create, { book: { title: "Mjallo!" } }
+    end
   end
 
   test "required parameters that are present will not raise" do
     post :create, { book: { name: "Mjallo!" } }
     assert_response :ok
   end
-
-  test "missing parameters will be mentioned in the return" do
-    post :create, { magazine: { name: "Mjallo!" } }
-    assert_equal "Required parameter missing: book", response.body
-  end
 end