diff options
| author | Jörg Stucke <joerg.stucke@fkie.fraunhofer.de> | 2019-05-21 19:44:40 +0200 |
|---|---|---|
| committer | Antoine Pitrou <antoine@python.org> | 2019-05-21 19:44:40 +0200 |
| commit | d5c120f7eb6f2a9cdab282a5d588afed307a23df (patch) | |
| tree | 60b1882411bd299e6d2332185a099ba3ee3af56a /Lib/test/test_pathlib.py | |
| parent | ccb7ca728e09b307f9e9fd36ec40353137e68a3b (diff) | |
| download | cpython-git-d5c120f7eb6f2a9cdab282a5d588afed307a23df.tar.gz | |
bpo-36035: fix Path.rglob for broken links (GH-11988)
Links creating an infinite symlink loop would raise an exception.
Diffstat (limited to 'Lib/test/test_pathlib.py')
| -rw-r--r-- | Lib/test/test_pathlib.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/Lib/test/test_pathlib.py b/Lib/test/test_pathlib.py index caad1c2387..069467a845 100644 --- a/Lib/test/test_pathlib.py +++ b/Lib/test/test_pathlib.py @@ -1221,7 +1221,8 @@ class _BasePathTest(object): # |-- dirE # No permissions # |-- fileA # |-- linkA -> fileA - # `-- linkB -> dirB + # |-- linkB -> dirB + # `-- brokenLinkLoop -> brokenLinkLoop # def setUp(self): @@ -1252,6 +1253,8 @@ class _BasePathTest(object): self.dirlink(os.path.join('..', 'dirB'), join('dirA', 'linkC')) # This one goes upwards, creating a loop. self.dirlink(os.path.join('..', 'dirB'), join('dirB', 'linkD')) + # Broken symlink (pointing to itself). + os.symlink('brokenLinkLoop', join('brokenLinkLoop')) if os.name == 'nt': # Workaround for http://bugs.python.org/issue13772. @@ -1384,7 +1387,7 @@ class _BasePathTest(object): paths = set(it) expected = ['dirA', 'dirB', 'dirC', 'dirE', 'fileA'] if support.can_symlink(): - expected += ['linkA', 'linkB', 'brokenLink'] + expected += ['linkA', 'linkB', 'brokenLink', 'brokenLinkLoop'] self.assertEqual(paths, { P(BASE, q) for q in expected }) @support.skip_unless_symlink @@ -1465,6 +1468,7 @@ class _BasePathTest(object): 'fileA', 'linkA', 'linkB', + 'brokenLinkLoop', } self.assertEqual(given, {p / x for x in expect}) |