unittest.mock test and coverage fixup (#130787)

* Mark functions that will never be called with # pragma: no cover

* Fix testpatch.PatchTest.test_exit_idempotent

.stop() and __exit__ have subtly different code paths, so to really test __exit__ idempotency, we need to call it specifically twice.

Backports: 04091c083340dde7d4eeb6d945c70f3b37d88f85
Signed-off-by: Chris Withers <chris@withers.org>

Author: cjw296

mock/tests/testhelpers.py

@@ -1097,7 +1097,7 @@ def test_dataclass_with_method(self):
         class WithMethod:
             a: int
             def b(self) -> int:
-                return 1
+                return 1  # pragma: no cover
 
         for mock in [
             create_autospec(WithMethod, instance=True),

mock/tests/testmock.py

@@ -317,7 +317,7 @@ def test_explicit_return_value_even_if_mock_wraps_object(self):
         passed to the wrapped object and the return_value is returned instead.
         """
         def my_func():
-            return None
+            return None  # pragma: no cover
         func_mock = create_autospec(spec=my_func, wraps=my_func)
         return_value = "explicit return value"
         func_mock.return_value = return_value

mock/tests/testpatch.py

@@ -746,7 +746,7 @@ def test_stop_idempotent(self):
     def test_exit_idempotent(self):
         patcher = patch(foo_name, 'bar', 3)
         with patcher:
-            patcher.stop()
+            patcher.__exit__(None, None, None)
 
 
     def test_second_start_failure(self):