fix(agentgateway): align gtids rename across fragments, tests and docstrings

- Fix ConnectedSystem construction to use global_tenant_id (not gtids)
- Rename list_mcp_fragments param global_tenant_ids → gtids in tests
- Replace GTID_LABEL_KEY usage with string literal in test_lob.py
- Update test names and MCPToolFilter usages to use gtids field
- Fix agw_client.py docstring example to use gtids instead of global_tenant_ids

Author: I542102

src/sap_cloud_sdk/agentgateway/_fragments.py

@@ -37,11 +37,11 @@
 def _list_fragments_by_label(
     label: FragmentLabel,
     tenant_subdomain: str,
-    global_tenant_ids: list[str] | None = None,
+    gtids: list[str] | None = None,
 ) -> list:
     filter_labels = [Label(key=LABEL_KEY, values=[label.value])]
-    if global_tenant_ids:
-        filter_labels.append(Label(key=_LABEL_GTID, values=global_tenant_ids))
+    if gtids:
+        filter_labels.append(Label(key=_LABEL_GTID, values=gtids))
     client = create_fragment_client(
         instance=_DESTINATION_INSTANCE,
         _telemetry_source=Module.AGENTGATEWAY,
@@ -54,13 +54,13 @@ def _list_fragments_by_label(
 
 def list_mcp_fragments(
     tenant_subdomain: str,
-    global_tenant_ids: list[str] | None = None,
+    gtids: list[str] | None = None,
 ) -> list:
     """List destination fragments with MCP server label.
 
     Args:
         tenant_subdomain: Tenant subdomain for multi-tenant lookup.
-        global_tenant_ids: Optional list of global tenant IDs of integrated
+        gtids: Optional list of global tenant IDs of integrated
             systems to filter by. When set, only fragments whose
             ``sap-managed-runtime-gtid`` label matches one of these values are
             returned (filter is applied server-side by the Destination Service).
@@ -71,7 +71,7 @@ def list_mcp_fragments(
     """
     logger.debug("Fetching MCP fragments for tenant '%s'", tenant_subdomain)
     return _list_fragments_by_label(
-        FragmentLabel.MCP, tenant_subdomain, global_tenant_ids
+        FragmentLabel.MCP, tenant_subdomain, gtids
     )
 
 
@@ -136,9 +136,6 @@ def get_ias_user_fragment_name(tenant_subdomain: str) -> str:
     return fragments[0].name
 
 
-# Backward-compatible alias
-ActiveIntegration = ConnectedSystem
-
 
 def _list_active_integrations(tenant_subdomain: str) -> list[ConnectedSystem]:
     """List all active backend system integrations for the given tenant.

src/sap_cloud_sdk/agentgateway/agw_client.py

@@ -403,7 +403,7 @@ async def list_mcp_tools(
                 filter=MCPToolFilter(
                     names=["get-sales-order"],
                     ord_ids=["sap.s4:apiAccess:salesOrder:v1"],
-                    global_tenant_ids=["<gtid>"],
+                    gtids=["<gtid>"],
                 )
             )
             ```

tests/agentgateway/unit/test_agw_client.py

@@ -448,8 +448,8 @@ async def test_with_callable_tenant(self):
             )
 
     @pytest.mark.asyncio
-    async def test_forwards_global_tenant_ids_from_filter_to_lob(self):
-        """MCPToolFilter.global_tenant_ids should reach get_mcp_tools_lob."""
+    async def test_forwards_gtids_from_filter_to_lob(self):
+        """MCPToolFilter.gtids should reach get_mcp_tools_lob."""
         with (
             patch(
                 "sap_cloud_sdk.agentgateway.agw_client.detect_customer_agent_credentials",
@@ -473,14 +473,14 @@ async def test_forwards_global_tenant_ids_from_filter_to_lob(self):
             agw_client = create_client(tenant_subdomain="my-tenant")
 
             await agw_client.list_mcp_tools(
-                filter=MCPToolFilter(global_tenant_ids=["gtid-a", "gtid-b"]),
+                filter=MCPToolFilter(gtids=["gtid-a", "gtid-b"]),
             )
 
             mock_lob.assert_called_once_with(
                 "my-tenant",
                 "system-token",
                 60.0,
-                filter=MCPToolFilter(global_tenant_ids=["gtid-a", "gtid-b"]),
+                filter=MCPToolFilter(gtids=["gtid-a", "gtid-b"]),
             )
 
     @pytest.mark.asyncio

tests/agentgateway/unit/test_fragments.py

@@ -36,7 +36,7 @@ def _full_labels(gtid: str, system_type: str, ord_id: str) -> list[Label]:
 # ============================================================
 
 
-class TestListActiveIntegrations:
+class TestListConnectedSystems:
     def test_returns_entries_from_fragment_labels(self):
         frag1 = _fragment("frag-mcp-1")
         frag2 = _fragment("frag-a2a-2")
@@ -209,7 +209,7 @@ def test_filters_by_mcp_label_type_only(self):
 # ============================================================
 
 
-class TestAgentGatewayClientListActiveIntegrations:
+class TestAgentGatewayClientListConnectedSystems:
     def test_delegates_to_fragments_helper(self):
         expected = [
             {

tests/agentgateway/unit/test_lob.py

@@ -230,31 +230,31 @@ def test_uses_correct_filter_labels(self):
             assert filter_opt.filter_labels[0].key == _LABEL_KEY
             assert filter_opt.filter_labels[0].values == [_MCP_LABEL_VALUE]
 
-    def test_adds_gtid_label_when_global_tenant_ids_provided(self):
-        """When global_tenant_ids is set, add a gtid label to the filter."""
+    def test_adds_gtid_label_when_gtids_provided(self):
+        """When gtids is set, add a gtid label to the filter."""
         with patch(
             "sap_cloud_sdk.agentgateway._fragments.create_fragment_client"
         ) as mock_client:
             mock_client.return_value.list_instance_fragments.return_value = []
 
-            list_mcp_fragments("tenant-sub", global_tenant_ids=["gtid-a", "gtid-b"])
+            list_mcp_fragments("tenant-sub", gtids=["gtid-a", "gtid-b"])
 
             call_args = mock_client.return_value.list_instance_fragments.call_args
             filter_opt = call_args.kwargs.get("filter")
             assert len(filter_opt.filter_labels) == 2
             gtid_label = next(
-                lb for lb in filter_opt.filter_labels if lb.key == GTID_LABEL_KEY
+                lb for lb in filter_opt.filter_labels if lb.key == "sap-managed-runtime-gtid"
             )
             assert gtid_label.values == ["gtid-a", "gtid-b"]
 
-    def test_omits_gtid_label_when_global_tenant_ids_is_empty(self):
+    def test_omits_gtid_label_when_gtids_is_empty(self):
         """Empty list is treated the same as None — no gtid label added."""
         with patch(
             "sap_cloud_sdk.agentgateway._fragments.create_fragment_client"
         ) as mock_client:
             mock_client.return_value.list_instance_fragments.return_value = []
 
-            list_mcp_fragments("tenant-sub", global_tenant_ids=[])
+            list_mcp_fragments("tenant-sub", gtids=[])
 
             call_args = mock_client.return_value.list_instance_fragments.call_args
             filter_opt = call_args.kwargs.get("filter")