Add support for status to the error serializer

Author: johvet

lib/jsonapi/active_model_error_serializer.rb

@@ -3,12 +3,34 @@
 module JSONAPI
   # [ActiveModel::Errors] serializer
   class ActiveModelErrorSerializer < ErrorSerializer
-    attribute :status do
-      '422'
+    class << self
+      ##
+      # Get the status code to render for the serializer, considering an eventual
+      # status provided through the serializer parameters
+      #
+      # @param params [Hash]
+      #     The serializer parameters
+      #
+      # @return [Integer]
+      #     The status code to use
+      def status_code(params)
+        case params[:status]
+        when Symbol
+          Rack::Utils::SYMBOL_TO_STATUS_CODE[params[:status]]
+        when Integer
+          params[:status]
+        else
+          422
+        end
+      end
+    end
+
+    attribute :status do |_, params|
+      status_code(params).to_s
     end
 
-    attribute :title do
-      Rack::Utils::HTTP_STATUS_CODES[422]
+    attribute :title do |_, params|
+      Rack::Utils::HTTP_STATUS_CODES[status_code(params)]
     end
 
     attribute :code do |object|

lib/jsonapi/rails.rb

@@ -79,7 +79,7 @@ def self.add_errors_renderer!
 
         JSONAPI::Rails.serializer_to_json(
           JSONAPI::ActiveModelErrorSerializer.new(
-            errors, params: { model: model, model_serializer: model_serializer }
+            errors, params: { model: model, model_serializer: model_serializer, status: options[:status] }
           )
         )
       end

spec/dummy.rb

@@ -45,6 +45,7 @@ class Note < ActiveRecord::Base
   validates_numericality_of :quantity, less_than: 100, if: :quantity?
   validate :title_check
   belongs_to :user, required: true
+  before_destroy :deletable?
 
   def self.ransackable_associations(auth_object = nil)
     %w(user)
@@ -58,6 +59,13 @@ def self.ransackable_attributes(auth_object = nil)
   def title_check
     errors.add(:base, :invalid) if title == 'n/a'
   end
+
+  def deletable?
+    return true unless title == 'Lovely'
+
+    errors.add(:base, "Can't delete lovely notes")
+    throw :abort
+  end
 end
 
 class CustomNoteSerializer
@@ -90,7 +98,7 @@ class Dummy < Rails::Application
   routes.draw do
     scope defaults: { format: :jsonapi } do
       resources :users, only: [:index]
-      resources :notes, only: [:update]
+      resources :notes, only: [:update, :destroy]
     end
   end
 end
@@ -157,6 +165,15 @@ def update
     end
   end
 
+  def destroy
+    note = Note.find(params[:id])
+    if note.destroy
+      head :no_content
+    else
+      render jsonapi_errors: note.errors, status: :conflict
+    end
+  end
+
   private
   def render_jsonapi_internal_server_error(exception)
     Rails.logger.error(exception)

spec/errors_spec.rb

@@ -191,4 +191,41 @@
       end
     end
   end
+
+  describe 'DELETE /nodes/:id' do
+    let(:note)    { create_note }
+    let(:note_id) { note.id }
+    let(:user)    { note.user }
+    let(:user_id) { user.id }
+
+    context 'with a random note' do
+      before { delete(note_path(note_id), headers: jsonapi_headers) }
+
+      it { expect(response).to have_http_status(:no_content) }
+    end
+
+    context 'with a lovely note' do
+      let(:errors) do
+        {
+          'errors' => [
+            {
+              'code'   => 'cant_delete_lovely_notes',
+              'detail' => "Can't delete lovely notes",
+              'source' => { 'pointer' => '/data' },
+              'status' => '409',
+              'title'  => 'Conflict'
+            }
+          ]
+        }
+      end
+
+      before do
+        note.update(title: 'Lovely')
+        delete(note_path(note_id), headers: jsonapi_headers)
+      end
+
+      it { expect(response).to have_http_status(:conflict) }
+      it { expect(response_json).to match(errors) }
+    end
+  end
 end