René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"`test_walk_above_recursion_limit` uses absolute limits","articleBody":"# Bug report\r\n\r\n### Bug description:\r\n\r\n`os.walk` and `Path.walk` both have a test verifying that they're not implemented recursively. However, their recursion limits are absolute rather than relative to the current frame. Since the test framework itself already consumes about 30 frames, it only takes a few extra frames to cause a spurious failure. This can easily happen when running the tests within another script, which happens in the Android testbed.\r\n\r\nFor example, if the limit in test_pathlib is reduced from 40 to 35:\r\n\r\n```\r\n======================================================================\r\nERROR: test_walk_above_recursion_limit (test.test_pathlib.test_pathlib.PosixPathTest.test_walk_above_recursion_limit)\r\n----------------------------------------------------------------------\r\nTraceback (most recent call last):\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/__init__.py\", line 241, in __str__\r\n    return self._str\r\n           ^^^^^^^^^\r\nAttributeError: 'PosixPath' object has no attribute '_str'\r\n\r\nDuring handling of the above exception, another exception occurred:\r\n\r\nTraceback (most recent call last):\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/__init__.py\", line 300, in drive\r\n    return self._drv\r\n           ^^^^^^^^^\r\nAttributeError: 'PosixPath' object has no attribute '_drv'\r\n\r\nDuring handling of the above exception, another exception occurred:\r\n\r\nTraceback (most recent call last):\r\n  File \"/Users/msmith/git/python/cpython/Lib/test/test_pathlib/test_pathlib.py\", line 1203, in test_walk_above_recursion_limit\r\n    list(base.walk())\r\n    ~~~~^^^^^^^^^^^^^\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/_abc.py\", line 875, in walk\r\n    scandir_obj = path._scandir()\r\n                  ~~~~~~~~~~~~~^^\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/__init__.py\", line 588, in _scandir\r\n    return os.scandir(self)\r\n           ~~~~~~~~~~^^^^^^\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/__init__.py\", line 177, in __fspath__\r\n    return str(self)\r\n           ~~~^^^^^^\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/__init__.py\", line 243, in __str__\r\n    self._str = self._format_parsed_parts(self.drive, self.root,\r\n                                          ^^^^^^^^^^\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/__init__.py\", line 302, in drive\r\n    self._drv, self._root, self._tail_cached = self._parse_path(self._raw_path)\r\n                                                                ^^^^^^^^^^^^^^\r\n  File \"/Users/msmith/git/python/cpython/Lib/pathlib/__init__.py\", line 293, in _raw_path\r\n    path = self.pathmod.join(*paths)\r\n           ~~~~~~~~~~~~~~~~~^^^^^^^^\r\n  File \"\u003cfrozen posixpath\u003e\", line 77, in join\r\nRecursionError: maximum recursion depth exceeded\r\n```\r\n\r\n\r\n### CPython versions tested on:\r\n\r\nCPython main branch\r\n\r\n### Operating systems tested on:\r\n\r\n* macOS\r\n* Android\r\n\r\n\u003c!-- gh-linked-prs --\u003e\r\n### Linked PRs\r\n* gh-116058\r\n\u003c!-- /gh-linked-prs --\u003e\r\n","author":{"url":"https://github.com/mhsmith","@type":"Person","name":"mhsmith"},"datePublished":"2024-02-28T17:48:20.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/116057/cpython/issues/116057"}