[Python-checkins] bpo-39450 Stripped whitespace before parsing the docstring in TestCase.shortDescription (GH-18175)

Steve Cirelli webhook-mailer at python.org
Mon Feb 3 02:06:58 EST 2020


https://github.com/python/cpython/commit/032de7324e30c6b44ef272cea3be205a3d768759
commit: 032de7324e30c6b44ef272cea3be205a3d768759
branch: master
author: Steve Cirelli <scirelli+git at gmail.com>
committer: GitHub <noreply at github.com>
date: 2020-02-03T07:06:50Z
summary:

bpo-39450 Stripped whitespace before parsing the docstring in TestCase.shortDescription (GH-18175)

files:
A Misc/NEWS.d/next/Library/2020-02-02-14-46-34.bpo-39450.48R274.rst
M Lib/unittest/case.py
M Lib/unittest/test/test_case.py

diff --git a/Lib/unittest/case.py b/Lib/unittest/case.py
index fa64a6ea2378c..5e5d535dc6938 100644
--- a/Lib/unittest/case.py
+++ b/Lib/unittest/case.py
@@ -512,7 +512,7 @@ def shortDescription(self):
         the specified test method's docstring.
         """
         doc = self._testMethodDoc
-        return doc and doc.split("\n")[0].strip() or None
+        return doc.strip().split("\n")[0].strip() if doc else None
 
 
     def id(self):
diff --git a/Lib/unittest/test/test_case.py b/Lib/unittest/test/test_case.py
index c2401c39b917e..f855c4dc00b31 100644
--- a/Lib/unittest/test/test_case.py
+++ b/Lib/unittest/test/test_case.py
@@ -610,6 +610,15 @@ def testShortDescriptionWithMultiLineDocstring(self):
                  'Tests shortDescription() for a method with a longer '
                  'docstring.')
 
+    def testShortDescriptionWhitespaceTrimming(self):
+        """
+            Tests shortDescription() whitespace is trimmed, so that the first
+            line of nonwhite-space text becomes the docstring.
+        """
+        self.assertEqual(
+            self.shortDescription(),
+            'Tests shortDescription() whitespace is trimmed, so that the first')
+
     def testAddTypeEqualityFunc(self):
         class SadSnake(object):
             """Dummy class for test_addTypeEqualityFunc."""
diff --git a/Misc/NEWS.d/next/Library/2020-02-02-14-46-34.bpo-39450.48R274.rst b/Misc/NEWS.d/next/Library/2020-02-02-14-46-34.bpo-39450.48R274.rst
new file mode 100644
index 0000000000000..55fed519a2d80
--- /dev/null
+++ b/Misc/NEWS.d/next/Library/2020-02-02-14-46-34.bpo-39450.48R274.rst
@@ -0,0 +1,2 @@
+Striped whitespace from docstring before returning it from
+:func:`unittest.case.shortDescription`.



More information about the Python-checkins mailing list