diff options
| author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2019-08-20 17:42:45 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-08-20 17:42:45 +0100 |
| commit | 72bc285669f9ccf0e95bb43afcdaf052845a3431 (patch) | |
| tree | ebc95dabd0df652ce1e57d24ebf2686b44e1c88e /tests/appservice/test_scheduler.py | |
| parent | Avoid deep recursion in appservice recovery (#5885) (diff) | |
| parent | Refactor the Appservice scheduler code (diff) | |
| download | synapse-72bc285669f9ccf0e95bb43afcdaf052845a3431.tar.xz | |
Refactor the Appservice scheduler code (#5886)
Get rid of the labyrinthine `recoverer_fn` code, and clean up the startup code (it seemed to be previously inexplicably split between `ApplicationServiceScheduler.start` and `_Recoverer.start`). Add some docstrings too.
Diffstat (limited to 'tests/appservice/test_scheduler.py')
| -rw-r--r-- | tests/appservice/test_scheduler.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/appservice/test_scheduler.py b/tests/appservice/test_scheduler.py index 04b8c2c07c..52f89d3f83 100644 --- a/tests/appservice/test_scheduler.py +++ b/tests/appservice/test_scheduler.py @@ -37,11 +37,9 @@ class ApplicationServiceSchedulerTransactionCtrlTestCase(unittest.TestCase): self.recoverer = Mock() self.recoverer_fn = Mock(return_value=self.recoverer) self.txnctrl = _TransactionController( - clock=self.clock, - store=self.store, - as_api=self.as_api, - recoverer_fn=self.recoverer_fn, + clock=self.clock, store=self.store, as_api=self.as_api ) + self.txnctrl.RECOVERER_CLASS = self.recoverer_fn def test_single_service_up_txn_sent(self): # Test: The AS is up and the txn is successfully sent. |