fix(dms): remove instance_name from DMSCredentials and update INTEGRATION_TESTS.md

Author: I559656

docs/INTEGRATION_TESTS.md

@@ -96,6 +96,18 @@ CLOUD_SDK_CFG_DATA_ANONYMIZATION_DEFAULT_DESTINATION_NAME=your-client-certificat
 
 The destination must be configured with `ClientCertificateAuthentication` and reference a certificate bundle containing the client certificate and private key.
 
+### DMS Integration Tests
+
+For DMS (Document Management Service) integration tests, configure the following variables in `.env_integration_tests`:
+
+```bash
+# DMS Configuration
+CLOUD_SDK_CFG_SDM_DEFAULT_URI=https://your-sdm-api-uri-here
+CLOUD_SDK_CFG_SDM_DEFAULT_UAA='{"url":"https://your-auth-url","clientid":"your-client-id","clientsecret":"your-client-secret","identityzone":"your-identity-zone"}'
+```
+
+**Note**: The test fixture automatically onboards test repositories (standard and version-enabled) at session start and cleans them up on teardown. No pre-existing repositories are required.
+
 ### ADMS Integration Tests
 
 For ADMS (Advanced Document Management Service) integration tests, configure the following variables in `.env_integration_tests`:
@@ -145,6 +157,7 @@ uv run pytest tests/core/integration/auditlog -v
 uv run pytest tests/core/integration/data_anonymization -v
 uv run pytest tests/objectstore/integration/ -v
 uv run pytest tests/destination/integration/ -v
+uv run pytest tests/dms/integration/ -v
 uv run pytest tests/agent_memory/integration/ -v
 uv run pytest tests/adms/integration/ -v
 uv run pytest tests/agentgateway/integration/ -v

src/sap_cloud_sdk/dms/client.py

@@ -110,9 +110,7 @@ def __init__(
             read_timeout=read_timeout,
         )
         self._telemetry_source: Optional[Module] = None
-        logger.debug(
-            "DMSClient initialized for instance '%s'", credentials.instance_name
-        )
+        logger.debug("DMSClient initialized")
 
     @record_metrics(Module.DMS, Operation.DMS_ONBOARD_REPOSITORY)
     def onboard_repository(

src/sap_cloud_sdk/dms/config.py

@@ -77,23 +77,19 @@ def _validate_uaa(self) -> None:
                 f"UAA credentials missing required fields: {', '.join(sorted(missing_fields))}"
             )
 
-    def to_credentials(self, instance_name: str) -> DMSCredentials:
+    def to_credentials(self) -> DMSCredentials:
         """Convert the binding data to DMSCredentials.
 
         Parses the UAA JSON and constructs a DMSCredentials object with the necessary information
         for authenticating and connecting to the DMS service.
 
-        Args:
-            instance_name: The logical instance name for these credentials.
-
         Returns:
             DMSCredentials: The credentials extracted from the binding data
         """
         uaa_data: Dict[str, Any] = json.loads(self.uaa)
         token_url = uaa_data["url"].rstrip("/") + "/oauth/token"
 
         return DMSCredentials(
-            instance_name=instance_name,
             uri=self.uri,
             client_id=uaa_data["clientid"],
             client_secret=uaa_data["clientsecret"],
@@ -131,7 +127,7 @@ def load_sdm_config_from_env_or_mount(instance: Optional[str] = None) -> DMSCred
         )
 
         binding.validate()
-        return binding.to_credentials(inst)
+        return binding.to_credentials()
 
     except Exception as e:
         # Rely on the central secret resolver to provide aggregated, generic guidance

src/sap_cloud_sdk/dms/model.py

@@ -30,7 +30,6 @@ def _to_dict_drop_none(obj: Any) -> dict[str, Any]:
 class DMSCredentials:
     """Credentials for authenticating with the DMS service."""
 
-    instance_name: str
     uri: str
     client_id: str
     client_secret: str

tests/dms/unit/test_client_admin.py

@@ -72,7 +72,6 @@ def client():
             mock_http = Mock()
             MockHttp.return_value = mock_http
             creds = DMSCredentials(
-                instance_name="test-instance",
                 uri="https://api.example.com",
                 client_id="test-client",
                 client_secret="test-secret",

tests/dms/unit/test_client_cmis.py

@@ -100,7 +100,6 @@ def _mock_response(data):
 def client():
     """Create a DMSClient with a mocked HttpInvoker."""
     creds = DMSCredentials(
-        instance_name="test-instance",
         uri="https://api.example.com",
         client_id="test-client",
         client_secret="test-secret",