diff --git a/lms/djangoapps/api_manager/groups_views.py b/lms/djangoapps/api_manager/groups_views.py index 70f8b81e5860..0ce739d06ac2 100644 --- a/lms/djangoapps/api_manager/groups_views.py +++ b/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: diff --git a/lms/djangoapps/api_manager/tests/test_groups_views.py b/lms/djangoapps/api_manager/tests/test_groups_views.py index a8740de9a856..2b55f8d0b3d3 100644 --- a/lms/djangoapps/api_manager/tests/test_groups_views.py +++ b/lms/djangoapps/api_manager/tests/test_groups_views.py @@ -574,7 +574,8 @@ def test_group_courses_list_post(self): 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}