diff options
author | Shay <hillerys@element.io> | 2023-06-09 00:00:46 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-06-09 09:00:46 +0200 |
commit | d84e66144dc12dacf71c987a2ba802dd59c0b68e (patch) | |
tree | de11408f278b7c1772c1061fd32d0c61151d21b2 /tests/http | |
parent | Merge branch 'master' into develop (diff) | |
download | synapse-d84e66144dc12dacf71c987a2ba802dd59c0b68e.tar.xz |
Allow for the configuration of max request retries and min/max retry delays in the matrix federation client (#12504)
Co-authored-by: Mathieu Velten <mathieuv@matrix.org> Co-authored-by: Erik Johnston <erik@matrix.org>
Diffstat (limited to 'tests/http')
-rw-r--r-- | tests/http/test_matrixfederationclient.py | 20 |
1 files changed, 19 insertions, 1 deletions
diff --git a/tests/http/test_matrixfederationclient.py b/tests/http/test_matrixfederationclient.py index 0dfc03ce50..8565f8ac64 100644 --- a/tests/http/test_matrixfederationclient.py +++ b/tests/http/test_matrixfederationclient.py @@ -40,7 +40,7 @@ from synapse.server import HomeServer from synapse.util import Clock from tests.server import FakeTransport -from tests.unittest import HomeserverTestCase +from tests.unittest import HomeserverTestCase, override_config def check_logcontext(context: LoggingContextOrSentinel) -> None: @@ -640,3 +640,21 @@ class FederationClientTests(HomeserverTestCase): self.cl.build_auth_headers( b"", b"GET", b"https://example.com", destination_is=b"" ) + + @override_config( + { + "federation": { + "client_timeout": 180, + "max_long_retry_delay": 100, + "max_short_retry_delay": 7, + "max_long_retries": 20, + "max_short_retries": 5, + } + } + ) + def test_configurable_retry_and_delay_values(self) -> None: + self.assertEqual(self.cl.default_timeout, 180) + self.assertEqual(self.cl.max_long_retry_delay, 100) + self.assertEqual(self.cl.max_short_retry_delay, 7) + self.assertEqual(self.cl.max_long_retries, 20) + self.assertEqual(self.cl.max_short_retries, 5) |