ft-#64922186 answer reactions

answer/tests/conftest.py

@@ -1,5 +1,5 @@
 import pytest
-
+from comment.models import Comment
 from answer.models import Answers
 from meetup.models import Meeting
 from question.models import Question
@@ -39,3 +39,24 @@ def answered_question(staff1, question1, meetup1):
         meetup=meetup1,
         question=question1,
     )
+
+
+@pytest.mark.django_db
+@pytest.fixture
+def answer_comment(staff1, question1):
+    return Comment.objects.create(
+        is_answer=True,
+        created_by=staff1,
+        comment="test comment",
+        question=question1,
+    )
+
+
+@pytest.mark.django_db
+@pytest.fixture
+def just_comment(staff1, question1):
+    return Comment.objects.create(
+        created_by=staff1,
+        comment="test comment",
+        question=question1,
+    )

answer/tests/test_answers.py

@@ -153,3 +153,44 @@ def test_admin_user_cannot_add_a_duplicate_answer_to_a_question(
         "error": {"non_field_errors": ["You cannot add a duplicate Answer."]},
     }:
         raise AssertionError()
+
+
+def test_post_a_reaction(api_client, db, admin_user, answer_comment):
+    api_client.force_authenticate(user=admin_user)
+    url = reverse('reaction', kwargs={'comment_id': answer_comment.id})
+    data = {
+        "reaction": "my cool reaction"
+    }
+    response = api_client.post(url, data, format="json")
+    if not response.status_code == 201:
+        raise AssertionError()
+
+def test_post_wrong_reaction(api_client, db, admin_user, answer_comment):
+    api_client.force_authenticate(user=admin_user)
+    url = reverse('reaction', kwargs={'comment_id': answer_comment.id})
+    data = {
+        "itisjoel": "my cool reaction"
+    }
+    response = api_client.post(url, data, format="json")
+    if not response.status_code == 400:
+        raise AssertionError()
+
+def test_reaction_on_non_answer(api_client, db, admin_user, just_comment):
+    api_client.force_authenticate(user=admin_user)
+    url = reverse('reaction', kwargs={'comment_id': just_comment.id})
+    data = {
+        "reaction": "my cool reaction"
+    }
+    response = api_client.post(url, data, format="json")
+    if not response.status_code == 403:
+        raise AssertionError()
+
+def test_reaction_on_missing_comment(api_client, db, admin_user, just_comment):
+    api_client.force_authenticate(user=admin_user)
+    url = reverse('reaction', kwargs={'comment_id': 100})
+    data = {
+        "reaction": "my cool reaction"
+    }
+    response = api_client.post(url, data, format="json")
+    if not response.status_code == 404:
+        raise AssertionError()

comment/migrations/0002_comment_is_answer.py

@@ -0,0 +1,18 @@
+# Generated by Django 2.1.7 on 2019-03-28 12:06
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('comment', '0001_initial'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='comment',
+            name='is_answer',
+            field=models.BooleanField(default=False),
+        ),
+    ]

comment/migrations/0003_reaction.py

@@ -0,0 +1,22 @@
+# Generated by Django 2.1.7 on 2019-03-28 17:51
+
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('comment', '0002_comment_is_answer'),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name='Reaction',
+            fields=[
+                ('id', models.AutoField(auto_created=True, primary_key=True, serialize=False, verbose_name='ID')),
+                ('reaction', models.TextField()),
+                ('comment', models.ForeignKey(on_delete=django.db.models.deletion.DO_NOTHING, related_name='reactions', to='comment.Comment')),
+            ],
+        ),
+    ]

comment/models.py

@@ -18,3 +18,12 @@ class Comment(models.Model):
     def __str__(self):
         """Return a readable representation of the comment model instance."""
         return self.comment
+
+
+class Reaction(models.Model):
+    comment = models.ForeignKey(Comment, on_delete=models.DO_NOTHING,
+                                related_name='reactions')
+    reaction = models.TextField()
+
+    def __str__(self):
+        return (self.reaction, self.comment,)

comment/serializers.py

@@ -1,15 +1,30 @@
 from rest_framework import serializers
-from .models import Comment
+from .models import Comment, Reaction
+
+
+class ReactionsField(serializers.RelatedField):
+    @classmethod
+    def to_representation(cls, value, queryset=Reaction.objects.all()):
+        return value.reaction
 
 
 class CommentSerializer(serializers.ModelSerializer):
     """Map the comment model instance into JSON format."""
 
+    reactions = ReactionsField(many=True, read_only=True)
+
     created_by = serializers.ReadOnlyField(source='created_by.username')
     # question_name = serializers.ReadOnlyField(source='question.title')
 
     class Meta:
         """Map serializer fields to comment model fields."""
         model = Comment
         fields = "__all__"
-        # read_only_fields = ("created_by_name", "question_name", )
+        read_only_fields = ("is_answer", "question", )
+
+
+class ReactionSerializer(serializers.ModelSerializer):
+    class Meta:
+        model = Reaction
+        fields = "__all__"
+        read_only_fields = ("comment",)

comment/tests/test_answer.py

@@ -53,10 +53,26 @@ def test_succesful_comment_answer_toggle(self):
                               }
                       )
         self.client.force_authenticate(user=self.admin)
-        response = self.client.patch(url, format="json")
+        response = self.client.get(url, format="json")
 
         self.assertEqual(response.status_code, 200)
 
