Review comments

Pijukatel · Pijukatel · commit 30ad913f3068 · 2025-05-20T10:03:53.000+02:00
diff --git a/src/apify/_actor.py b/src/apify/_actor.py
@@ -711,8 +711,8 @@ async def start(
             memory_mbytes: Memory limit for the run, in megabytes. By default, the run uses a memory limit specified
                 in the default run configuration for the Actor.
             timeout: Optional timeout for the run, in seconds. By default, the run uses timeout specified in
-                the default run configuration for the Actor. Using `RemainingTime` will set timeout of the other actor
-                to the time remaining from this actor timeout.
+                the default run configuration for the Actor. Using `RemainingTime` will set timeout of the other Actor
+                to the time remaining from this Actor timeout.
             wait_for_finish: The maximum number of seconds the server waits for the run to finish. By default,
                 it is 0, the maximum value is 300.
             webhooks: Optional ad-hoc webhooks (https://docs.apify.com/webhooks/ad-hoc-webhooks) associated with
@@ -740,9 +740,7 @@ async def start(
         elif isinstance(timeout, timedelta):
             actor_start_timeout = timeout
         else:
-            raise ValueError(
-                f'Invalid timeout {timeout!r}: expected `None`, `"RemainingTime"`, or a `timedelta`.'
-            )
+            raise ValueError(f'Invalid timeout {timeout!r}: expected `None`, `"RemainingTime"`, or a `timedelta`.')
 
         api_result = await client.actor(actor_id).start(
             run_input=run_input,
@@ -761,7 +759,11 @@ def _get_remaining_time(self) -> timedelta | None:
         if self.is_at_home() and self.configuration.timeout_at:
             return self.configuration.timeout_at - datetime.now(tz=timezone.utc)
 
-        self.log.warning('Using `RemainingTime` argument for timeout outside of the Apify platform. Returning `None`')
+        self.log.warning(
+            'Returning `None` instead of remaining time. Using `RemainingTime` argument is only possible when the Actor'
+            ' is running on the Apifyplatform and when the timeout for the Actor run is set. '
+            f'{self.is_at_home()=}, {self.configuration.timeout_at=}'
+        )
         return None
 
     async def abort(
@@ -825,8 +827,8 @@ async def call(
             memory_mbytes: Memory limit for the run, in megabytes. By default, the run uses a memory limit specified
                 in the default run configuration for the Actor.
             timeout: Optional timeout for the run, in seconds. By default, the run uses timeout specified in
-                the default run configuration for the Actor. Using `RemainingTime` will set timeout of the other actor
-                to the time remaining from this actor timeout.
+                the default run configuration for the Actor. Using `RemainingTime` will set timeout of the other Actor
+                to the time remaining from this Actor timeout.
             webhooks: Optional webhooks (https://docs.apify.com/webhooks) associated with the Actor run, which can
                 be used to receive a notification, e.g. when the Actor finished or failed. If you already have
                 a webhook set up for the Actor, you do not have to add it again here.
@@ -849,12 +851,12 @@ async def call(
 
         if timeout == 'RemainingTime':
             actor_call_timeout = self._get_remaining_time()
-        elif isinstance(timeout, str):
-            raise ValueError(
-                f'`timeout` can be `None`, `RemainingTime` literal or `timedelta` instance, but is {timeout=}'
-            )
-        else:
+        elif timeout is None:
+            actor_call_timeout = None
+        elif isinstance(timeout, timedelta):
             actor_call_timeout = timeout
+        else:
+            raise ValueError(f'Invalid timeout {timeout!r}: expected `None`, `"RemainingTime"`, or a `timedelta`.')
 
         api_result = await client.actor(actor_id).call(
             run_input=run_input,
diff --git a/tests/integration/test_actor_call_timeouts.py b/tests/integration/test_actor_call_timeouts.py
@@ -13,12 +13,17 @@ async def test_actor_start_remaining_timeout(
     make_actor: MakeActorFunction,
     run_actor: RunActorFunction,
 ) -> None:
+    """In this test one Actor starts itself again and checks that the timeout is correctly set on the second Actor run.
+
+    This timeout should be the remaining time of the first Actor run."""
+
     async def main() -> None:
         from datetime import datetime, timezone
 
         async with Actor:
             actor_input = (await Actor.get_input()) or {}
             if actor_input.get('called_from_another_actor', False) is True:
+                # If this Actor run was started with specific arguement (the second actor run), return immediately.
                 return
 
             # Start another run of this actor with timeout set to the time remaining in this actor run
@@ -53,16 +58,19 @@ async def test_actor_call_remaining_timeout(
     make_actor: MakeActorFunction,
     run_actor: RunActorFunction,
 ) -> None:
+    """In this test one Actor starts itself again and checks that the timeout is correctly set on the second Actor run.
+
+    This timeout should be the remaining time of the first Actor run."""
+
     async def main() -> None:
         from datetime import datetime, timezone
 
         async with Actor:
             actor_input = (await Actor.get_input()) or {}
             if actor_input.get('called_from_another_actor', False) is True:
+                # If this Actor run was started with specific arguement (the second actor run), return immediately.
                 return
 
-            await asyncio.sleep(1)
-
             # Start another run of this actor with timeout set to the time remaining in this actor run
             other_run_data = await Actor.call(
                 actor_id=Actor.configuration.actor_id or '',
