fix: Fixed double wrapping of inherited Hypothesis tests.

Pytest-asyncio identifies Hypothesis test cases by their `is_hypothesis_test` flag. When setting up an async Hypothesis test pytest-asyncio replaces the function's `hypothesis.inner_test` attribute. The the top level function never changes.

When a Hypothesis test case is defined in a base class and inherited by subclasses, the test is collected in each subclass. Since the top-level Hypothesis test never changes, its inner test will be wrapped multiple times.

Double wrapping leads to execution errors caused by stale (closed) event loops in all test executions after the first.

This change adds an `original_test_function` attribute to the async function wrapper, in order to keep track of the original Hypothesis test. When re-wrapping would occur in subclasses pytest-asyncio wraps the original test function rather than the wrapper function.

Closes #231

Signed-off-by: Michael Seifert <[email protected]>

Author: seifertm

README.rst

@@ -175,6 +175,7 @@ Changelog
 ~~~~~~~~~~~~~~~~~~~
 - `pytest-asyncio` no longer alters existing event loop policies. `#168 <https://github.com/pytest-dev/pytest-asyncio/issues/168>`_, `#188 <https://github.com/pytest-dev/pytest-asyncio/issues/168>`_
 - Drop support for Python 3.6
+- Fixed an issue that prevented inherited asynchronous Hypothesis tests from working. `#231 <https://github.com/pytest-dev/pytest-asyncio/issues/231>`_
 
 0.16.0 (2021-10-16)
 ~~~~~~~~~~~~~~~~~~~

pytest_asyncio/plugin.py

@@ -176,7 +176,7 @@ def wrap_in_sync(func, _loop):
     # if the function is already wrapped, we rewrap using the original one
     # not using __wrapped__ because the original function may already be
     # a wrapped one
-    if hasattr(func, '_raw_test_func'):
+    if hasattr(func, "_raw_test_func"):
         func = func._raw_test_func
 
     @functools.wraps(func)

tests/hypothesis/test_inherited_test.py

@@ -0,0 +1,22 @@
+import hypothesis.strategies as st
+from hypothesis import given
+import pytest
+
+
+class BaseClass:
+    @pytest.mark.asyncio
+    @given(value=st.integers())
+    async def test_hypothesis(self, value: int) -> None:
+        assert True
+
+
+class TestOne(BaseClass):
+    """During the first execution the Hypothesis test is wrapped in a synchronous function."""
+
+    pass
+
+
+class TestTwo(BaseClass):
+    """Execute the test a second time to ensure that the test receives a fresh event loop."""
+
+    pass