add course updates location to loc_mapper on accessing course_info_update_handler

STUD-1248

cms/djangoapps/contentstore/views/course.py

@@ -466,7 +466,15 @@ def course_info_update_handler(request, tag=None, package_id=None, branch=None,
     """
     if 'application/json' not in request.META.get('HTTP_ACCEPT', 'application/json'):
         return HttpResponseBadRequest("Only supports json requests")
-    updates_locator = BlockUsageLocator(package_id=package_id, branch=branch, version_guid=version_guid, block_id=block)
+
+    course_location = loc_mapper().translate_locator_to_location(
+        CourseLocator(package_id=package_id), get_course=True
+    )
+    # add location to loc_mapper
+    updates_locator = loc_mapper().translate_location(
+        course_location.course_id, course_location.replace(category='course_info', name='updates'),
+        False, True
+    )
     updates_location = loc_mapper().translate_locator_to_location(updates_locator)
     if provided_id == '':
         provided_id = None

cms/djangoapps/contentstore/views/tests/test_course_updates.py

-'''unit tests for course_info views and models.'''
-from contentstore.tests.test_course_settings import CourseTestCase
+"""
+unit tests for course_info views and models.
+"""
 import json
+
+from contentstore.tests.test_course_settings import CourseTestCase
+from xmodule.modulestore import Location
 from xmodule.modulestore.django import modulestore, loc_mapper
+from xmodule.modulestore.locator import BlockUsageLocator
 
 
 class CourseUpdateTest(CourseTestCase):
@@ -142,3 +147,41 @@ class CourseUpdateTest(CourseTestCase):
         resp = self.client.get_json(course_update_url)
         payload = json.loads(resp.content)
         self.assertTrue(len(payload) == 2)
+
+    def test_post_course_update(self):
+        """
+        Test posting a course update through api on a newly created course updates location map with course_updates
+        """
+        # create a course via the view handler
+        course_location = Location(['i4x', 'Org_1', 'Course_1', 'course', 'Run_1'])
+        course_locator = loc_mapper().translate_location(
+            course_location.course_id, course_location, False, True
+        )
+        self.client.ajax_post(
+            course_locator.url_reverse('course'),
+            {
+                'org': course_location.org,
+                'number': course_location.course,
+                'display_name': 'test course',
+                'run': course_location.name,
+            }
+        )
+
+        branch = u'draft'
+        version = None
+        block = u'updates'
+        updates_locator = BlockUsageLocator(
+            package_id=course_location.course_id.replace('/', '.'), branch=branch, version_guid=version, block_id=block
+        )
+
+        # test that user can successfully post on course updates whose location in not added in loc_mapper
+        content = u"Sample update"
+        payload = {'content': content, 'date': 'January 8, 2013'}
+        course_update_url = updates_locator.url_reverse('course_info_update')
+        resp = self.client.ajax_post(course_update_url, payload)
+
+        # check that response status is 200 not 400
+        self.assertEqual(resp.status_code, 200)
+
+        payload = json.loads(resp.content)
+        self.assertHTMLEqual(payload['content'], content)