Preserve synchronous results when async retries are disabled

releasenotes/notes/disabled-async-wrapper-sync-function-7b27bfa4ca214e98.yaml

@@ -0,0 +1,6 @@
+---
+fixes:
+  - |
+    Disabling retries for a synchronous function with an asynchronous sleep
+    callable now returns the function's result instead of raising TypeError
+    when the asynchronous wrapper tries to await it.

tenacity/asyncio/__init__.py

@@ -222,7 +222,7 @@ def wraps(self, fn: t.Callable[P, R]) -> _RetryDecorated[P, R]:
         )
         async def async_wrapped(*args: t.Any, **kwargs: t.Any) -> t.Any:
             if not self.enabled:
-                return await fn(*args, **kwargs)  # type: ignore[misc]
+                return await _utils.wrap_to_async_func(fn)(*args, **kwargs)
             # Always create a copy to prevent overwriting the local contexts when
             # calling the same wrapped functions multiple times in the same stack
             copy = self.copy()

tests/test_asyncio.py

@@ -638,6 +638,21 @@ async def foo() -> None:
 
 
 class TestSyncFunctionWithAsyncSleep(unittest.TestCase):
+    @asynctest
+    async def test_disabled_sync_function_with_async_sleep(self) -> None:
+        mock_sleep = mock.AsyncMock()
+        calls = 0
+
+        @retry(sleep=mock_sleep, enabled=False, retry=retry_if_result(lambda _: True))
+        def sync_function() -> Any:
+            nonlocal calls
+            calls += 1
+            return "ok"
+
+        assert await sync_function() == "ok"
+        assert calls == 1
+        mock_sleep.assert_not_called()
+
     @asynctest
     async def test_sync_function_with_async_sleep(self) -> None:
         """A sync function with an async sleep callable uses AsyncRetrying."""