Merge pull request #3184 from GerardPaligot/hotfix_create_mp

fix(api): Returns all info of pm when we create one.
zestedesavoir · Nov 12, 2015 · 91b2e08 · 91b2e08
2 parents a229312 + 8e9e118
commit 91b2e08
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/zds/mp/api/serializers.py b/zds/mp/api/serializers.py
@@ -42,8 +42,9 @@ class PrivateTopicCreateSerializer(serializers.ModelSerializer, TitleValidator,
 
     class Meta:
         model = PrivateTopic
-        fields = ('id', 'title', 'subtitle', 'participants', 'text')
-        read_only_fields = ('id',)
+        fields = ('id', 'title', 'subtitle', 'participants', 'text',
+                  'author', 'participants', 'last_message', 'pubdate')
+        read_only_fields = ('id', 'author', 'last_message', 'pubdate')
 
     def create(self, validated_data):
         # This hack is necessary because `text` isn't a field of PrivateTopic.

diff --git a/zds/mp/api/tests.py b/zds/mp/api/tests.py
@@ -248,6 +248,9 @@ def test_create_of_private_topics(self):
         self.assertEqual(response.data.get('subtitle'), private_topics[0].subtitle)
         self.assertEqual(response.data.get('participants')[0], private_topics[0].participants.all()[0].id)
         self.assertEqual(data.get('text'), private_topics[0].last_message.text)
+        self.assertEqual(response.data.get('author'), self.profile.user.id)
+        self.assertIsNotNone(response.data.get('last_message'))
+        self.assertIsNotNone(response.data.get('pubdate'))
 
     def test_create_of_private_topics_without_subtitle(self):
         """
@@ -267,6 +270,9 @@ def test_create_of_private_topics_without_subtitle(self):
         self.assertEqual(response.data.get('subtitle'), private_topics[0].subtitle)
         self.assertEqual(response.data.get('participants')[0], private_topics[0].participants.all()[0].id)
         self.assertEqual(data.get('text'), private_topics[0].last_message.text)
+        self.assertEqual(response.data.get('author'), self.profile.user.id)
+        self.assertIsNotNone(response.data.get('last_message'))
+        self.assertIsNotNone(response.data.get('pubdate'))
 
     def test_create_of_private_topics_without_title(self):
         """