mirror of
https://github.com/bookwyrm-social/bookwyrm.git
synced 2024-12-29 19:40:32 +00:00
Adds tests for boosts in activitystreams
This commit is contained in:
parent
f284eebece
commit
9c21f4d8e6
2 changed files with 65 additions and 1 deletions
|
@ -273,7 +273,7 @@ def add_status_on_create(sender, instance, created, *args, **kwargs):
|
||||||
created_date__lt=instance.created_date,
|
created_date__lt=instance.created_date,
|
||||||
)
|
)
|
||||||
for stream in streams.values():
|
for stream in streams.values():
|
||||||
audience = streams.get_stores_for_object(instance)
|
audience = stream.get_stores_for_object(instance)
|
||||||
stream.remove_object_from_related_stores(boosted, stores=audience)
|
stream.remove_object_from_related_stores(boosted, stores=audience)
|
||||||
for status in old_versions:
|
for status in old_versions:
|
||||||
stream.remove_object_from_related_stores(status, stores=audience)
|
stream.remove_object_from_related_stores(status, stores=audience)
|
||||||
|
|
|
@ -286,3 +286,67 @@ class Activitystreams(TestCase):
|
||||||
# yes book, yes audience
|
# yes book, yes audience
|
||||||
result = activitystreams.BooksStream().get_statuses_for_user(self.local_user)
|
result = activitystreams.BooksStream().get_statuses_for_user(self.local_user)
|
||||||
self.assertEqual(list(result), [status])
|
self.assertEqual(list(result), [status])
|
||||||
|
|
||||||
|
@patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
|
||||||
|
@patch("bookwyrm.activitystreams.BooksStream.remove_object_from_related_stores")
|
||||||
|
def test_boost_to_another_timeline(self, *_):
|
||||||
|
"""add a boost and deduplicate the boosted status on the timeline"""
|
||||||
|
status = models.Status.objects.create(user=self.local_user, content="hi")
|
||||||
|
boost = models.Boost.objects.create(
|
||||||
|
boosted_status=status,
|
||||||
|
user=self.another_user,
|
||||||
|
)
|
||||||
|
with patch(
|
||||||
|
"bookwyrm.activitystreams.HomeStream.remove_object_from_related_stores"
|
||||||
|
) as mock:
|
||||||
|
activitystreams.add_status_on_create(models.Boost, boost, True)
|
||||||
|
self.assertTrue(mock.called)
|
||||||
|
call_args = mock.call_args
|
||||||
|
self.assertEqual(call_args[0][0], status)
|
||||||
|
self.assertEqual(
|
||||||
|
call_args[1]["stores"], ["{:d}-home".format(self.another_user.id)]
|
||||||
|
)
|
||||||
|
|
||||||
|
@patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
|
||||||
|
@patch("bookwyrm.activitystreams.BooksStream.remove_object_from_related_stores")
|
||||||
|
def test_boost_to_following_timeline(self, *_):
|
||||||
|
"""add a boost and deduplicate the boosted status on the timeline"""
|
||||||
|
self.local_user.following.add(self.another_user)
|
||||||
|
status = models.Status.objects.create(user=self.local_user, content="hi")
|
||||||
|
boost = models.Boost.objects.create(
|
||||||
|
boosted_status=status,
|
||||||
|
user=self.another_user,
|
||||||
|
)
|
||||||
|
with patch(
|
||||||
|
"bookwyrm.activitystreams.HomeStream.remove_object_from_related_stores"
|
||||||
|
) as mock:
|
||||||
|
activitystreams.add_status_on_create(models.Boost, boost, True)
|
||||||
|
self.assertTrue(mock.called)
|
||||||
|
call_args = mock.call_args
|
||||||
|
self.assertEqual(call_args[0][0], status)
|
||||||
|
self.assertTrue(
|
||||||
|
"{:d}-home".format(self.another_user.id) in call_args[1]["stores"]
|
||||||
|
)
|
||||||
|
self.assertTrue(
|
||||||
|
"{:d}-home".format(self.local_user.id) in call_args[1]["stores"]
|
||||||
|
)
|
||||||
|
|
||||||
|
@patch("bookwyrm.activitystreams.LocalStream.remove_object_from_related_stores")
|
||||||
|
@patch("bookwyrm.activitystreams.BooksStream.remove_object_from_related_stores")
|
||||||
|
def test_boost_to_same_timeline(self, *_):
|
||||||
|
"""add a boost and deduplicate the boosted status on the timeline"""
|
||||||
|
status = models.Status.objects.create(user=self.local_user, content="hi")
|
||||||
|
boost = models.Boost.objects.create(
|
||||||
|
boosted_status=status,
|
||||||
|
user=self.local_user,
|
||||||
|
)
|
||||||
|
with patch(
|
||||||
|
"bookwyrm.activitystreams.HomeStream.remove_object_from_related_stores"
|
||||||
|
) as mock:
|
||||||
|
activitystreams.add_status_on_create(models.Boost, boost, True)
|
||||||
|
self.assertTrue(mock.called)
|
||||||
|
call_args = mock.call_args
|
||||||
|
self.assertEqual(call_args[0][0], status)
|
||||||
|
self.assertEqual(
|
||||||
|
call_args[1]["stores"], ["{:d}-home".format(self.local_user.id)]
|
||||||
|
)
|
||||||
|
|
Loading…
Reference in a new issue