Skip to content

Commit

Permalink
Add failing test case for comment progress clobbering start_date
Browse files Browse the repository at this point in the history
  • Loading branch information
dato committed Jul 28, 2024
1 parent eef0551 commit 8122214
Showing 1 changed file with 35 additions and 0 deletions.
35 changes: 35 additions & 0 deletions bookwyrm/tests/views/test_status.py
Original file line number Diff line number Diff line change
@@ -1,9 +1,12 @@
""" test for app action functionality """
import json
from unittest import expectedFailure
from unittest.mock import patch
import dateutil
from django.core.exceptions import PermissionDenied
from django.test import TestCase, TransactionTestCase
from django.test.client import RequestFactory
from django.utils import timezone

from bookwyrm import forms, models, views
from bookwyrm.views.status import find_mentions, find_or_create_hashtags
Expand Down Expand Up @@ -167,6 +170,38 @@ def test_create_status_rating(self, *_):
self.assertEqual(status.rating, 4.0)
self.assertIsNone(status.edited_date)

@expectedFailure # https://github.com/bookwyrm-social/bookwyrm/issues/3164
def test_create_status_progress(self, *_):
"""create a status that updates a readthrough"""
start_date = timezone.make_aware(dateutil.parser.parse("2024-07-27"))
readthrough = models.ReadThrough.objects.create(
book=self.book, user=self.local_user, start_date=start_date
)

self.assertEqual(start_date, readthrough.start_date)
self.assertIsNone(readthrough.progress)

view = views.CreateStatus.as_view()
form = forms.CommentForm(
{
"progress": 1,
"progress_mode": "PG",
"content": "I started the book",
"id": readthrough.id,
"book": self.book.id,
"user": self.local_user.id,
"privacy": "public",
}
)
request = self.factory.post("", form.data)
request.user = self.local_user

view(request, "comment")
readthrough.refresh_from_db()

self.assertEqual(1, readthrough.progress)
self.assertEqual(start_date, readthrough.start_date)

def test_create_status_wrong_user(self, *_):
"""You can't compose statuses for someone else"""
view = views.CreateStatus.as_view()
Expand Down

0 comments on commit 8122214

Please sign in to comment.