[RM-37939] tests: consume the tmpdir fixture directly

Signed-off-by: Alfredo Deza <adeza@redhat.com>

diff --git a/ceph_deploy/tests/conftest.py b/ceph_deploy/tests/conftest.py
index 3b673d1bda86daf7084bdbb1e0f53899daa5bb80..ee7fcf1f8f052681434012f017e489d4f6aa8745 100644 (file)
--- a/ceph_deploy/tests/conftest.py
+++ b/ceph_deploy/tests/conftest.py
@@ -87,7 +87,7 @@ class CLITester(object):
 
 
 @pytest.fixture
-def cli(request):
+def cli(request, tmpdir):
     """
     Test command line behavior.
     """
@@ -95,6 +95,4 @@ def cli(request):
     # the tmpdir here will be the same value as the test function
     # sees; we rely on that to let caller prepare and introspect
     # any files the cli tool will read or create
-    tmpdir = request.getfuncargvalue('tmpdir')
-
     return CLITester(tmpdir=tmpdir)

diff --git a/ceph_deploy/tests/test_cli_new.py b/ceph_deploy/tests/test_cli_new.py
index 2ba221ec313b6b010e907567984ff5fc5faa190a..056bc7883d6805929cf508a37dc24c02b7d74949 100644 (file)
--- a/ceph_deploy/tests/test_cli_new.py
+++ b/ceph_deploy/tests/test_cli_new.py
@@ -24,8 +24,7 @@ def test_write_global_conf_section(tmpdir):
 
 
 @pytest.fixture
-def newcfg(request):
-    tmpdir = request.getfuncargvalue('tmpdir')
+def newcfg(request, tmpdir):
     fake_ip_addresses = lambda x: ['10.0.0.1']
 
     def new(*args):