diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2023-04-30 18:27:24 -0400 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2023-05-09 20:23:48 -0400 |
| commit | 60b31198311eedfa3814e7098c94d3aa29338fdd (patch) | |
| tree | 5fe3a55ef67ab14fa63a0a3122a1326b830ceb72 /test/base/test_concurrency_py3k.py | |
| parent | 228490ead7048f2e558c25b0f055bdb952272ec4 (diff) | |
| download | sqlalchemy-60b31198311eedfa3814e7098c94d3aa29338fdd.tar.gz | |
fix test suite warnings
fix a handful of warnings that were emitting but not raising,
usually because they were inside an "expect_warnings" block.
modify "expect_warnings" to always use "raise_on_any_unexpected"
behavior; remove this parameter.
Fixed issue in semi-private ``await_only()`` and ``await_fallback()``
concurrency functions where the given awaitable would remain un-awaited if
the function threw a ``GreenletError``, which could cause "was not awaited"
warnings later on if the program continued. In this case, the given
awaitable is now cancelled before the exception is thrown.
Change-Id: I33668c5e8c670454a3d879e559096fb873b57244
Diffstat (limited to 'test/base/test_concurrency_py3k.py')
| -rw-r--r-- | test/base/test_concurrency_py3k.py | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/test/base/test_concurrency_py3k.py b/test/base/test_concurrency_py3k.py index 6a3098a6a..17a0fafb5 100644 --- a/test/base/test_concurrency_py3k.py +++ b/test/base/test_concurrency_py3k.py @@ -95,7 +95,12 @@ class TestAsyncioCompat(fixtures.TestBase): ): await_only(to_await) - # ensure no warning + # existing awaitable is done + with expect_raises(RuntimeError): + await greenlet_spawn(await_fallback, to_await) + + # no warning for a new one... + to_await = run1() await greenlet_spawn(await_fallback, to_await) @async_test @@ -118,7 +123,8 @@ class TestAsyncioCompat(fixtures.TestBase): ): await greenlet_spawn(go) - await to_await + with expect_raises(RuntimeError): + await to_await @async_test async def test_await_only_error(self): @@ -141,7 +147,8 @@ class TestAsyncioCompat(fixtures.TestBase): ): await greenlet_spawn(go) - await to_await + with expect_raises(RuntimeError): + await to_await @async_test async def test_contextvars(self): |