Fix RuntimeError if user code calls patcher.stopall

Fix #137

Author: nicoddemus

Diff for: CHANGELOG.rst

@@ -1,3 +1,12 @@
+1.10.2
+------
+
+* Fix bug at the end of the test session when a call to ``patch.stopall`` is done explicitly by
+  user code. Thanks `@craiga`_ for the report (`#137`_).
+
+.. _#137: https://github.com/pytest-dev/pytest-mock/issues/137
+.. _@craiga: https://github.com/craiga
+
 1.10.1
 ------
 

Diff for: pytest_mock.py

@@ -288,7 +288,17 @@ def wrap_assert_methods(config):
 
 def unwrap_assert_methods():
     for patcher in _mock_module_patches:
-        patcher.stop()
+        try:
+            patcher.stop()
+        except RuntimeError as e:
+            # a patcher might have been stopped by user code (#137)
+            # so we need to catch this error here and ignore it;
+            # unfortunately there's no public API to check if a patch
+            # has been started, so catching the error it is
+            if text_type(e) == "stop called on unstarted patcher":
+                pass
+            else:
+                raise
     _mock_module_patches[:] = []
     _mock_module_originals.clear()
 

Diff for: test_pytest_mock.py

@@ -610,3 +610,25 @@ def test_assert_called_with_unicode_arguments(mocker):
 
     with pytest.raises(AssertionError):
         stub.assert_called_with(u"lak")
+
+
+def test_plain_stopall(testdir):
+    """Calling patch.stopall() in a test would cause an error during unconfigure (#137)"""
+    testdir.makepyfile(
+        """
+        from unittest.mock import patch
+        import random
+
+        def get_random_number():
+            return random.randint(0, 100)
+
+        def test_get_random_number():
+            patcher = patch("random.randint", lambda x, y: 5)
+            patcher.start()
+            assert get_random_number() == 5
+            patch.stopall()
+    """
+    )
+    result = testdir.runpytest_subprocess()
+    result.stdout.fnmatch_lines("* 1 passed in *")
+    assert "RuntimeError" not in result.stderr.str()