Merge pull request #528 from nicoddemus/fix-warnings

Fix some of the warnings displayed by pytest 3.1.0

Author: obestwalter

tests/test_quickstart.py

@@ -63,7 +63,7 @@ def test_quickstart_main_choose_individual_pythons_and_pytest(
 deps =
     pytest
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_choose_individual_pythons_and_nose_adds_deps(
@@ -105,7 +105,7 @@ def test_quickstart_main_choose_individual_pythons_and_nose_adds_deps(
 deps =
     nose
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_choose_individual_pythons_and_trial_adds_deps(
@@ -147,7 +147,7 @@ def test_quickstart_main_choose_individual_pythons_and_trial_adds_deps(
 deps =
     twisted
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_choose_individual_pythons_and_pytest_adds_deps(
@@ -188,7 +188,7 @@ def test_quickstart_main_choose_individual_pythons_and_pytest_adds_deps(
 deps =
     pytest
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_choose_py27_and_pytest_adds_deps(
@@ -221,7 +221,7 @@ def test_quickstart_main_choose_py27_and_pytest_adds_deps(
 deps =
     pytest
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_choose_py27_and_py33_and_pytest_adds_deps(
@@ -254,7 +254,7 @@ def test_quickstart_main_choose_py27_and_py33_and_pytest_adds_deps(
 deps =
     pytest
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_choose_all_pythons_and_pytest_adds_deps(
@@ -287,7 +287,7 @@ def test_quickstart_main_choose_all_pythons_and_pytest_adds_deps(
 deps =
     pytest
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_choose_individual_pythons_and_defaults(
@@ -329,7 +329,7 @@ def test_quickstart_main_choose_individual_pythons_and_defaults(
 deps =
 
 """.lstrip()
-        result = open('tox.ini').read()
+        result = read_tox()
         assert(result == expected_tox_ini)
 
     def test_quickstart_main_existing_tox_ini(self, monkeypatch):
@@ -376,7 +376,7 @@ def test_quickstart_main_existing_tox_ini(self, monkeypatch):
 deps =
 
 """.lstrip()
-        result = open('tox-generated.ini').read()
+        result = read_tox('tox-generated.ini')
         assert(result == expected_tox_ini)
 
 
@@ -408,7 +408,7 @@ def test_pytest(self):
 """.lstrip()
         d = tox._quickstart.process_input(d)
         tox._quickstart.generate(d)
-        result = open('tox.ini').read()
+        result = read_tox()
         # print(result)
         assert(result == expected_tox_ini)
 
@@ -435,7 +435,7 @@ def test_setup_py_test(self):
 """.lstrip()
         d = tox._quickstart.process_input(d)
         tox._quickstart.generate(d)
-        result = open('tox.ini').read()
+        result = read_tox()
         # print(result)
         assert(result == expected_tox_ini)
 
@@ -461,7 +461,7 @@ def test_trial(self):
 """.lstrip()
         d = tox._quickstart.process_input(d)
         tox._quickstart.generate(d)
-        result = open('tox.ini').read()
+        result = read_tox()
         # print(result)
         assert(result == expected_tox_ini)
 
@@ -492,6 +492,11 @@ def test_nosetests(self):
 """.lstrip()
         d = tox._quickstart.process_input(d)
         tox._quickstart.generate(d)
-        result = open('tox.ini').read()
+        result = read_tox()
         # print(result)
         assert(result == expected_tox_ini)
+
+
+def read_tox(fname='tox.ini'):
+    with open(fname) as f:
+        return f.read()

tox.ini

@@ -7,7 +7,7 @@ commands=echo {posargs}
 [testenv]
 commands= py.test --timeout=180 {posargs:tests}
 
-deps=pytest>=2.3.5
+deps=pytest>=3.0.0
     pytest-timeout
 
 [testenv:py26-bare]

tox/_pytestplugin.py

@@ -142,7 +142,7 @@ def mocksession(request):
     class MockSession(Session):
         def __init__(self):
             self._clearmocks()
-            self.config = request.getfuncargvalue("newconfig")([], "")
+            self.config = request.getfixturevalue("newconfig")([], "")
             self.resultlog = ResultLog()
             self._actions = []
 
@@ -168,8 +168,8 @@ def popen(self, args, cwd, shell=None,
 
 @pytest.fixture
 def newmocksession(request):
-    mocksession = request.getfuncargvalue("mocksession")
-    newconfig = request.getfuncargvalue("newconfig")
+    mocksession = request.getfixturevalue("mocksession")
+    newconfig = request.getfixturevalue("newconfig")
 
     def newmocksession(args, source, plugins=()):
         mocksession.config = newconfig(args, source, plugins=plugins)
@@ -179,7 +179,7 @@ def newmocksession(args, source, plugins=()):
 
 class Cmd:
     def __init__(self, request):
-        self.tmpdir = request.getfuncargvalue("tmpdir")
+        self.tmpdir = request.getfixturevalue("tmpdir")
         self.request = request
         current = py.path.local()
         self.request.addfinalizer(current.chdir)