paulmataruso/zulip
1
0
Fork 0
mirror of https://github.com/zulip/zulip.git synced 2025-11-04 14:03:30 +00:00
Code Issues Packages Projects Releases Wiki Activity

test-subs: Fix comment in test_notification_bot_dm_on_subscription.

Browse Source
This commit is contained in:
Lauryn Menard
2025-09-05 19:20:01 +02:00
committed by Tim Abbott
parent 7e43ee625c
commit 14581de0de
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
zerver/tests/test_subs.py
View File

@@ -5119,9 +5119,10 @@ class SubscriptionAPITest(ZulipTestCase):
) )
self.assertEqual(announcement_channel_message.count(), 0) self.assertEqual(announcement_channel_message.count(), 0)
# When subscribing to an already existing channel, if the number of new # The max number of new subscriptions limit only impacts
# subscriptions exceeds the limit, no DMs are sent, but an announcement # DM notifications, so for a newly created channel, we
# message and new channel message should be sent. # still expect an announcement message and new channel
# message (and no DM notifications).
now = timezone_now() now = timezone_now()
with self.settings(MAX_BULK_NEW_SUBSCRIPTION_MESSAGES=5): with self.settings(MAX_BULK_NEW_SUBSCRIPTION_MESSAGES=5):
response = self.subscribe_via_post( response = self.subscribe_via_post(