[WIP] Deprecate TerminalReporter.writer

_pytest/deprecated.py

@@ -50,3 +50,6 @@ class RemovedInPytest4Warning(DeprecationWarning):
     "Metafunc.addcall is deprecated and scheduled to be removed in pytest 4.0.\n"
     "Please use Metafunc.parametrize instead."
 )
+
+TERMINAL_REPORTER_WARNING = RemovedInPytest4Warning('The "writer" attribute is deprecated and '
+                                                    'will be removed in a future release.')

_pytest/fixtures.py

@@ -553,7 +553,7 @@ def _getscopeitem(self, scope):
         if node is None and scope == "class":
             # fallback to function item itself
             node = self._pyfuncitem
-        assert node
+        assert node, 'Could not obtain a node for scope "{}" for function {!r}'.format(scope, self._pyfuncitem)
         return node
 
     def __repr__(self):

_pytest/terminal.py

@@ -8,13 +8,15 @@
 import platform
 import sys
 import time
+import warnings
 
 import pluggy
 import py
 import six
 
 import pytest
 from _pytest import nodes
+from _pytest.deprecated import TERMINAL_REPORTER_WARNING
 from _pytest.main import EXIT_OK, EXIT_TESTSFAILED, EXIT_INTERRUPTED, \
     EXIT_USAGEERROR, EXIT_NOTESTSCOLLECTED
 
@@ -153,6 +155,16 @@ def __init__(self, config, file=None):
         self._progress_items_reported = 0
         self._show_progress_info = self.config.getini('console_output_style') == 'progress'
 
+    @property
+    def writer(self):
+        warnings.warn(TERMINAL_REPORTER_WARNING, stacklevel=2)
+        return self._tw
+
+    @writer.setter
+    def writer(self, writer):
+        warnings.warn(TERMINAL_REPORTER_WARNING, stacklevel=2)
+        self._tw = writer
+
     def hasopt(self, char):
         char = {'xfailed': 'x', 'skipped': 's'}.get(char, char)
         return char in self.reportchars

changelog/2984.bugfix

@@ -0,0 +1 @@
+Bring back ``TerminalReporter.writer`` as an alias to ``TerminalReporter._tw``. This alias was removed by accident in the ``3.3.0`` release.

changelog/2984.removal

@@ -0,0 +1 @@
+``TerminalReporter.writer`` has been deprecated and will be removed in a future release. This attribute is an internal API which was not meant to be exposed as public, if needed use the functions of ``TerminalReporter`` directly.

doc/en/backwards-compatibility.rst

@@ -79,6 +79,10 @@ Use ``[tool:pytest]`` instead for compatibility with other tools.
 
 Deprecated in ``3.0``.
 
+**TerminalReporter.writer**
+
+This attribute was never meant to be part of the public API and has been deprecated in ``3.4``.
+
 Past Releases
 ~~~~~~~~~~~~~
 
@@ -102,4 +106,4 @@ Past Releases
 3.3
 ^^^
 
-* Dropped support for EOL Python 2.6 and 3.3.
+* Dropped support for EOL Python 2.6 and 3.3.

testing/deprecated_test.py

@@ -101,6 +101,23 @@ def test_func(i):
     ])
 
 
+def test_terminal_reporter_writer_deprecated(pytestconfig):
+    """TerminalReporter.writer is deprecated and meant to be removed in a future release (#2984)
+    """
+    try:
+        import xdist  # noqa
+        pytest.skip('xdist workers disable the terminal reporter plugin')
+    except ImportError:
+        pass
+    terminal_reporter = pytestconfig.pluginmanager.get_plugin('terminalreporter')
+    with pytest.deprecated_call():
+        # getter
+        _ = terminal_reporter.writer  # noqa
+    with pytest.deprecated_call():
+        # setter
+        terminal_reporter.writer = terminal_reporter._tw
+
+
 def test_pytest_catchlog_deprecated(testdir):
     testdir.makepyfile("""
         def test_func(pytestconfig):