PLEASE APPROVE AND MERGE - autofix [PP-225] fix @pytest.yield_fixture is deprecated

pytest-fixture-config/tests/unit/test_fixture_config.py

@@ -48,7 +48,7 @@ def test_requires_config_doesnt_skip(another_fixture):
 
 
 
-@pytest.yield_fixture
+@pytest.fixture
 @yield_requires_config(CONFIG1, ('foo', 'bar'))
 def yet_another_fixture():
     raise ValueError('Should also not run')
@@ -59,7 +59,7 @@ def test_yield_requires_config_skips(yet_another_fixture):
     raise ValueError('Should also not run')
 
 
-@pytest.yield_fixture
+@pytest.fixture
 @yield_requires_config(CONFIG1, ('bar',))
 def yet_some_other_fixture():
     yield 'yyyy'

pytest-git/pytest_git.py

@@ -5,7 +5,7 @@
 from git import Repo
 
 
-@pytest.yield_fixture
+@pytest.fixture
 def git_repo(request):
     """ Function-scoped fixture to create a new git repo in a temporary workspace.
 

pytest-listener/pytest_listener.py

@@ -24,7 +24,7 @@
 )
 
 
-@pytest.yield_fixture(scope='module')
+@pytest.fixture(scope='module')
 def listener(request):
     """ Simple module-scoped network listener.
 

pytest-qt-app/pytest_qt_app.py

@@ -14,7 +14,7 @@ class _TestQtApp(object):
 TestQtApp = _TestQtApp()
 
 
-@pytest.yield_fixture(scope="session")
+@pytest.fixture(scope="session")
 def q_application():
     """ Initialise a QT application with a Xvfb server
     """

pytest-server-fixtures/pytest_server_fixtures/http.py

@@ -17,7 +17,7 @@
 log = logging.getLogger(__name__)
 
 
-@pytest.yield_fixture
+@pytest.fixture
 def simple_http_test_server():
     """ Function-scoped py.test fixture to serve up a directory via HTTP.
     """

pytest-server-fixtures/pytest_server_fixtures/httpd.py

@@ -18,7 +18,7 @@ def is_rhel():
     """"Check if OS is RHEL/Centos"""
     return 'el' in platform.uname()[2]
 
-@pytest.yield_fixture(scope='function')
+@pytest.fixture(scope='function')
 @yield_requires_config(CONFIG, ['httpd_executable', 'httpd_modules'])
 def httpd_server():
     """ Function-scoped httpd server in a local thread.

pytest-server-fixtures/pytest_server_fixtures/jenkins.py

@@ -14,7 +14,7 @@
 from .http import HTTPTestServer
 
 
-@pytest.yield_fixture(scope='session')
+@pytest.fixture(scope='session')
 @yield_requires_config(CONFIG, ['jenkins_war', 'java_executable'])
 def jenkins_server():
     """ Session-scoped Jenkins server instance
@@ -29,7 +29,7 @@ def jenkins_server():
         yield p
 
 
-@pytest.yield_fixture(scope='module')
+@pytest.fixture(scope='module')
 @yield_requires_config(CONFIG, ['jenkins_war', 'java_executable'])
 def jenkins_server_module():
     """ Module-scoped Jenkins server instance

pytest-server-fixtures/pytest_server_fixtures/mongo.py

@@ -21,7 +21,7 @@ def _mongo_server():
         test_server.teardown()
 
 
-@pytest.yield_fixture(scope='function')
+@pytest.fixture(scope='function')
 @yield_requires_config(CONFIG, ['mongo_bin'])
 def mongo_server():
     """ Function-scoped MongoDB server started in a local thread.
@@ -40,7 +40,7 @@ def mongo_server():
         yield server
 
 
-@pytest.yield_fixture(scope='session')
+@pytest.fixture(scope='session')
 @yield_requires_config(CONFIG, ['mongo_bin'])
 def mongo_server_sess():
     """ Same as mongo_server fixture, scoped as session instead.
@@ -49,7 +49,7 @@ def mongo_server_sess():
         yield server
 
 
-@pytest.yield_fixture(scope='class')
+@pytest.fixture(scope='class')
 @yield_requires_config(CONFIG, ['mongo_bin'])
 def mongo_server_cls(request):
     """ Same as mongo_server fixture, scoped for test classes.
@@ -59,7 +59,7 @@ def mongo_server_cls(request):
         yield server
 
 
-@pytest.yield_fixture(scope='module')
+@pytest.fixture(scope='module')
 @yield_requires_config(CONFIG, ['mongo_bin'])
 def mongo_server_module():
     """ Same as mongo_server fixture, scoped for test modules.

pytest-server-fixtures/pytest_server_fixtures/xvfb.py

@@ -13,7 +13,7 @@
 from pytest_server_fixtures import CONFIG
 
 
-@pytest.yield_fixture(scope='function')
+@pytest.fixture(scope='function')
 @yield_requires_config(CONFIG, ['xvfb_executable'])
 def xvfb_server():
     """ Function-scoped Xvfb (X-Windows Virtual Frame Buffer) in a local thread.
@@ -23,7 +23,7 @@ def xvfb_server():
     test_server.close()
 
 
-@pytest.yield_fixture(scope='session')
+@pytest.fixture(scope='session')
 @yield_requires_config(CONFIG, ['xvfb_executable'])
 def xvfb_server_sess():
     """ Session-scoped Xvfb (X-Windows Virtual Frame Buffer) in a local thread.

pytest-shutil/pytest_shutil/workspace.py

@@ -14,7 +14,7 @@
 log = logging.getLogger(__name__)
 
 
-@pytest.yield_fixture()
+@pytest.fixture()
 def workspace():
     """ Function-scoped temporary workspace that cleans up on exit.
 

pytest-svn/pytest_svn.py

@@ -4,7 +4,7 @@
 from pytest_shutil.workspace import Workspace
 
 
-@pytest.yield_fixture()
+@pytest.fixture()
 def svn_repo():
     """ Function-scoped fixture to create a new svn repo in a temporary workspace.
 

pytest-webdriver/pytest_webdriver.py

@@ -46,7 +46,7 @@ def browser_to_use(webdriver, browser):
     return b
 
 
-@pytest.yield_fixture(scope='function')
+@pytest.fixture(scope='function')
 def webdriver(request):
     """ Connects to a remote selenium webdriver and returns the browser handle.
         Scoped on a per-function level so you get one browser window per test.