From: Nikita Sobolev <mail@sobolevn.me>
Date: Fri, 24 Dec 2021 08:07:50 +0000 (+0300)
Subject: bpo-46150: ensure `fakeuser` does not exist in `PosixPathTest.test_expanduser` (GH... 
X-Git-Tag: v3.11.0a4~145
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=b8de8b7039cee47465b2af9950b0b9ed2d3f2903;p=thirdparty%2FPython%2Fcpython.git

bpo-46150: ensure `fakeuser` does not exist in `PosixPathTest.test_expanduser` (GH-30240)

Ensure `fakeuser` does not exist in `PosixPathTest.test_expanduser`
---

diff --git a/Lib/test/test_pathlib.py b/Lib/test/test_pathlib.py
index 2268d36b0639..555c7ee795bd 100644
--- a/Lib/test/test_pathlib.py
+++ b/Lib/test/test_pathlib.py
@@ -2558,13 +2558,21 @@ class PosixPathTest(_BasePathTest, unittest.TestCase):
             othername = username
             otherhome = userhome
 
+        fakename = 'fakeuser'
+        # This user can theoretically exist on a test runner. Create unique name:
+        try:
+            while pwd.getpwnam(fakename):
+                fakename += '1'
+        except KeyError:
+            pass  # Non-existent name found
+
         p1 = P('~/Documents')
-        p2 = P('~' + username + '/Documents')
-        p3 = P('~' + othername + '/Documents')
-        p4 = P('../~' + username + '/Documents')
-        p5 = P('/~' + username + '/Documents')
+        p2 = P(f'~{username}/Documents')
+        p3 = P(f'~{othername}/Documents')
+        p4 = P(f'../~{username}/Documents')
+        p5 = P(f'/~{username}/Documents')
         p6 = P('')
-        p7 = P('~fake800813user/Documents')
+        p7 = P(f'~{fakename}/Documents')
 
         with os_helper.EnvironmentVarGuard() as env:
             env.pop('HOME', None)
diff --git a/Misc/NEWS.d/next/Tests/2021-12-23-13-42-15.bpo-46150.RhtADs.rst b/Misc/NEWS.d/next/Tests/2021-12-23-13-42-15.bpo-46150.RhtADs.rst
new file mode 100644
index 000000000000..8ef9cd9b4a59
--- /dev/null
+++ b/Misc/NEWS.d/next/Tests/2021-12-23-13-42-15.bpo-46150.RhtADs.rst
@@ -0,0 +1,2 @@
+Now ``fakename`` in ``test_pathlib.PosixPathTest.test_expanduser`` is checked
+to be non-existent.