fix: don't inject null cursor in _list_generator_raw_responses (#2684)

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

Author: andersfylling

cognite/client/_api_client.py

@@ -313,7 +313,9 @@ async def _list_generator_raw_responses(
             if limit and (n_remaining := limit - total_retrieved) < current_limit:
                 current_limit = n_remaining
 
-            params.update(limit=current_limit, cursor=next_cursor)
+            params["limit"] = current_limit
+            if next_cursor is not None:
+                params["cursor"] = next_cursor
             if method == "GET":
                 res = await self._get(url_path=url_path, params=params, headers=headers, semaphore=semaphore)
             else:

tests/tests_unit/test_api_client.py

@@ -999,6 +999,47 @@ async def test_cognite_client_is_not_set(self, api_client_with_token: APIClient,
         with pytest.raises(AttributeError):
             res2._cognite_client  # type: ignore[attr-defined]
 
+    async def test_list_generator_does_not_send_null_cursor(
+        self, api_client_with_token: APIClient, httpx_mock: HTTPXMock
+    ) -> None:
+        httpx_mock.add_response(
+            method="POST",
+            url=BASE_URL + URL_PATH + "/list",
+            status_code=200,
+            json={"items": [{"x": 1, "y": 2}]},
+        )
+        async for _ in api_client_with_token._list_generator(
+            method="POST",
+            list_cls=SomeResourceListWithClient,
+            resource_cls=SomeResourceWithClient,
+            resource_path=URL_PATH,
+        ):
+            pass
+        body = jsgz_load(httpx_mock.get_requests()[0].content)
+        assert "cursor" not in body
+
+    async def test_list_generator_raw_responses_does_not_send_null_cursor(
+        self, api_client_with_token: APIClient, httpx_mock: HTTPXMock
+    ) -> None:
+        httpx_mock.add_response(
+            method="POST",
+            url=BASE_URL + URL_PATH + "/list",
+            status_code=200,
+            json={"items": [{"x": 1, "y": 2}]},
+        )
+        responses = [
+            r
+            async for r in api_client_with_token._list_generator_raw_responses(
+                method="POST",
+                settings_forcing_raw_response_loading=["include_typing"],
+                resource_path=URL_PATH,
+                chunk_size=api_client_with_token._LIST_LIMIT,
+            )
+        ]
+        assert len(responses) == 1
+        body = jsgz_load(httpx_mock.get_requests()[0].content)
+        assert "cursor" not in body
+
 
 class TestStandardAggregate:
     async def test_standard_aggregate_OK(self, api_client_with_token: APIClient, httpx_mock: HTTPXMock) -> None: