Merge pull request #56 from andela/bg-fix-rating-163383181

163383181 Implement Bug Fix User can rate an article

authors/apps/articles/tests/test_rate_article.py

@@ -40,7 +40,7 @@ def test_rate_article_author(self):
             format='json')
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
         self.assertIn("Author can not rate their own article",
-                      response.data.get('errors'))
+                      response.data.get('message'))
 
     # integration tests for all modules under rating articles start here
 

authors/apps/articles/views.py

@@ -244,25 +244,37 @@ def post(self, request, slug):
         article = get_single_article_using_slug(slug)
         user = request.user
 
+        serializer = self.serializer_class(data=request.data)
+        serializer.is_valid(raise_exception=True)
+
         rate = request.data.get('rate_score', {})
         if rate > 5 or rate < 1:
             return Response({
                 'errors': 'Rate score should be between 1 and 5'},
                 status=status.HTTP_400_BAD_REQUEST)
 
         if article.author.pk == user.pk:
-            return Response({
-                'errors': 'Author can not rate their own article'},
-                status=status.HTTP_403_FORBIDDEN)
-
-        serializer = self.serializer_class(data=request.data)
-        serializer.is_valid(raise_exception=True)
+            message = 'Author can not rate their own article'
+            data = {
+                "title": article.title,
+                "author": article.author.username,
+                "average_ratings": article.average_ratings
+            }
+            return Response({'articles': data,
+                             'message': message},
+                            status=status.HTTP_403_FORBIDDEN)
 
         try:
             Rating.objects.get(user=user.pk, article_id=article.pk)
-            return Response(
-                {'message': 'You already rated this article'},
-                status=status.HTTP_422_UNPROCESSABLE_ENTITY)
+            message = 'You already rated this article'
+            data = {
+                "title": article.title,
+                "author": article.author.username,
+                "average_ratings": article.average_ratings
+            }
+            return Response({'articles': data,
+                             'message': message},
+                            status=status.HTTP_422_UNPROCESSABLE_ENTITY)
         except Rating.DoesNotExist:
             serializer.save(user=user, article=article)
             rate_data = serializer.data