get updates_location from course_location directly + update tests

STUD-1248

cms/djangoapps/contentstore/views/course.py

@@ -470,12 +470,7 @@ def course_info_update_handler(request, tag=None, package_id=None, branch=None,
     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)
+    updates_location = course_location.replace(category='course_info', name=block)
     if provided_id == '':
         provided_id = None
 

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

@@ -150,7 +150,7 @@ class CourseUpdateTest(CourseTestCase):
 
     def test_post_course_update(self):
         """
-        Test posting a course update through api on a newly created course updates location map with course_updates
+        Test that a user can successfully post on course updates of a course whose location in not in loc_mapper
         """
         # create a course via the view handler
         course_location = Location(['i4x', 'Org_1', 'Course_1', 'course', 'Run_1'])
@@ -174,7 +174,6 @@ class CourseUpdateTest(CourseTestCase):
             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')
@@ -185,3 +184,9 @@ class CourseUpdateTest(CourseTestCase):
 
         payload = json.loads(resp.content)
         self.assertHTMLEqual(payload['content'], content)
+
+        # now test that calling translate_location returns a locator whose block_id is 'updates'
+        updates_location = course_location.replace(category='course_info', name=block)
+        updates_locator = loc_mapper().translate_location(course_location.course_id, updates_location)
+        self.assertTrue(isinstance(updates_locator, BlockUsageLocator))
+        self.assertEqual(updates_locator.block_id, block)