fix: prevent infinite loop in _wait_for_finish on persistent 404s (#619)

## Summary

- `_wait_for_finish` could spin indefinitely when polling a non-existent
job
- `seconds_elapsed` was only updated inside the `try` block (on
success), so on persistent 404 errors the timeout check `seconds_elapsed
> 3` always evaluated as `0 > 3 = False`
- Moved the `seconds_elapsed` update into a `finally` block so it is
always updated regardless of success or failure (both sync and async
variants)

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: vdusek

src/apify_client/clients/base/actor_job_base_client.py

@@ -39,7 +39,6 @@ def _wait_for_finish(self, wait_secs: int | None = None) -> dict | None:
                 )
                 job = parse_date_fields(pluck_data(response.json()))
 
-                seconds_elapsed = math.floor((datetime.now(timezone.utc) - started_at).total_seconds())
                 if ActorJobStatus(job['status']).is_terminal or (
                     wait_secs is not None and seconds_elapsed >= wait_secs
                 ):
@@ -57,6 +56,9 @@ def _wait_for_finish(self, wait_secs: int | None = None) -> dict | None:
                 if seconds_elapsed > DEFAULT_WAIT_WHEN_JOB_NOT_EXIST_SEC:
                     return None
 
+            finally:
+                seconds_elapsed = math.floor((datetime.now(timezone.utc) - started_at).total_seconds())
+
             # It might take some time for database replicas to get up-to-date so sleep a bit before retrying
             time.sleep(0.25)
 
@@ -93,7 +95,6 @@ async def _wait_for_finish(self, wait_secs: int | None = None) -> dict | None:
                 )
                 job = parse_date_fields(pluck_data(response.json()))
 
-                seconds_elapsed = math.floor((datetime.now(timezone.utc) - started_at).total_seconds())
                 if ActorJobStatus(job['status']).is_terminal or (
                     wait_secs is not None and seconds_elapsed >= wait_secs
                 ):
@@ -111,6 +112,9 @@ async def _wait_for_finish(self, wait_secs: int | None = None) -> dict | None:
                 if seconds_elapsed > DEFAULT_WAIT_WHEN_JOB_NOT_EXIST_SEC:
                     return None
 
+            finally:
+                seconds_elapsed = math.floor((datetime.now(timezone.utc) - started_at).total_seconds())
+
             # It might take some time for database replicas to get up-to-date so sleep a bit before retrying
             await asyncio.sleep(0.25)