diff options
author | Erik Johnston <erik@matrix.org> | 2021-02-24 13:23:18 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-02-24 13:23:18 +0000 |
commit | 292792194232e68eb68c5135bb59d037d38b870b (patch) | |
tree | 3951c198903b824b6db2cb05564ab3600eaad51f /tests/replication/tcp | |
parent | Refactor to ensure we call check_consistency (#9470) (diff) | |
download | synapse-292792194232e68eb68c5135bb59d037d38b870b.tar.xz |
Clean up `ShardedWorkerHandlingConfig` (#9466)
* Split ShardedWorkerHandlingConfig This is so that we have a type level understanding of when it is safe to call `get_instance(..)` (as opposed to `should_handle(..)`). * Remove special cases in ShardedWorkerHandlingConfig. `ShardedWorkerHandlingConfig` tried to handle the various different ways it was possible to configure federation senders and pushers. This led to special cases that weren't hit during testing. To fix this the handling of the different cases is moved from there and `generic_worker` into the worker config class. This allows us to have the logic in one place and allows the rest of the code to ignore the different cases.
Diffstat (limited to 'tests/replication/tcp')
-rw-r--r-- | tests/replication/tcp/streams/test_federation.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/replication/tcp/streams/test_federation.py b/tests/replication/tcp/streams/test_federation.py index 2babea4e3e..aa4bf1c7e3 100644 --- a/tests/replication/tcp/streams/test_federation.py +++ b/tests/replication/tcp/streams/test_federation.py @@ -24,7 +24,7 @@ class FederationStreamTestCase(BaseStreamTestCase): # enable federation sending on the worker config = super()._get_worker_hs_config() # TODO: make it so we don't need both of these - config["send_federation"] = True + config["send_federation"] = False config["worker_app"] = "synapse.app.federation_sender" return config |