summary refs log tree commit diff
path: root/tests/util/test_async_helpers.py
diff options
context:
space:
mode:
authorErik Johnston <erikj@element.io>2024-11-07 16:22:09 +0000
committerGitHub <noreply@github.com>2024-11-07 16:22:09 +0000
commitc7a1d0aa1afc8349dd7839f5ba6ea6a0a2830013 (patch)
tree5487e59e46e45f75e1b18afe8b49d23d7df14825 /tests/util/test_async_helpers.py
parentSwitch portdb CI to python 3.13, pg 17 (#17909) (diff)
downloadsynapse-c7a1d0aa1afc8349dd7839f5ba6ea6a0a2830013.tar.xz
Fix Twisted tests with latest release (#17911)
c.f. #17906 and #17907
Diffstat (limited to '')
-rw-r--r--tests/util/test_async_helpers.py15
1 files changed, 11 insertions, 4 deletions
diff --git a/tests/util/test_async_helpers.py b/tests/util/test_async_helpers.py

index 350a2b7c8c..cfd2882410 100644 --- a/tests/util/test_async_helpers.py +++ b/tests/util/test_async_helpers.py
@@ -320,12 +320,19 @@ class ConcurrentlyExecuteTest(TestCase): await concurrently_execute(callback, [1], 2) except _TestException as e: tb = traceback.extract_tb(e.__traceback__) - # we expect to see "caller", "concurrently_execute", "callback", - # and some magic from inside ensureDeferred that happens when .fail - # is called. + + # Remove twisted internals from the stack, as we don't care + # about the precise details. + tb = traceback.StackSummary( + t for t in tb if "/twisted/" not in t.filename + ) + + # we expect to see "caller", "concurrently_execute" at the top of the stack self.assertEqual(tb[0].name, "caller") self.assertEqual(tb[1].name, "concurrently_execute") - self.assertEqual(tb[-2].name, "callback") + # ... some stack frames from the implementation of `concurrently_execute` ... + # and at the bottom of the stack we expect to see "callback" + self.assertEqual(tb[-1].name, "callback") else: self.fail("No exception thrown")