Merge pull request #21 from edx-solutions/api

fix broken unit test

lms/djangoapps/api_manager/groups_views.py

@@ -131,7 +131,7 @@ def group_detail(request, group_id):
         # update GroupProfile data
 
         group_type = request.DATA.get('group_type')
-        data = request.DATA.get('data')
+        data = json.dumps(request.DATA.get('data')) if request.DATA.get('data') else None
 
         if not group_type and not data:
             return Response({}, status.HTTP_400_BAD_REQUEST)

lms/djangoapps/api_manager/tests/test_groups_views.py

@@ -53,7 +53,7 @@ class GroupsApiTests(TestCase):
             'Content-Type': 'application/json',
             'X-Edx-Api-Key': str(TEST_API_KEY),
         }
-        response = self.client.post(uri, headers=headers, data=data)
+        response = self.client.post(uri, headers=headers, data=json.dumps(data), content_type='application/json')
         return response
 
     def do_get(self, uri):
@@ -108,10 +108,6 @@ class GroupsApiTests(TestCase):
         self.assertEqual(response.data['name'], self.test_group_name)
 
     def test_group_list_get_with_profile(self):
-        # skip test. Seems like the Django test client is not serializing the nested JSON
-        # post arguments. Need to investigate
-        return
-
         data = {
             'name': self.test_group_name,
             'group_type': 'series',