The test imports ASYNC_TRANSPORT_ENABLED from sentry_sdk.transport, but this variable does not exist. The correct variable name in transport.py is ASYNC_TRANSPORT_AVAILABLE (defined at lines 34-36). This will cause an ImportError when the test runs, preventing it from executing.

Also found at:

• scripts/populate_tox/tox.jinja:91

The test mocks sentry_sdk.client.logger.warning and expects it to be called with a specific message. However, the actual close() implementation uses warnings.warn() instead of logger.warning(). Additionally, the expected message ("close() used with AsyncHttpTransport. Prefer close_async() for graceful async shutdown. Performing synchronous best-effort cleanup.") differs from the actual message ("close() used with AsyncHttpTransport. Use close_async() instead."). This test will always fail.

Also found at:

• tests/test_client.py:1737-1741

The lambda lambda t: self._on_task_complete(t, queue_ref) captures queue_ref by reference rather than by value. In Python, closures created in loops capture variables by name, so if multiple tasks are spawned before their callbacks fire, all callbacks will use the most recent queue_ref assignment. This defeats the intended purpose of capturing the queue reference at dispatch time to survive kill()/start() cycles, potentially causing task_done() to be called on the wrong queue.

The test_async_transport_concurrent_requests function is missing the @skip_under_gevent, @pytest.mark.asyncio, and @pytest.mark.skipif(not PY38, ...) decorators that all other async transport tests in this file have. Without @pytest.mark.asyncio, pytest will not properly execute this as an async test, causing it to appear to pass without actually running the async code. This could lead to false confidence in test coverage.

The _make_async_transport_options() helper function is defined but never called anywhere in the test file. This appears to be dead code that adds maintenance burden without providing value. The async tests in this file all construct options inline rather than using this helper.