Merge branch 'fix/skip-sut-coroutines'

* fix/skip-sut-coroutines:
  Skip SUT coroutines instead of aborting the whole module

Author: LuKrO2011

src/pynguin/analyses/module.py

@@ -82,7 +82,6 @@
 from pynguin.utils.exceptions import (
     ConstraintValidationError,
     ConstructionFailedException,
-    CoroutineFoundException,
 )
 from pynguin.utils.generic.genericaccessibleobject import (
     GenericAccessibleObject,
@@ -1407,10 +1406,16 @@ def __analyse_function(
         LOGGER.debug("Skipping function %s from analysis", func_name)
         return
     if inspect.iscoroutinefunction(func) or inspect.isasyncgenfunction(func):
+        # Pynguin cannot execute coroutines (see issue #62 / MR !59), so we skip
+        # them instead of aborting the whole module and still test its
+        # synchronous members.
         if add_to_test:
-            raise CoroutineFoundException("Found coroutine in SUT: %s", func_name)
-        # Coroutine outside the SUT are not problematic, just exclude them.
-        LOGGER.debug("Skipping coroutine %s outside of SUT", func_name)
+            LOGGER.warning(
+                "Skipping coroutine %s: Pynguin only tests the non-async parts of this module.",
+                func_name,
+            )
+        else:
+            LOGGER.debug("Skipping coroutine %s outside of SUT", func_name)
         return
 
     LOGGER.debug("Analysing function %s", func_name)
@@ -1610,10 +1615,16 @@ def __analyse_method(
         LOGGER.debug("Skipping method %s from analysis", method_name)
         return
     if inspect.iscoroutinefunction(method) or inspect.isasyncgenfunction(method):
+        # Pynguin cannot execute coroutines (see issue #62 / MR !59), so we skip
+        # them instead of aborting the whole module and still test its
+        # synchronous members.
         if add_to_test:
-            raise CoroutineFoundException("Found coroutine in SUT: %s", method_name)
-        # Coroutine outside the SUT are not problematic, just exclude them.
-        LOGGER.debug("Skipping coroutine %s outside of SUT", method_name)
+            LOGGER.warning(
+                "Skipping coroutine %s: Pynguin only tests the non-async parts of this module.",
+                method_name,
+            )
+        else:
+            LOGGER.debug("Skipping coroutine %s outside of SUT", method_name)
         return
 
     LOGGER.debug("Analysing method %s.%s", type_info.full_name, method_name)

src/pynguin/generator.py

@@ -81,7 +81,7 @@
     TestCaseExecutor,
 )
 from pynguin.utils import randomness
-from pynguin.utils.exceptions import ConfigurationException, CoroutineFoundException
+from pynguin.utils.exceptions import ConfigurationException
 from pynguin.utils.llm import LLM, LLMProvider, extract_code
 from pynguin.utils.report import (
     get_coverage_report,
@@ -164,11 +164,6 @@ def _setup_test_cluster() -> ModuleTestCluster | None:
             ex.name,
         )
         return None
-    except CoroutineFoundException as ex:
-        _LOGGER.exception(
-            "Pynguin does not support test generation for coroutines (async def): %s", ex
-        )
-        return None
 
     if test_cluster.num_accessible_objects_under_test() == 0:
         _LOGGER.error("SUT contains nothing we can test.")

tests/analyses/test_module.py

@@ -29,7 +29,7 @@
 from pynguin.analyses.typesystem import ANY, AnyType, ProperType, TypeInfo, UnionType
 from pynguin.configuration import ElementVisibility
 from pynguin.ga.operators.selection import RandomSelection, RankSelection
-from pynguin.utils.exceptions import ConstructionFailedException, CoroutineFoundException
+from pynguin.utils.exceptions import ConstructionFailedException
 from pynguin.utils.generic.genericaccessibleobject import (
     GenericAccessibleObject,
     GenericCallableAccessibleObject,
@@ -619,9 +619,18 @@ def test_enums():
     "module_name",
     ["async_func", "async_gen", "async_class_gen", "async_class_method"],
 )
-def test_analyse_async_function_or_method(module_name):
-    with pytest.raises(CoroutineFoundException):
-        generate_test_cluster(f"tests.fixtures.cluster.{module_name}")
+def test_analyse_async_function_or_method_does_not_abort(module_name):
+    # A coroutine in the SUT must be skipped, not abort the whole module.
+    cluster = generate_test_cluster(f"tests.fixtures.cluster.{module_name}")
+    assert cluster is not None
+
+
+def test_analyse_mixed_async_and_sync_keeps_sync():
+    cluster = generate_test_cluster("tests.fixtures.cluster.async_and_sync")
+    names = {getattr(obj, "function_name", None) for obj in cluster.accessible_objects_under_test}
+    # The synchronous function stays testable, the coroutine is skipped.
+    assert "sync_bar" in names
+    assert "async_foo" not in names
 
 
 def test_analyse_async_as_dependency():

tests/fixtures/cluster/async_and_sync.py

@@ -0,0 +1,12 @@
+#  This file is part of Pynguin.
+#
+#  SPDX-FileCopyrightText: 2019–2026 Pynguin Contributors
+#
+#  SPDX-License-Identifier: MIT
+#
+async def async_foo():
+    return 45
+
+
+def sync_bar(x):
+    return x + 1