fix: Fix test assertions for warnings.warn and missing decorators

- Update test_close_with_async_transport_warns and
  test_flush_with_async_transport_warns to check warnings.warn
  instead of logger.warning
- Add missing @skip_under_gevent, @pytest.mark.asyncio, and
  @pytest.mark.skipif decorators on test_async_transport_concurrent_requests

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: BYK

tests/test_client.py

@@ -1655,11 +1655,9 @@ async def test_async_proxy(monkeypatch, testcase):
 @skip_under_gevent
 @pytest.mark.asyncio
 @pytest.mark.skipif(not PY38, reason="Async client methods require Python 3.8+")
-async def test_close_with_async_transport_warns(caplog):
-    """Test close() with AsyncHttpTransport logs a warning."""
-    import logging
-
-    caplog.set_level(logging.WARNING)
+async def test_close_with_async_transport_warns():
+    """Test close() with AsyncHttpTransport emits a warning."""
+    import warnings as _warnings
 
     client = Client(
         "https://foo@sentry.io/123",
@@ -1668,13 +1666,10 @@ async def test_close_with_async_transport_warns(caplog):
     )
     assert isinstance(client.transport, AsyncHttpTransport)
 
-    with mock.patch("sentry_sdk.client.logger") as mock_logger:
+    with _warnings.catch_warnings(record=True) as w:
+        _warnings.simplefilter("always")
         client.close()
-        mock_logger.warning.assert_called_with(
-            "close() used with AsyncHttpTransport. "
-            "Prefer close_async() for graceful async shutdown. "
-            "Performing synchronous best-effort cleanup."
-        )
+        assert any("close_async()" in str(warning.message) for warning in w)
 
 
 @skip_under_gevent
@@ -1726,19 +1721,20 @@ async def test_close_async_no_transport():
 @pytest.mark.asyncio
 @pytest.mark.skipif(not PY38, reason="Async client methods require Python 3.8+")
 async def test_flush_with_async_transport_warns():
-    """Test flush() with AsyncHttpTransport logs a warning and returns."""
+    """Test flush() with AsyncHttpTransport emits a warning and returns."""
+    import warnings as _warnings
+
     client = Client(
         "https://foo@sentry.io/123",
         _experiments={"transport_async": True},
         integrations=[AsyncioIntegration()],
     )
     assert isinstance(client.transport, AsyncHttpTransport)
 
-    with mock.patch("sentry_sdk.client.logger") as mock_logger:
+    with _warnings.catch_warnings(record=True) as w:
+        _warnings.simplefilter("always")
         client.flush(timeout=1.0)
-        mock_logger.warning.assert_called_with(
-            "flush() used with AsyncHttpTransport. Please use flush_async() instead."
-        )
+        assert any("flush_async()" in str(warning.message) for warning in w)
     await client.close_async()
 
 

tests/test_transport.py

@@ -1055,6 +1055,9 @@ async def test_async_two_way_ssl_authentication():
     await client.close_async()
 
 
+@skip_under_gevent
+@pytest.mark.asyncio
+@pytest.mark.skipif(not PY38, reason="Async transport requires Python 3.8+")
 async def test_async_transport_concurrent_requests(
     capturing_server, make_client, caplog
 ):