fix: Return False (skip) when no datetime-based cursors found for retention check

Co-Authored-By: alfredo.garcia@airbyte.io <freddy.garcia7.fg@gmail.com>

Author: devin-ai-integration[bot]

airbyte_cdk/sources/declarative/parsers/model_to_component_factory.py

@@ -3621,10 +3621,14 @@ def _is_cursor_older_than_retention_period(
         if "state_type" in stream_state or "slices" in stream_state:
             return False
 
+        datetime_cursor_sources = [
+            s for s in incremental_sync_sources if isinstance(s, DatetimeBasedCursorModel)
+        ]
+        if not datetime_cursor_sources:
+            return False
+
         cursor_datetime: datetime.datetime | None = None
-        for incremental_sync in incremental_sync_sources:
-            if not isinstance(incremental_sync, DatetimeBasedCursorModel):
-                continue
+        for incremental_sync in datetime_cursor_sources:
             cursor = self._create_cursor_for_age_check(incremental_sync, config)
             cursor_datetime = cursor.get_cursor_datetime_from_state(stream_state)
             if cursor_datetime is not None:

unit_tests/sources/declarative/test_state_delegating_stream.py

@@ -533,7 +533,7 @@ def test_cursor_age_validation_skips_incrementing_count_cursor():
 
     with HttpMocker() as http_mocker:
         http_mocker.get(
-            HttpRequest(url="https://api.test.com/items"),
+            HttpRequest(url="https://api.test.com/items_with_filtration"),
             HttpResponse(body=json.dumps([{"id": 101, "updated_at": "2024-07-14"}])),
         )