[1217] Change course endpoint to use include param

DFE-Digital · Apr 4, 2019 · c44454e · c44454e
1 parent eb71778
commit c44454e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/controllers/api/v2/courses_controller.rb b/app/controllers/api/v2/courses_controller.rb
@@ -12,7 +12,7 @@ def index
       end
 
       def show
-        render jsonapi: @course, include: [site_statuses: [:site]]
+        render jsonapi: @course, include: params[:include]
       end
 
       def sync_with_search_and_compare

diff --git a/spec/requests/api/v2/providers/courses_spec.rb b/spec/requests/api/v2/providers/courses_spec.rb
@@ -73,7 +73,7 @@
       before do
         get "/api/v2/providers/#{provider.provider_code.downcase}/courses/#{findable_open_course.course_code.downcase}",
             headers: { 'HTTP_AUTHORIZATION' => credentials },
-            params: { includes: "site_statuses" }
+            params: { include: 'site_statuses.site' }
       end
 
       it { should have_http_status(:success) }