summary refs log tree commit diff
diff options
context:
space:
mode:
authorRichard van der Hoff <richard@matrix.org>2019-10-24 11:13:48 +0100
committerRichard van der Hoff <richard@matrix.org>2019-10-24 11:13:48 +0100
commit04ec48283ee69b20ff04d18a548ba2e04afd116f (patch)
tree197c25335435a8e62b945313ed4c5c53bd2ead4f
parentAddress review comments and final cleanups (diff)
downloadsynapse-neilj/disable-mau-alerting-for-small-instances.tar.xz
-rw-r--r--synapse/server_notices/resource_limits_server_notices.py4
-rw-r--r--tests/server_notices/test_resource_limits_server_notices.py2
2 files changed, 3 insertions, 3 deletions
diff --git a/synapse/server_notices/resource_limits_server_notices.py b/synapse/server_notices/resource_limits_server_notices.py

index 88c638add4..c0e7f475c9 100644 --- a/synapse/server_notices/resource_limits_server_notices.py +++ b/synapse/server_notices/resource_limits_server_notices.py
@@ -113,10 +113,10 @@ class ResourceLimitsServerNotices(object): self._remove_limit_block_notification(user_id, ref_events) return - if currently_blocked and not limit_type: + if currently_blocked and not limit_msg: # Room is notifying of a block, when it ought not to be. yield self._remove_limit_block_notification(user_id, ref_events) - elif not currently_blocked and limit_type: + elif not currently_blocked and limit_msg: # Room is not notifying of a block, when it ought to be. yield self._apply_limit_block_notification( user_id, limit_msg, limit_type diff --git a/tests/server_notices/test_resource_limits_server_notices.py b/tests/server_notices/test_resource_limits_server_notices.py
index b5085944e5..eb540e34f6 100644 --- a/tests/server_notices/test_resource_limits_server_notices.py +++ b/tests/server_notices/test_resource_limits_server_notices.py
@@ -133,7 +133,7 @@ class TestResourceLimitsServerNotices(unittest.HomeserverTestCase): self.get_success(self._rlsn.maybe_send_server_notice_to_user(self.user_id)) # Would be better to check contents, but 2 calls == set blocking event - self.assertTrue(self._send_notice.call_count == 2) + self.assertEqual(self._send_notice.call_count, 2) def test_maybe_send_server_notice_to_user_add_blocked_notice_noop(self): """