Merge pull request #20126 from edx/depr/mobile-video-outline-api

DEPR-4: Remove Mobile Video Outlines API

common/lib/xmodule/xmodule/modulestore/tests/test_mongo_call_count.py

@@ -164,8 +164,6 @@ def test_number_mongo_calls(self, store_builder, depth, lazy, access_all_block_f
         with store_builder.build(request_cache=request_cache) as (content_store, modulestore):
             course_key = self._import_course(content_store, modulestore)
 
-            # Course traversal modeled after the traversal done here:
-            # lms/djangoapps/mobile_api/video_outlines/serializers.py:BlockOutline
             # Starting at the root course block, do a breadth-first traversal using
             # get_children() to retrieve each block's children.
             with check_mongo_calls(num_mongo_calls):

lms/djangoapps/mobile_api/urls.py

@@ -9,6 +9,5 @@
 urlpatterns = [
     url(r'^users/', include('mobile_api.users.urls')),
     url(r'^my_user_info', my_user_info, name='user-info'),
-    url(r'^video_outlines/', include('mobile_api.video_outlines.urls')),
     url(r'^course_info/', include('mobile_api.course_info.urls')),
 ]

lms/djangoapps/mobile_api/users/serializers.py

@@ -73,11 +73,10 @@ def to_representation(self, course_overview):
                 request=request,
             ) if course_overview.is_discussion_tab_enabled() else None,
 
-            'video_outline': reverse(
-                'video-summary-list',
-                kwargs={'api_version': api_version, 'course_id': course_id},
-                request=request,
-            ),
+            # This is an old API that was removed as part of DEPR-4. We keep the
+            # field present in case API parsers expect it, but this API is now
+            # removed.
+            'video_outline': None,
         }
 
 

lms/djangoapps/mobile_api/users/tests.py

@@ -111,7 +111,6 @@ def verify_success(self, response):
         self.assertIn('courses/{}/about'.format(self.course.id), found_course['course_about'])
         self.assertIn('course_info/{}/updates'.format(self.course.id), found_course['course_updates'])
         self.assertIn('course_info/{}/handouts'.format(self.course.id), found_course['course_handouts'])
-        self.assertIn('video_outlines/courses/{}'.format(self.course.id), found_course['video_outline'])
         self.assertEqual(found_course['id'], unicode(self.course.id))
         self.assertEqual(courses[0]['mode'], CourseMode.DEFAULT_MODE_SLUG)
         self.assertEqual(courses[0]['course']['subscription_id'], self.course.clean_id(padding_char='_'))