Merge pull request #2 from basit3407/fix-list-chapter-reciters-api

Fix eager loading for chapter reciters

Author: basit3407

app/presenters/audio/recitation_presenter.rb

@@ -9,8 +9,8 @@ class Audio::RecitationPresenter < BasePresenter
 
   def recitations
     relation = Audio::Recitation
-                 .includes(:recitation_style, :qirat_type, reciter: :translated_name)
-                 .eager_load(reciter: :translated_name)
+                 .includes(:translated_name, :recitation_style, :qirat_type, reciter: :translated_name)
+                 .eager_load(:translated_name, reciter: :translated_name)
                  .order('priority ASC, language_priority DESC')
 
     eager_load_translated_name filter_recitations(relation)

spec/apis/v4/chapter_reciters_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe 'Chapter reciters endpoint', type: :api do
+  context 'when no recitations exist' do
+    it 'returns an empty list' do
+      get '/api/v4/resources/chapter_reciters', chapter_id: 1
+      expect(last_response.status).to eq(200)
+      expect(Oj.load(last_response.body)['reciters']).to eq([])
+    end
+  end
+end