Merge pull request #16 from edx-solutions/api

return the display names when listing all courses in a group (program)

lms/djangoapps/api_manager/groups_views.py

@@ -342,18 +342,24 @@ def group_courses_list(request, group_id):
     except ObjectDoesNotExist:
         return Response({}, status.HTTP_404_NOT_FOUND)
 
+    store = modulestore()
+
     if request.method == 'GET':
         members = CourseGroupRelationship.objects.filter(group=existing_group)
         response_data['courses'] = []
         for member in members:
-            response_data['courses'].append(member.course_id)
+            course = store.get_course(member.course_id)
+            course_data = {
+                'course_id': member.course_id,
+                'display_name': course.display_name
+            }
+            response_data['courses'].append(course_data)
         response_status = status.HTTP_200_OK
     else:
         course_id = request.DATA['course_id']
 
         base_uri = _generate_base_uri(request)
         response_data['uri'] = '{}/{}'.format(base_uri, course_id)
-        store = modulestore()
 
         existing_course = store.get_course(course_id)
         if not existing_course:

lms/djangoapps/api_manager/tests/test_groups_views.py

@@ -574,7 +574,8 @@ class GroupsApiTests(TestCase):
         response = self.do_get(test_uri)
         self.assertEqual(response.status_code, 200)
         self.assertEqual(len(response.data['courses']), 1)
-        self.assertEqual(response.data['courses'][0], self.test_course_id)
+        self.assertEqual(response.data['courses'][0]['course_id'], self.test_course_id)
+        self.assertEqual(response.data['courses'][0]['display_name'], self.course.display_name)
 
     def test_group_courses_list_post_duplicate(self):
         data = {'name': self.test_group_name}