Fix handling recursive symlinks

When pytest was run on a directory containing a recursive symlink it failed
with ELOOP as the library was not able to determine the type of the
direntry:

src/_pytest/main.py:685: in collect
    if not direntry.is_file():
E   OSError: [Errno 40] Too many levels of symbolic links: '/home/florian/proj/pytest/tests/recursive'

This is fixed now by handling ELOOP on Linux and ERROR_CANT_RESOLVE_FILENAME
error on windows.

Fixes #7951

Author: csernazs

changelog/7951.bugfix.rst

@@ -0,0 +1 @@
+Fixed handling of recursive symlinks when collecting tests.

src/_pytest/main.py

@@ -1,5 +1,6 @@
 """Core implementation of the testing process: init, session, runtest loop."""
 import argparse
+import errno
 import fnmatch
 import functools
 import importlib
@@ -676,8 +677,18 @@ def collect(self) -> Iterator[Union[nodes.Item, nodes.Collector]]:
 
                 seen_dirs: Set[py.path.local] = set()
                 for direntry in visit(str(argpath), self._recurse):
-                    if not direntry.is_file():
-                        continue
+                    try:
+                        if not direntry.is_file():
+                            continue
+                    except OSError as err:
+                        # 1921: ERROR_CANT_RESOLVE_FILENAME - fix for broken symlink pointing to itself
+                        if (
+                            getattr(err, "errno", None) == errno.ELOOP
+                            or getattr(err, "winerror", None) == 1921
+                        ):
+                            continue
+                        else:
+                            raise
 
                     path = py.path.local(direntry.path)
                     dirpath = path.dirpath()

testing/test_collection.py

@@ -1404,3 +1404,17 @@ def a(): return 4
     result = testdir.runpytest()
     # Not INTERNAL_ERROR
     assert result.ret == ExitCode.INTERRUPTED
+
+
+def test_does_not_crash_on_recursive_symlink(testdir: Testdir) -> None:
+    """Regression test for an issue around recursive symlinks (#7951)."""
+    testdir.makepyfile(
+        """
+        def test_foo(): assert True
+        """
+    )
+    os.symlink("recursive", str(testdir.tmpdir.join("recursive")))
+    result = testdir.runpytest()
+
+    assert result.ret == ExitCode.OK
+    assert result.parseoutcomes() == {"passed": 1}