As per the comment in pluggy._MultiCall we should be able to drop the line here. Grepping the pytest sources reveals that only a few spots still reference it.
This change also allows us to remove the recursion induced by _wrapped_call() getting passed _MultiCall.execute. This means we can remove _wrapped_call and simply loop through hookwrapper pre and post yield calls at the beginning and end of execute(). I'd actually be interested to see if contextlib.contextmanager wrapping could be used directly for this.
As per the comment in
pluggy._MultiCallwe should be able to drop the line here. Grepping thepytestsources reveals that only a few spots still reference it.This change also allows us to remove the recursion induced by
_wrapped_call()getting passed_MultiCall.execute. This means we can remove_wrapped_calland simply loop through hookwrapper pre and postyieldcalls at the beginning and end ofexecute(). I'd actually be interested to see ifcontextlib.contextmanagerwrapping could be used directly for this.