diff --git a/bookwyrm/models/user.py b/bookwyrm/models/user.py index b87ffd7d..f697d474 100644 --- a/bookwyrm/models/user.py +++ b/bookwyrm/models/user.py @@ -405,7 +405,7 @@ def set_remote_server(user_id): user = User.objects.get(id=user_id) actor_parts = urlparse(user.remote_id) user.federated_server = get_or_create_remote_server(actor_parts.netloc) - user.save(broadcast=False) + user.save(broadcast=False, update_fields=["federated_server"]) if user.bookwyrm_user and user.outbox: get_remote_reviews.delay(user.outbox) diff --git a/bookwyrm/tests/models/test_user_model.py b/bookwyrm/tests/models/test_user_model.py index 0868bc4b..177b2ad6 100644 --- a/bookwyrm/tests/models/test_user_model.py +++ b/bookwyrm/tests/models/test_user_model.py @@ -155,7 +155,8 @@ class User(TestCase): self.assertIsNone(server.application_type) self.assertIsNone(server.application_version) - def test_delete_user(self): + @patch("bookwyrm.suggested_users.remove_user_task.delay") + def test_delete_user(self, _): """deactivate a user""" self.assertTrue(self.user.is_active) with patch( diff --git a/bookwyrm/tests/views/inbox/test_inbox_delete.py b/bookwyrm/tests/views/inbox/test_inbox_delete.py index cc3299aa..1566c05a 100644 --- a/bookwyrm/tests/views/inbox/test_inbox_delete.py +++ b/bookwyrm/tests/views/inbox/test_inbox_delete.py @@ -106,7 +106,8 @@ class InboxActivities(TestCase): self.assertEqual(models.Notification.objects.count(), 1) self.assertEqual(models.Notification.objects.get(), notif) - def test_delete_user(self): + @patch("bookwyrm.suggested_users.remove_user_task.delay") + def test_delete_user(self, _): """delete a user""" self.assertTrue(models.User.objects.get(username="rat@example.com").is_active) activity = { diff --git a/bookwyrm/tests/views/inbox/test_inbox_update.py b/bookwyrm/tests/views/inbox/test_inbox_update.py index 77c7ab76..4abb0fa0 100644 --- a/bookwyrm/tests/views/inbox/test_inbox_update.py +++ b/bookwyrm/tests/views/inbox/test_inbox_update.py @@ -80,7 +80,8 @@ class InboxUpdate(TestCase): self.assertEqual(book_list.description, "summary text") self.assertEqual(book_list.remote_id, "https://example.com/list/22") - def test_update_user(self): + @patch("bookwyrm.suggested_users.rerank_user_task.delay") + def test_update_user(self, _): """update an existing user""" models.UserFollows.objects.create( user_subject=self.local_user, diff --git a/bookwyrm/tests/views/test_feed.py b/bookwyrm/tests/views/test_feed.py index 406310e4..8a38b808 100644 --- a/bookwyrm/tests/views/test_feed.py +++ b/bookwyrm/tests/views/test_feed.py @@ -39,6 +39,7 @@ class FeedViews(TestCase): ) models.SiteSettings.objects.create() + @patch("bookwyrm.suggested_users.SuggestedUsers.get_suggestions") def test_feed(self, *_): """there are so many views, this just makes sure it LOADS""" view = views.Feed.as_view() diff --git a/bookwyrm/tests/views/test_get_started.py b/bookwyrm/tests/views/test_get_started.py index 31e5e86c..eb9d67b5 100644 --- a/bookwyrm/tests/views/test_get_started.py +++ b/bookwyrm/tests/views/test_get_started.py @@ -106,7 +106,8 @@ class GetStartedViews(TestCase): self.assertEqual(shelfbook.book, self.book) self.assertEqual(shelfbook.user, self.local_user) - def test_users_view(self): + @patch("bookwyrm.suggested_users.SuggestedUsers.get_suggestions") + def test_users_view(self, _): """there are so many views, this just makes sure it LOADS""" view = views.GetStartedUsers.as_view() request = self.factory.get("") @@ -118,7 +119,8 @@ class GetStartedViews(TestCase): result.render() self.assertEqual(result.status_code, 200) - def test_users_view_with_query(self): + @patch("bookwyrm.suggested_users.SuggestedUsers.get_suggestions") + def test_users_view_with_query(self, _): """there are so many views, this just makes sure it LOADS""" view = views.GetStartedUsers.as_view() request = self.factory.get("?query=rat")