Show "short test summary info" after tracebacks and warnings

_pytest/hookspec.py

@@ -489,8 +489,16 @@ def pytest_report_teststatus(report):
  Stops at first non-None result, see :ref:`firstresult` """
 
 
-def pytest_terminal_summary(terminalreporter, exitstatus):
- """ add additional section in terminal summary reporting. """
+def pytest_terminal_summary(config, terminalreporter, exitstatus):
+ """Add a section to terminal summary reporting.
+
+ :param _pytest.config.Config config: pytest config object
+ :param _pytest.terminal.TerminalReporter terminalreporter: the internal terminal reporter object
+ :param int exitstatus: the exit status that will be reported back to the OS
+
+ .. versionadded:: 3.5
+ The ``config`` parameter.
+ """
 
 
 @hookspec(historic=True)

_pytest/skipping.py

@@ -1,7 +1,6 @@
 """ support for skip/xfail functions and markers. """
 from __future__ import absolute_import, division, print_function
 
-
 from _pytest.config import hookimpl
 from _pytest.mark import MarkInfo, MarkDecorator
 from _pytest.mark.evaluate import MarkEvaluator
@@ -14,11 +13,11 @@ def pytest_addoption(parser):
  action="store_true", dest="runxfail", default=False,
  help="run tests even if they are marked xfail")
 
- parser.addini("xfail_strict", "default for the strict parameter of xfail "
-  "markers when not given explicitly (default: "
-  "False)",
-  default=False,
-  type="bool")
+ parser.addini("xfail_strict",
+ "default for the strict parameter of xfail "
+ "markers when not given explicitly (default: False)",
+ default=False,
+ type="bool")
 
 
 def pytest_configure(config):
@@ -130,7 +129,7 @@ def pytest_runtest_makereport(item, call):
  rep.outcome = "passed"
  rep.wasxfail = rep.longrepr
  elif item.config.option.runxfail:
- pass  # don't interefere
+ pass # don't interefere
  elif call.excinfo and call.excinfo.errisinstance(xfail.Exception):
  rep.wasxfail = "reason: " + call.excinfo.value.msg
  rep.outcome = "skipped"
@@ -160,6 +159,7 @@ def pytest_runtest_makereport(item, call):
  filename, line = item.location[:2]
  rep.longrepr = filename, line, reason
 
+
 # called by terminalreporter progress reporting
 
 
@@ -170,6 +170,7 @@ def pytest_report_teststatus(report):
  elif report.passed:
  return "xpassed", "X", ("XPASS", {'yellow': True})
 
+
 # called by the terminalreporter instance/plugin
 
 
@@ -233,7 +234,7 @@ def folded_skips(skipped):
  # TODO: revisit after marks scope would be fixed
  when = getattr(event, 'when', None)
  if when == 'setup' and 'skip' in keywords and 'pytestmark' not in keywords:
- key = (key[0], None, key[2], )
+ key = (key[0], None, key[2])
  d.setdefault(key, []).append(event)
  values = []
  for key, events in d.items():
@@ -269,6 +270,7 @@ def show_skipped(terminalreporter, lines):
 def shower(stat, format):
  def show_(terminalreporter, lines):
  return show_simple(terminalreporter, lines, stat, format)
+
  return show_
 
 

_pytest/terminal.py

@@ -480,16 +480,21 @@ def pytest_sessionfinish(self, exitstatus):
  EXIT_NOTESTSCOLLECTED)
  if exitstatus in summary_exit_codes:
  self.config.hook.pytest_terminal_summary(terminalreporter=self,
+ config=self.config,
  exitstatus=exitstatus)
- self.summary_errors()
- self.summary_failures()
- self.summary_warnings()
- self.summary_passes()
  if exitstatus == EXIT_INTERRUPTED:
  self._report_keyboardinterrupt()
  del self._keyboardinterrupt_memo
  self.summary_stats()
 
+ @pytest.hookimpl(hookwrapper=True)
+ def pytest_terminal_summary(self):
+ self.summary_errors()
+ self.summary_failures()
+ yield
+ self.summary_warnings()
+ self.summary_passes()
+
  def pytest_keyboard_interrupt(self, excinfo):
  self._keyboardinterrupt_memo = excinfo.getrepr(funcargs=True)
 

changelog/3255.feature.rst

@@ -0,0 +1 @@
+The *short test summary info* section now is displayed after tracebacks and warnings in the terminal.

testing/test_skipping.py

@@ -1065,3 +1065,18 @@ def pytest_collect_file(path, parent):
  assert not failed
  xfailed = [r for r in skipped if hasattr(r, 'wasxfail')]
  assert xfailed
+
+
+def test_summary_list_after_errors(testdir):
+ """Ensure the list of errors/fails/xfails/skips appear after tracebacks in terminal reporting."""
+ testdir.makepyfile("""
+ import pytest
+ def test_fail():
+ assert 0
+ """)
+ result = testdir.runpytest('-ra')
+ result.stdout.fnmatch_lines([
+ '=* FAILURES *=',
+ '*= short test summary info =*',
+ 'FAIL test_summary_list_after_errors.py::test_fail',
+ ])