From 2ad43ee442571f685c91b5d23792ae9d31d5e12f Mon Sep 17 00:00:00 2001 From: Bruno Oliveira Date: Wed, 24 Oct 2018 17:33:50 -0300 Subject: [PATCH] Show node that originated a warning in the warnings summary Fix #4221 --- changelog/4221.bugfix.rst | 1 + src/_pytest/terminal.py | 5 ++--- testing/test_warnings.py | 5 +++-- 3 files changed, 6 insertions(+), 5 deletions(-) create mode 100644 changelog/4221.bugfix.rst diff --git a/changelog/4221.bugfix.rst b/changelog/4221.bugfix.rst new file mode 100644 index 000000000..f55722f1c --- /dev/null +++ b/changelog/4221.bugfix.rst @@ -0,0 +1 @@ +Fix bug where the warning summary at the end of the test session was not showing the test where the warning was originated. diff --git a/src/_pytest/terminal.py b/src/_pytest/terminal.py index 8deb330cc..37186a121 100644 --- a/src/_pytest/terminal.py +++ b/src/_pytest/terminal.py @@ -725,11 +725,10 @@ class TerminalReporter(object): # legacy warnings show their location explicitly, while standard warnings look better without # it because the location is already formatted into the message warning_records = list(warning_records) - is_legacy = warning_records[0].legacy - if location and is_legacy: + if location: self._tw.line(str(location)) for w in warning_records: - if is_legacy: + if location: lines = w.message.splitlines() indented = "\n".join(" " + x for x in lines) message = indented.rstrip() diff --git a/testing/test_warnings.py b/testing/test_warnings.py index 10eb5ea33..569414300 100644 --- a/testing/test_warnings.py +++ b/testing/test_warnings.py @@ -48,6 +48,7 @@ def test_normal_flow(testdir, pyfile_with_warnings): result.stdout.fnmatch_lines( [ "*== %s ==*" % WARNINGS_SUMMARY_HEADER, + "test_normal_flow.py::test_func", "*normal_flow_module.py:3: UserWarning: user warning", '* warnings.warn(UserWarning("user warning"))', "*normal_flow_module.py:4: RuntimeWarning: runtime warning", @@ -369,8 +370,8 @@ def test_collection_warnings(testdir): result.stdout.fnmatch_lines( [ "*== %s ==*" % WARNINGS_SUMMARY_HEADER, - "*collection_warnings.py:3: UserWarning: collection warning", - ' warnings.warn(UserWarning("collection warning"))', + " *collection_warnings.py:3: UserWarning: collection warning", + ' warnings.warn(UserWarning("collection warning"))', "* 1 passed, 1 warnings*", ] )