Merge pull request #2932 from nicoddemus/deprecate-add-call

Deprecate metafunc.addcall

Author: RonnyPfannschmidt

_pytest/deprecated.py

@@ -45,3 +45,8 @@ class RemovedInPytest4Warning(DeprecationWarning):
     "pycollector makeitem was removed "
     "as it is an accidentially leaked internal api"
 )
+
+METAFUNC_ADD_CALL = (
+    "Metafunc.addcall is deprecated and scheduled to be removed in pytest 4.0.\n"
+    "Please use Metafunc.parametrize instead."
+)

_pytest/python.py

@@ -837,9 +837,13 @@ def parametrize(self, argnames, argvalues, indirect=False, ids=None,
         self._calls = newcalls
 
     def addcall(self, funcargs=None, id=NOTSET, param=NOTSET):
-        """ (deprecated, use parametrize) Add a new call to the underlying
-        test function during the collection phase of a test run.  Note that
-        request.addcall() is called during the test collection phase prior and
+        """ Add a new call to the underlying test function during the collection phase of a test run.
+
+        .. deprecated:: 3.3
+
+            Use :meth:`parametrize` instead.
+
+        Note that request.addcall() is called during the test collection phase prior and
         independently to actual test execution.  You should only use addcall()
         if you need to specify multiple arguments of a test function.
 
@@ -852,6 +856,8 @@ def addcall(self, funcargs=None, id=NOTSET, param=NOTSET):
         :arg param: a parameter which will be exposed to a later fixture function
             invocation through the ``request.param`` attribute.
         """
+        if self.config:
+            self.config.warn('C1', message=deprecated.METAFUNC_ADD_CALL, fslocation=None)
         assert funcargs is None or isinstance(funcargs, dict)
         if funcargs is not None:
             for name in funcargs:

changelog/2876.trivial

@@ -0,0 +1 @@
+Calls to ``Metafunc.addcall`` now emit a deprecation warning. This function is scheduled to be removed in ``pytest-4.0``.

testing/deprecated_test.py

@@ -82,3 +82,20 @@ def test():
         '*--result-log is deprecated and scheduled for removal in pytest 4.0*',
         '*See https://docs.pytest.org/*/usage.html#creating-resultlog-format-files for more information*',
     ])
+
+
+@pytest.mark.filterwarnings('always:Metafunc.addcall is deprecated')
+def test_metafunc_addcall_deprecated(testdir):
+    testdir.makepyfile("""
+        def pytest_generate_tests(metafunc):
+            metafunc.addcall({'i': 1})
+            metafunc.addcall({'i': 2})
+        def test_func(i):
+            pass
+    """)
+    res = testdir.runpytest('-s')
+    assert res.ret == 0
+    res.stdout.fnmatch_lines([
+        "*Metafunc.addcall is deprecated*",
+        "*2 passed, 2 warnings*",
+    ])