Docs + tests fixes

Author: barneygale

Doc/library/pathlib.rst

@@ -872,10 +872,9 @@ conforming to :rfc:`8089`.
    .. versionadded:: 3.13
 
    .. versionchanged:: next
-      If a URL authority matches the current hostname, it is discarded.
-      Otherwise, if the authority isn't empty or ``localhost``, then on
-      Windows a UNC path is returned (as before), and on other platforms a
-      :exc:`ValueError` is raised.
+      The URL authority is discarded if it matches the machine hostname.
+      Otherwise, on Windows a UNC path is returned (as before), and on other
+      platforms a :exc:`ValueError` is raised.
 
 
 .. method:: Path.as_uri()

Doc/library/urllib.request.rst

@@ -187,9 +187,9 @@ The :mod:`urllib.request` module defines the following functions:
 
    The URL authority is discarded if it empty, ``localhost``, or the current
    hostname. Otherwise, if *resolve_netloc* is set to true, the authority is
-   resolved to an IP address using :func:`socket.gethostbyname`, and discarded
-   if it matches a local IP. If the authority is still unhandled, then on
-   Windows a UNC path is returned, and on other platforms a
+   resolved using :func:`socket.gethostbyname`, and discarded if it matches a
+   local IP address. If the authority is still unhandled, then on Windows a
+   UNC path is returned, and on other platforms a
    :exc:`~urllib.error.URLError` is raised.
 
    This example shows the function being used on Windows::

Lib/test/test_urllib.py

@@ -1572,7 +1572,6 @@ def test_url2pathname_require_scheme_errors(self):
     def test_url2pathname_resolve_netloc(self):
         fn = urllib.request.url2pathname
         sep = os.path.sep
-        self.assertRaises(urllib.error.URLError, fn, '//127.0.0.1/foo/bar')
         self.assertEqual(fn('//127.0.0.1/foo/bar', resolve_netloc=True), f'{sep}foo{sep}bar')
         self.assertEqual(fn(f'//{socket.gethostname()}/foo/bar'), f'{sep}foo{sep}bar')
         self.assertEqual(fn(f'//{socket.gethostname()}/foo/bar', resolve_netloc=True), f'{sep}foo{sep}bar')
@@ -1607,6 +1606,7 @@ def test_url2pathname_win(self):
         self.assertEqual(fn('//server/path/to/file'), '\\\\server\\path\\to\\file')
         self.assertEqual(fn('////server/path/to/file'), '\\\\server\\path\\to\\file')
         self.assertEqual(fn('/////server/path/to/file'), '\\\\server\\path\\to\\file')
+        self.assertEqual(fn('//127.0.0.1/path/to/file'), '\\\\127.0.0.1\\path\\to\\file')
         # Localhost paths
         self.assertEqual(fn('//localhost/C:/path/to/file'), 'C:\\path\\to\\file')
         self.assertEqual(fn('//localhost/C|/path/to/file'), 'C:\\path\\to\\file')
@@ -1631,6 +1631,7 @@ def test_url2pathname_posix(self):
         self.assertRaises(urllib.error.URLError, fn, '//:80/foo/bar')
         self.assertRaises(urllib.error.URLError, fn, '//:/foo/bar')
         self.assertRaises(urllib.error.URLError, fn, '//c:80/foo/bar')
+        self.assertRaises(urllib.error.URLError, fn, '//127.0.0.1/foo/bar')
 
     @unittest.skipUnless(os_helper.FS_NONASCII, 'need os_helper.FS_NONASCII')
     def test_url2pathname_nonascii(self):