+    def test_missing_comment_toggle(self):
+        """
+        Ensure admin can toggle a comment to an answer
+        """
+
+        url = reverse("toggle_answer",
+                      kwargs={'meetup_id': self.meetup.id,
+                              'question_id': self.question.id,
+                              'pk': 100
+                              }
+                      )
+        self.client.force_authenticate(user=self.admin)
+        response = self.client.get(url, format="json")
+
+        self.assertEqual(response.status_code, 404)
+
     def test_non_admin_comment_answer_toggle(self):
         """
         Ensure non admin cannot toggle a comment to an answer
@@ -69,7 +85,7 @@ def test_non_admin_comment_answer_toggle(self):
                               }
                       )
         self.client.force_authenticate(user=self.user)
-        response = self.client.patch(url, format="json")
+        response = self.client.get(url, format="json")
 
         self.assertEqual(response.status_code, 403)
 
@@ -80,6 +96,6 @@ def test_non_existent_comment(self):
 
         url = f"/meetups/{self.meetup.id}/questions/{self.question.id}/comments/100/toggle_answer/"
         self.client.force_authenticate(user=self.admin)
-        response = self.client.patch(url, format="json")
+        response = self.client.get(url, format="json")
 
         self.assertEqual(response.status_code, 404)

comment/views.py

@@ -9,7 +9,7 @@
 from meetup.models import Meeting
 from question.models import Question
 from .models import Comment
-from .serializers import CommentSerializer
+from .serializers import CommentSerializer, ReactionSerializer
 
 
 class CommentList(APIView):
@@ -116,7 +116,7 @@ class ToggleAnswer(APIView):
 
     permission_classes = (IsAuthenticated,)
 
-    def patch(self, request, *args, **kwags):
+    def get(self, request, *args, **kwags):
 
         try:
             question_id = self.kwargs['question_id']
@@ -136,7 +136,7 @@ def patch(self, request, *args, **kwags):
                                                partial=True
                                                )
                 if serializer.is_valid():
-                    serializer.save()
+                    serializer.save(is_answer=True)
                     return Response(
                                 {
                                     "status": status.HTTP_200_OK,
@@ -290,3 +290,75 @@ def delete(self, request, pk, **kwargs):
             },
             status=status.HTTP_404_NOT_FOUND
         )
+
+# Add a reaction
+class AddReaction(APIView):
+    """
+    post: reaction
+    """
+
+    permission_classes = (IsAuthenticated,)
+    serializer_class = ReactionSerializer
+
+    @classmethod
+    # @swagger_auto_schema(
+    #     operation_description="Add a tag to a meetup",
+    #     operation_id="Add a tag to a meetup.",
+    #     request_body=MeetingTagSerializer,
+    #     responses={
+    #         201: MeetingTagSerializer(many=False),
+    #         401: "Unathorized Access",
+    #         403: "Tag is disabled",
+    #         404: "Tag Does not exist",
+    #         400: "Meet up does not exist or Tag already exists",
+    #     },
+    # )
+    def post(cls, request, comment_id):
+
+        try:
+            comment = Comment.objects.get(pk=comment_id)
+            serializer = ReactionSerializer(data=request.data)
+
+        except Exception:
+            return Response(
+                data={
+                    "status": status.HTTP_404_NOT_FOUND,
+                    "error": "Comment does not exist!",
+                },
+                status=status.HTTP_404_NOT_FOUND,
+            )
+
+        response = None
+        if not comment.is_answer:
+            response = Response(
+                data={
+                    "status": status.HTTP_403_FORBIDDEN,
+                    "error": "Sorry, reactions can only be made on answers.",
+                },
+                status=status.HTTP_403_FORBIDDEN,
+            )
+
+        elif serializer.is_valid():
+            serializer.save(comment_id=comment_id)
+
+            response = Response(
+                data={
+                    "status": status.HTTP_201_CREATED,
+                    "data": [
+                        {
+                            "success": "Reaction successfully added.",
+                        }
+                    ],
+                },
+                status=status.HTTP_201_CREATED,
+            )
+
+        else:
+            response = Response(
+                data={
+                    "status": status.HTTP_400_BAD_REQUEST,
+                    "detail": serializer.errors,
+                },
+                status=status.HTTP_400_BAD_REQUEST,
+            )
+        return response

questioner/settings.py

@@ -22,13 +22,9 @@
 SECRET_KEY = "n%fg(03shlp=ipqb_u%_o@=&7bhcwq8hpedk-sn)+5tnx6#*kb"
 
 # SECURITY WARNING: don't run with debug turned on in production!
-DEBUG = True
+DEBUG = False
 
-ALLOWED_HOSTS = [
-    "127.0.0.1",
-    "snaketech.herokuapp.com",
-    "questionerdojo.herokuapp.com",
-]
+ALLOWED_HOSTS = ["*", ]
 
 # Application definition
 

questioner/urls.py

@@ -23,6 +23,7 @@
 from rest_framework.urlpatterns import format_suffix_patterns
 
 from tag import views as tag_views
+from comment import views as comment_views
 from . import views
 
 
@@ -50,6 +51,8 @@
     path("meetups/", include("vote.urls")),
     path("meetups/", include("comment.urls")),
     path("tags/", tag_views.TagList.as_view(), name="tags"),
+    path("comments/<int:comment_id>/reactions",
+         comment_views.AddReaction.as_view(), name="reaction"),
     path("tags/<int:tag_id>", tag_views.ATag.as_view(), name="tag"),
     path("admin/", admin.site.urls),
     path(