Fix PT022 pytest-useless-yield-fixture

Author: TimPansino

tests/agent_unittests/test_package_version_utils.py

@@ -47,7 +47,7 @@ def patched_pytest_module(monkeypatch):
         if hasattr(pytest, attr):
             monkeypatch.delattr(pytest, attr)
 
-    yield pytest
+    return pytest
 
 
 @pytest.fixture(autouse=True)

tests/agent_unittests/test_sampler_metrics.py

@@ -46,7 +46,7 @@ def cpu_data_source():
 @pytest.fixture
 def memory_data_source():
     sampler = memory_usage_data_source(settings=())["factory"](environ=())
-    yield sampler
+    return sampler
 
 
 PID = os.getpid()

tests/application_celery/conftest.py

@@ -42,4 +42,4 @@ def celery_worker_parameters():
 
 @pytest.fixture(scope="session", autouse=True)
 def celery_worker_available(celery_session_worker):
-    yield celery_session_worker
+    return celery_session_worker

tests/datastore_cassandradriver/conftest.py

@@ -77,12 +77,12 @@ def cluster_options(connection_class):
         "connection_class": connection_class,
         "protocol_version": 4,
     }
-    yield cluster_options
+    return cluster_options
 
 
 @pytest.fixture
 def cluster(cluster_options):
     from cassandra.cluster import Cluster
 
     cluster = Cluster(**cluster_options)
-    yield cluster
+    return cluster

tests/datastore_firestore/conftest.py

@@ -84,7 +84,7 @@ def async_client(loop):
 @pytest.fixture
 def async_collection(async_client, collection):
     # Use the same collection name as the collection fixture
-    yield async_client.collection(collection.id)
+    return async_client.collection(collection.id)
 
 
 @pytest.fixture(scope="session")

tests/datastore_firestore/test_async_query.py

@@ -172,7 +172,6 @@ async def _mock_partition_query():
         return _mock_partition_query()
 
     monkeypatch.setattr(async_client._firestore_api, "partition_query", mock_partition_query)
-    yield
 
 
 @pytest.fixture

tests/datastore_firestore/test_query.py

@@ -165,7 +165,6 @@ def mock_partition_query(*args, **kwargs):
         yield Cursor(before=False, values=[Value(reference_value=documents[0].path)])
 
     monkeypatch.setattr(client._firestore_api, "partition_query", mock_partition_query)
-    yield
 
 
 @pytest.fixture

tests/external_httpx/conftest.py

@@ -46,7 +46,7 @@ class RealHTTP2Server:
         host = settings["host"]
         port = settings["port"]
 
-    yield RealHTTP2Server
+    return RealHTTP2Server
 
 
 @pytest.fixture

tests/logger_structlog/test_processor_formatter.py

@@ -73,7 +73,7 @@ def emit(self, record):
 
     structlog_logger = structlog.get_logger(logger_attr=2)
 
-    yield logging_logger, structlog_logger
+    return logging_logger, structlog_logger
 
 
 @pytest.fixture

tests/messagebroker_kombu/conftest.py

@@ -77,7 +77,7 @@ def producer(producer_connection, queue):
 
     producer = producer_connection.Producer(serializer="json")
 
-    yield producer
+    return producer
 
 
 @pytest.fixture