diff options
| author | Jacob Walls <jacobtylerwalls@gmail.com> | 2023-11-27 18:05:55 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-11-27 18:05:55 (GMT) |
| commit | 99a73c3465a45fe57cac01a917fc50e0743b5964 (patch) | |
| tree | be3c8979398f0e81a6e9d5bccb7b3d715f1cb418 /Lib/test/test_getpass.py | |
| parent | 936c503a442ee062c837e334f237796554c792ff (diff) | |
| download | cpython-99a73c3465a45fe57cac01a917fc50e0743b5964.zip cpython-99a73c3465a45fe57cac01a917fc50e0743b5964.tar.gz cpython-99a73c3465a45fe57cac01a917fc50e0743b5964.tar.bz2 | |
gh-76912: Raise OSError from any failure in getpass.getuser() (#29739)
* bpo-32731: Raise OSError from any failure in getpass.getuser()
Previously, if the username was not set in certain environment variables, ImportError escaped on Windows systems, and it was possible for KeyError to escape on other systems if getpwuid() failed.
Diffstat (limited to 'Lib/test/test_getpass.py')
| -rw-r--r-- | Lib/test/test_getpass.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Lib/test/test_getpass.py b/Lib/test/test_getpass.py index 98ecec9..80dda2c 100644 --- a/Lib/test/test_getpass.py +++ b/Lib/test/test_getpass.py @@ -26,7 +26,7 @@ class GetpassGetuserTest(unittest.TestCase): environ.get.return_value = None try: getpass.getuser() - except ImportError: # in case there's no pwd module + except OSError: # in case there's no pwd module pass except KeyError: # current user has no pwd entry @@ -47,7 +47,7 @@ class GetpassGetuserTest(unittest.TestCase): getpass.getuser()) getpw.assert_called_once_with(42) else: - self.assertRaises(ImportError, getpass.getuser) + self.assertRaises(OSError, getpass.getuser) class GetpassRawinputTest(unittest.TestCase): |
