changeset:   95477:a48e76252952
user:        Serhiy Storchaka <storchaka@gmail.com>
date:        Tue Apr 07 19:09:01 2015 +0300
files:       Lib/test/test_urlparse.py Lib/urllib/parse.py Misc/NEWS
description:
Issue #23411: Added DefragResult, ParseResult, SplitResult, DefragResultBytes,
ParseResultBytes, and SplitResultBytes to urllib.parse.__all__.
Patch by Martin Panter.


diff -r ea21b99d002e -r a48e76252952 Lib/test/test_urlparse.py
--- a/Lib/test/test_urlparse.py	Tue Apr 07 12:00:37 2015 -0400
+++ b/Lib/test/test_urlparse.py	Tue Apr 07 19:09:01 2015 +0300
@@ -840,6 +840,22 @@
         quoter = urllib.parse.Quoter(urllib.parse._ALWAYS_SAFE)
         self.assertIn('Quoter', repr(quoter))
 
+    def test_all(self):
+        expected = []
+        undocumented = {
+            'splitattr', 'splithost', 'splitnport', 'splitpasswd',
+            'splitport', 'splitquery', 'splittag', 'splittype', 'splituser',
+            'splitvalue',
+            'Quoter', 'ResultBase', 'clear_cache', 'to_bytes', 'unwrap',
+        }
+        for name in dir(urllib.parse):
+            if name.startswith('_') or name in undocumented:
+                continue
+            object = getattr(urllib.parse, name)
+            if getattr(object, '__module__', None) == 'urllib.parse':
+                expected.append(name)
+        self.assertCountEqual(urllib.parse.__all__, expected)
+
 
 class Utility_Tests(unittest.TestCase):
     """Testcase to test the various utility functions in the urllib."""
diff -r ea21b99d002e -r a48e76252952 Lib/urllib/parse.py
--- a/Lib/urllib/parse.py	Tue Apr 07 12:00:37 2015 -0400
+++ b/Lib/urllib/parse.py	Tue Apr 07 19:09:01 2015 +0300
@@ -34,7 +34,9 @@
 __all__ = ["urlparse", "urlunparse", "urljoin", "urldefrag",
            "urlsplit", "urlunsplit", "urlencode", "parse_qs",
            "parse_qsl", "quote", "quote_plus", "quote_from_bytes",
-           "unquote", "unquote_plus", "unquote_to_bytes"]
+           "unquote", "unquote_plus", "unquote_to_bytes",
+           "DefragResult", "ParseResult", "SplitResult",
+           "DefragResultBytes", "ParseResultBytes", "SplitResultBytes"]
 
 # A classification of schemes ('' means apply by default)
 uses_relative = ['ftp', 'http', 'gopher', 'nntp', 'imap',
diff -r ea21b99d002e -r a48e76252952 Misc/NEWS
--- a/Misc/NEWS	Tue Apr 07 12:00:37 2015 -0400
+++ b/Misc/NEWS	Tue Apr 07 19:09:01 2015 +0300
@@ -19,6 +19,10 @@
 Library
 -------
 
+- Issue #23411: Added DefragResult, ParseResult, SplitResult, DefragResultBytes,
+  ParseResultBytes, and SplitResultBytes to urllib.parse.__all__.
+  Patch by Martin Panter.
+
 - Issue #23881: urllib.request.ftpwrapper constructor now closes the socket if
   the FTP connection failed to fix a ResourceWarning.