Patchwork ui: add configlist doctest to document a bit more of the whitespace behavior

login
register
mail settings
Submitter Augie Fackler
Date Nov. 1, 2017, 11:31 p.m.
Message ID <7e5b4ce0c445d4fc8f90.1509579109@augie-macbookpro2.roam.corp.google.com>
Download mbox | patch
Permalink /patch/25323/
State Accepted
Headers show

Comments

Augie Fackler - Nov. 1, 2017, 11:31 p.m.
# HG changeset patch
# User Augie Fackler <augie@google.com>
# Date 1508986914 14400
#      Wed Oct 25 23:01:54 2017 -0400
# Node ID 7e5b4ce0c445d4fc8f906f6926c71163a27c85a2
# Parent  3ce0e4b51f789eff195ec900a07c1fa5e8d5c5f2
ui: add configlist doctest to document a bit more of the whitespace behavior

I'm surprised this wasn't explicitly tested, so I added a test.
Yuya Nishihara - Nov. 2, 2017, 12:40 p.m.
On Wed, 01 Nov 2017 19:31:49 -0400, Augie Fackler wrote:
> # HG changeset patch
> # User Augie Fackler <augie@google.com>
> # Date 1508986914 14400
> #      Wed Oct 25 23:01:54 2017 -0400
> # Node ID 7e5b4ce0c445d4fc8f906f6926c71163a27c85a2
> # Parent  3ce0e4b51f789eff195ec900a07c1fa5e8d5c5f2
> ui: add configlist doctest to document a bit more of the whitespace behavior

Queued, thanks.

Patch

diff --git a/mercurial/ui.py b/mercurial/ui.py
--- a/mercurial/ui.py
+++ b/mercurial/ui.py
@@ -695,6 +695,9 @@  class ui(object):
         >>> u.setconfig(s, b'list1', b'this,is "a small" ,test')
         >>> u.configlist(s, b'list1')
         ['this', 'is', 'a small', 'test']
+        >>> u.setconfig(s, b'list2', b'this, is "a small" , test ')
+        >>> u.configlist(s, b'list2')
+        ['this', 'is', 'a small', 'test']
         """
         # default is not always a list
         v = self.configwith(config.parselist, section, name, default,