summary refs log tree commit diff
path: root/tests/appservice
diff options
context:
space:
mode:
authorRichard van der Hoff <richard@matrix.org>2018-05-02 16:16:01 +0100
committerRichard van der Hoff <richard@matrix.org>2018-05-02 16:16:01 +0100
commit415c6b672ef08f4640cf48e80b62ca508e45279b (patch)
treed2e1598e8258c33bac7f370c3e703b7c14403a52 /tests/appservice
parentRemove spurious unittest.DEBUG (diff)
parentMerge pull request #3172 from matrix-org/rav/fix_test_logcontext_leaks (diff)
downloadsynapse-415c6b672ef08f4640cf48e80b62ca508e45279b.tar.xz
Merge branch 'develop' into rav/more_logcontext_leaks
Diffstat (limited to 'tests/appservice')
-rw-r--r--tests/appservice/test_scheduler.py11
1 files changed, 9 insertions, 2 deletions
diff --git a/tests/appservice/test_scheduler.py b/tests/appservice/test_scheduler.py
index e5a902f734..9181692771 100644
--- a/tests/appservice/test_scheduler.py
+++ b/tests/appservice/test_scheduler.py
@@ -17,6 +17,8 @@ from synapse.appservice.scheduler import (
     _ServiceQueuer, _TransactionController, _Recoverer
 )
 from twisted.internet import defer
+
+from synapse.util.logcontext import make_deferred_yieldable
 from ..utils import MockClock
 from mock import Mock
 from tests import unittest
@@ -204,7 +206,9 @@ class ApplicationServiceSchedulerQueuerTestCase(unittest.TestCase):
 
     def test_send_single_event_with_queue(self):
         d = defer.Deferred()
-        self.txn_ctrl.send = Mock(return_value=d)
+        self.txn_ctrl.send = Mock(
+            side_effect=lambda x, y: make_deferred_yieldable(d),
+        )
         service = Mock(id=4)
         event = Mock(event_id="first")
         event2 = Mock(event_id="second")
@@ -235,7 +239,10 @@ class ApplicationServiceSchedulerQueuerTestCase(unittest.TestCase):
         srv_2_event2 = Mock(event_id="srv2b")
 
         send_return_list = [srv_1_defer, srv_2_defer]
-        self.txn_ctrl.send = Mock(side_effect=lambda x, y: send_return_list.pop(0))
+
+        def do_send(x, y):
+            return make_deferred_yieldable(send_return_list.pop(0))
+        self.txn_ctrl.send = Mock(side_effect=do_send)
 
         # send events for different ASes and make sure they are sent
         self.queuer.enqueue(srv1, srv_1_event)