Merge pull request #2165 from bjfish/bjfish/initialize-custom-error-data

Initialize error data using constructors directly

Author: dblock

lib/grape/endpoint.rb

@@ -255,7 +255,7 @@ def run
           run_filters befores, :before
 
           if (allowed_methods = env[Grape::Env::GRAPE_ALLOWED_METHODS])
-            raise Grape::Exceptions::MethodNotAllowed, header.merge('Allow' => allowed_methods) unless options?
+            raise Grape::Exceptions::MethodNotAllowed.new(header.merge('Allow' => allowed_methods)) unless options?
             header 'Allow', allowed_methods
             response_object = ''
             status 204

lib/grape/parser/json.rb

@@ -8,7 +8,7 @@ def call(object, _env)
           ::Grape::Json.load(object)
         rescue ::Grape::Json::ParseError
           # handle JSON parsing errors via the rescue handlers or provide error message
-          raise Grape::Exceptions::InvalidMessageBody, 'application/json'
+          raise Grape::Exceptions::InvalidMessageBody.new('application/json')
         end
       end
     end

lib/grape/parser/xml.rb

@@ -8,7 +8,7 @@ def call(object, _env)
           ::Grape::Xml.parse(object)
         rescue ::Grape::Xml::ParseError
           # handle XML parsing errors via the rescue handlers or provide error message
-          raise Grape::Exceptions::InvalidMessageBody, 'application/xml'
+          raise Grape::Exceptions::InvalidMessageBody.new('application/xml')
         end
       end
     end

lib/grape/request.rb

@@ -16,7 +16,7 @@ def initialize(env, **options)
     def params
       @params ||= build_params
     rescue EOFError
-      raise Grape::Exceptions::EmptyMessageBody, content_type
+      raise Grape::Exceptions::EmptyMessageBody.new(content_type)
     end
 
     def headers

lib/grape/validations/validators/base.rb

@@ -55,7 +55,7 @@ def validate!(params)
           end
         end
 
-        raise Grape::Exceptions::ValidationArrayErrors, array_errors if array_errors.any?
+        raise Grape::Exceptions::ValidationArrayErrors.new(array_errors) if array_errors.any?
       end
 
       def self.convert_to_short_name(klass)

lib/grape/validations/validators/multiple_params_base.rb

@@ -16,7 +16,7 @@ def validate!(params)
           end
         end
 
-        raise Grape::Exceptions::ValidationArrayErrors, array_errors if array_errors.any?
+        raise Grape::Exceptions::ValidationArrayErrors.new(array_errors) if array_errors.any?
       end
 
       private