Merge 25e3095 into 350ddf7

andela · Dec 21, 2018 · ab3155b · ab3155b
2 parents 350ddf7 + 25e3095
commit ab3155b
Showing 1 changed file with 32 additions and 1 deletion.
diff --git a/authors/apps/ratings/tests/test_ratings.py b/authors/apps/ratings/tests/test_ratings.py
@@ -1,6 +1,7 @@
 from rest_framework.reverse import reverse
 from rest_framework import status
 import logging
+import json
 
 # local imports
 from .test_base import TestBase
@@ -20,6 +21,10 @@ def test_post_rating(self):
             data=self.rating,
             format='json')
         self.assertEqual(response.status_code, status.HTTP_201_CREATED)
+        self.assertEqual(
+            json.loads(
+                response.content.decode('utf-8'))['message'],
+            "Rating added successfully")
 
     def test_post_bad_rating(self):
         self.client.credentials(HTTP_AUTHORIZATION='Token ' + self.rater_token)
@@ -31,6 +36,10 @@ def test_post_bad_rating(self):
             data=self.bad_rating,
             format='json')
         self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+        self.assertEqual(
+            json.loads(
+                response.content.decode('utf-8'))['errors']['rating'],
+            ['"10" is not a valid choice.'])
 
     def test_article_author_cannot_rate(self):
         self.client.credentials(
@@ -44,6 +53,10 @@ def test_article_author_cannot_rate(self):
             data=self.rating,
             format='json')
         self.assertEqual(response.status_code, status.HTTP_403_FORBIDDEN)
+        self.assertEqual(
+            json.loads(
+                response.content.decode('utf-8'))['message'],
+            "You cannot rate your own article")
 
     def test_get_average_rating(self):
         self.client.credentials(
@@ -56,8 +69,11 @@ def test_get_average_rating(self):
                     "slug": self.slug
                 }),
             format='json')
-        logger.error(response.content)
         self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual(
+            json.loads(
+                response.content.decode('utf-8'))['rating'],
+            5.0)
 
     def test_delete_rating(self):
         self.client.credentials(HTTP_AUTHORIZATION='Token ' + self.rater_token)
@@ -69,7 +85,12 @@ def test_delete_rating(self):
                     "id": self.id
                 }),
             format='json')
+        logger.error(response.content)
         self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.assertEqual(
+            json.loads(
+                response.content.decode('utf-8'))['message'],
+            "Rating removed successfully")
 
     def test_article_not_found_rate(self):
         self.client.credentials(HTTP_AUTHORIZATION='Token ' + self.rater_token)
@@ -80,7 +101,12 @@ def test_article_not_found_rate(self):
                     "slug": "not-found"}),
             data=self.rating,
             format='json')
+
         self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
+        self.assertEqual(
+            json.loads(
+                response.content.decode('utf-8'))['detail'],
+            "Article Not found")
 
     def test_delete_rating_not_found(self):
         self.client.credentials(HTTP_AUTHORIZATION='Token ' + self.rater_token)
@@ -92,4 +118,9 @@ def test_delete_rating_not_found(self):
                     "id": 12
                 }),
             format='json')
+        logger.error(response.content)
         self.assertEqual(response.status_code, status.HTTP_404_NOT_FOUND)
+        self.assertEqual(
+            json.loads(
+                response.content.decode('utf-8'))['detail'],
+            "Rating Not found")