fix(adms): narrow dict type in get_all to satisfy ty; bump to 0.25.1

1. _document_api.py: type rel_params as dict[str, str | int] (not bare
   dict) and wrap the get() result in str() before calling .split() — ty
   couldn't prove the value was str when the dict held int values.
   Applies to both sync and async get_all.

2. tests/dms/integration: remove two now-unused # ty: ignore directives
   (conftest.py:17, test_dms_bdd.py:131) that ty 0.25 no longer needs.
   These were flagged as warnings-turned-errors after the upstream merge
   bumped ty.

3. pyproject.toml / uv.lock: bump version 0.25.0 → 0.25.1 to satisfy the
   CI version-bump gate (src/ files changed, version must increase).

Author: adwitiyasushant

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sap-cloud-sdk"
-version = "0.25.0"
+version = "0.25.1"
 description = "SAP Cloud SDK for Python"
 readme = "README.md"
 license = "Apache-2.0"

src/sap_cloud_sdk/adms/_document_api.py

@@ -58,10 +58,10 @@ def get_all(
             ordered by first occurrence across relations.
         """
         # Build RelationQueryOptions that always includes Document in $expand.
-        rel_params: dict = {}
+        rel_params: dict[str, str | int] = {}
         if options:
             rel_params = options.to_query_params()
-        existing_expand = rel_params.get("$expand", "")
+        existing_expand = str(rel_params.get("$expand", ""))
         if existing_expand:
             if "Document" not in existing_expand.split(","):
                 rel_params["$expand"] = existing_expand + ",Document"
@@ -261,10 +261,10 @@ async def get_all(
         options: DocumentQueryOptions | None = None,
     ) -> list[Document]:
         """Async variant of :meth:`_DocumentApi.get_all` — same semantics."""
-        rel_params: dict = {}
+        rel_params: dict[str, str | int] = {}
         if options:
             rel_params = options.to_query_params()
-        existing_expand = rel_params.get("$expand", "")
+        existing_expand = str(rel_params.get("$expand", ""))
         if existing_expand:
             if "Document" not in existing_expand.split(","):
                 rel_params["$expand"] = existing_expand + ",Document"

tests/dms/integration/conftest.py

@@ -14,7 +14,7 @@ def dms_client():
         client = create_client(instance="default")
         return client
     except Exception as e:
-        pytest.skip(f"DMS integration tests require credentials: {e}")  # ty: ignore[invalid-argument-type, too-many-positional-arguments]
+        pytest.skip(f"DMS integration tests require credentials: {e}")
 
 
 def _setup_cloud_mode():

tests/dms/integration/test_dms_bdd.py

@@ -128,7 +128,7 @@ def select_version_repo(context: DMSTestContext, dms_client: DMSClient):
             version_repo = r
             break
     if version_repo is None:
-        pytest.skip("No version-enabled repository available")  # ty: ignore[invalid-argument-type, too-many-positional-arguments]
+        pytest.skip("No version-enabled repository available")
     context.repo = version_repo
     context.repo_id = version_repo.id