From d7a5c5716f2386c083a2658708c4a3d9eda522c4 Mon Sep 17 00:00:00 2001
From: wanghui <wh_linux@126.com>
Date: Fri, 26 May 2017 13:12:02 +0800
Subject: [PATCH] Add UnicodeWarning for unicode warnings in Python2

---
 _pytest/compat.py        |  2 ++
 _pytest/warnings.py      | 15 ++++++++++++++-
 testing/test_warnings.py | 31 +++++++++++++++++++++++++++++++
 3 files changed, 47 insertions(+), 1 deletion(-)

diff --git a/_pytest/compat.py b/_pytest/compat.py
index 25610b645..8c200af5f 100644
--- a/_pytest/compat.py
+++ b/_pytest/compat.py
@@ -126,6 +126,7 @@ if _PY3:
     import codecs
     imap = map
     STRING_TYPES = bytes, str
+    UNICODE_TYPES = str,
 
     def _escape_strings(val):
         """If val is pure ascii, returns it as a str().  Otherwise, escapes
@@ -157,6 +158,7 @@ if _PY3:
             return val.encode('unicode_escape').decode('ascii')
 else:
     STRING_TYPES = bytes, str, unicode
+    UNICODE_TYPES = unicode,
 
     from itertools import imap  # NOQA
 
diff --git a/_pytest/warnings.py b/_pytest/warnings.py
index 7596b0098..cbed43b8f 100644
--- a/_pytest/warnings.py
+++ b/_pytest/warnings.py
@@ -63,11 +63,24 @@ def catch_warnings_for_item(item):
         yield
 
         for warning in log:
+            warn_msg = warning.message
+            unicode_warning = False
+
+            if compat._PY2 and any(isinstance(m, compat.UNICODE_TYPES) for m in warn_msg.args):
+                warn_msg.args = [compat.safe_str(m) for m in warn_msg.args]
+                unicode_warning = True
+
             msg = warnings.formatwarning(
-                compat.safe_str(warning.message), warning.category,
+                warn_msg, warning.category,
                 warning.filename, warning.lineno, warning.line)
             item.warn("unused", msg)
 
+            if unicode_warning:
+                warnings.warn(
+                    "This warning %s is broken as it's message is not a str instance"
+                    "(after all this is a stdlib problem workaround)" % msg,
+                    UnicodeWarning)
+
 
 @pytest.hookimpl(hookwrapper=True)
 def pytest_runtest_protocol(item):
diff --git a/testing/test_warnings.py b/testing/test_warnings.py
index 85fda430e..df9c4cdc4 100644
--- a/testing/test_warnings.py
+++ b/testing/test_warnings.py
@@ -1,6 +1,8 @@
 # -*- coding: utf8 -*-
 from __future__ import unicode_literals
 
+import sys
+
 import pytest
 
 
@@ -111,6 +113,8 @@ def test_ignore(testdir, pyfile_with_warnings, method):
 
 
 
+@pytest.mark.skipif(sys.version_info < (3, 0),
+                    reason='warnings message is unicode is ok in python3')
 def test_unicode(testdir, pyfile_with_warnings):
     testdir.makepyfile('''
         # -*- coding: utf8 -*-
@@ -135,3 +139,30 @@ def test_unicode(testdir, pyfile_with_warnings):
         '* 1 passed, 1 warnings*',
     ])
 
+
+@pytest.mark.skipif(sys.version_info >= (3, 0),
+                    reason='warnings message is broken as it is not str instance')
+def test_py2_unicode(testdir, pyfile_with_warnings):
+    testdir.makepyfile('''
+        # -*- coding: utf8 -*-
+        import warnings
+        import pytest
+
+
+        @pytest.fixture
+        def fix():
+            warnings.warn(u"测试")
+            yield
+
+        def test_func(fix):
+            pass
+    ''')
+    result = testdir.runpytest()
+    result.stdout.fnmatch_lines([
+        '*== %s ==*' % WARNINGS_SUMMARY_HEADER,
+
+        '*test_py2_unicode.py:8: UserWarning: \u6d4b\u8bd5',
+        '*warnings.warn(u"\u6d4b\u8bd5")',
+        '*warnings.py:82: UnicodeWarning: This warning*\u6d4b\u8bd5',
+        '* 1 passed, 2 warnings*',
+    ])