paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-10-23 04:52:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

push_notification: Remove redundant @activate_push_notification_service.

Browse Source 
In 'test_e2ee_push_notifications', the `SendPushNotificationTest` class
is already decorated with `@activate_push_notification_service()`,
so individual test methods do not need to repeat it.
This commit is contained in:
Prakhar Pratyush
2025-07-24 15:05:15 +05:30
committed by Tim Abbott
parent 6f8cb2b787
commit 945f27f099
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
zerver/tests/test_e2ee_push_notifications.py
View File

@@ -258,7 +258,6 @@ class SendPushNotificationTest(E2EEPushNotificationTestCase):
zerver_logger.output[1],
)
@activate_push_notification_service()
@responses.activate
@override_settings(ZILENCER_ENABLED=False)
def test_success_self_hosted(self, unused_mock: mock.MagicMock) -> None:
@@ -353,7 +352,6 @@ class SendPushNotificationTest(E2EEPushNotificationTestCase):
self.assertTrue(realm.push_notifications_enabled)
self.assertIsNone(realm.push_notifications_enabled_end_timestamp)
@activate_push_notification_service()
@responses.activate
@override_settings(ZILENCER_ENABLED=False)
def test_missing_remote_realm_error(self, unused_mock: mock.MagicMock) -> None:
@@ -408,7 +406,6 @@ class SendPushNotificationTest(E2EEPushNotificationTestCase):
self.assertFalse(realm.push_notifications_enabled)
self.assertIsNone(realm.push_notifications_enabled_end_timestamp)
@activate_push_notification_service()
@responses.activate
@override_settings(ZILENCER_ENABLED=False)
def test_no_plan_error(self, unused_mock: mock.MagicMock) -> None: