[RonnyPfannschmidt]

Requested by Ronny · project thread

🤖 Written by Claude Opus 5.5 via Claude Code for the pytest maintainers; I prompted it, it did the work, I read it.

closes #15281

_find_wrapped_fixture_def (added in #13745) calls inspect.unwrap, whose depth limit is sys.getrecursionlimit(). unittest.mock.call returns a new _Call for every attribute, including __wrapped__, and it got past the mock guard because its _mock_name is None. Once a conftest or library raised the recursion limit (to 100 000, for example), collection followed that chain until memory ran out.

Changes (after review, see f68dbb4):

• The inspect.unwrap call now also stops at mock-like objects, via the _is_mocked check that doctest.py already uses for the same mock.call problem. This replaces the _mock_name guard, which missed mock.call.
• _is_mocked moved from _pytest.doctest to _pytest.compat so both modules can share it.

The first commits replaced inspect.unwrap with a capped hand-written loop. The later commit drops that loop in favour of the smaller change above.

Tests:

• An end-to-end subprocess test using the issue's reproducer, which fails without the fix.
• A unit test that mock.call and Mock() are not unwrapped.

A related, pre-existing case under --doctest-modules is tracked separately.

• Include new tests or update existing tests when applicable.
• Allow maintainers to push and squash when merging my commits.
• If AI agents were used, they are credited in Co-authored-by commit trailers.
• Create a new changelog file in the changelog directory.

🤖 Generated with Claude Code

https://claude.ai/code/session_014GZsRF9tm1CrxGUNz1r4UC

[bluetech]

I haven't reviewed 264846d but it seems pretty complex, and this seems to make it even more complex. Have you considered reverting the change instead? 9.2.0 is still new so I think a revert would be fine.

[RonnyPfannschmidt]

i'd prefer keeping - i checking if we can simplify a bit

[RonnyPfannschmidt]

🤖 Written by Claude Opus 5.5 via Claude Code for the pytest maintainers; I prompted it, it did the work, I read it.

@bluetech I pushed f68dbb4 as a separate commit so the change is visible. It drops the custom __wrapped__ loop and goes back to plain inspect.unwrap. The unwrap now also stops at mock-like objects, using the _is_mocked check that doctest.py already relies on for the same mock.call problem. That check moved to compat so both modules share it. The old _mock_name guard missed mock.call, so this replaces it too.

In fixtures.py the net change against main is now a few lines. The repro from #15281 collects in about 0.1 s with the recursion limit at 100 000. The regression test still fails without the fix.

---

Generated by Claude Code