Merge pull request #3584 from pre-commit/exitstack

use ExitStack instead of start + stop

tests/main_test.py

@@ -1,6 +1,7 @@
 from __future__ import annotations
 
 import argparse
+import contextlib
 import os.path
 from unittest import mock
 
@@ -97,11 +98,9 @@ CMDS = tuple(fn.replace('_', '-') for fn in FNS)
 
 @pytest.fixture
 def mock_commands():
-    mcks = {fn: mock.patch.object(main, fn).start() for fn in FNS}
+    with contextlib.ExitStack() as ctx:
-    ret = auto_namedtuple(**mcks)
+        mcks = {f: ctx.enter_context(mock.patch.object(main, f)) for f in FNS}
-    yield ret
+        yield auto_namedtuple(**mcks)
-    for mck in ret:
-        mck.stop()
 
 
 @pytest.fixture