Skip to content
Browse files

Merge pull request #13819 from arthurnn/one_require_test

unify param.require tests
  • Loading branch information...
2 parents bc0820c + aae53d2 commit f1a0b15139a955cf67bc2fd2909840b32ce73485 @guilleiguaran guilleiguaran committed Jan 23, 2014
View
10 actionpack/test/controller/parameters/parameters_require_test.rb
@@ -1,10 +0,0 @@
-require 'abstract_unit'
-require 'action_controller/metal/strong_parameters'
-
-class ParametersRequireTest < ActiveSupport::TestCase
- test "required parameters must be present not merely not nil" do
- assert_raises(ActionController::ParameterMissing) do
- ActionController::Parameters.new(person: {}).require(:person)
- end
- end
-end
View
8 actionpack/test/controller/required_params_test.rb
@@ -25,3 +25,11 @@ class ActionControllerRequiredParamsTest < ActionController::TestCase
assert_response :ok
end
end
+
+class ParametersRequireTest < ActiveSupport::TestCase
+ test "required parameters must be present not merely not nil" do
+ assert_raises(ActionController::ParameterMissing) do
+ ActionController::Parameters.new(person: {}).require(:person)
+ end
+ end
+end

0 comments on commit f1a0b15

Please sign in to comment.
Something went wrong with that request. Please try again.