Change list_jobs() limit to 100

instead of fake "unlimited" which was capped in practice anyway

ref: #677

Author: soxofaan

CHANGELOG.md

@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Eliminate deprecated `utcnow` usage patterns. Introduce `Rfc3339.now_utc()` method (as replacement for deprecated `utcnow()` method) to simplify finding deprecated `utcnow` usage in user code. ([#760](https://github.com/Open-EO/openeo-python-client/issues/760))
+- `Connection.list_jobs()`: change default `limit` to 100 (instead of fake "unlimited" which was arbitrarily capped in practice anyway) ([#677](https://github.com/Open-EO/openeo-python-client/issues/677))
 
 ### Removed
 

openeo/rest/connection.py

@@ -889,11 +889,13 @@ def describe_process(self, id: str, namespace: Optional[str] = None) -> dict:
 
         raise OpenEoClientException("Process does not exist.")
 
-    def list_jobs(self, limit: Union[int, None] = None) -> JobListingResponse:
+    def list_jobs(self, limit: Union[int, None] = 100) -> JobListingResponse:
         """
         Lists (batch) jobs metadata of the authenticated user.
 
-        :param limit: maximum number of jobs to return. Setting this limit enables pagination.
+        :param limit: maximum number of jobs to return (with pagination).
+            Can be set to ``None`` to disable pagination,
+            but note that the backend might still silently cap the listing in practice.
 
         :return: job_list: Dict of all jobs of the user.
 
@@ -903,6 +905,9 @@ def list_jobs(self, limit: Union[int, None] = None) -> JobListingResponse:
         .. versionchanged:: 0.38.0
             Returns a :py:class:`~openeo.rest.models.general.JobListingResponse` object
             instead of simple ``List[dict]``.
+
+        .. versionchanged:: 0.41.0
+            Change default value of ``limit`` to 100 (instead of unlimited).
         """
         # TODO: Parse the result so that Job classes returned?
         # TODO: when pagination is enabled: how to expose link to next page?

tests/rest/test_job.py

@@ -812,7 +812,8 @@ def download_tiff(request, context):
 @pytest.mark.parametrize(
     ["list_jobs_kwargs", "expected_qs"],
     [
-        ({}, {}),
+        ({}, {"limit": ["100"]}),
+        ({"limit": None}, {}),
         ({"limit": 123}, {"limit": ["123"]}),
     ],
 )