SAC-31205: addressed review comments

Author: atttiwari

tap_gitlab/client.py

@@ -77,7 +77,8 @@ def __init__(self, config: Mapping[str, Any]) -> None:
         self.config = config
         self._session = session()
 
-        api_url = config.get("api_url", "https://gitlab.com").rstrip("/")
+        api_url = (config.get("api_url") or "").strip() or "https://gitlab.com"
+        api_url = api_url.rstrip("/")
         if not api_url.startswith(("http://", "https://")):
             api_url = f"https://{api_url}"
         # Strip any existing /api/vN suffix so both 'https://gitlab.com' and

tests/unittests/test_client.py

@@ -110,6 +110,14 @@ def test_default_base_url_uses_gitlab_cloud(self):
         client = Client(config)
         self.assertEqual(client.base_url, "https://gitlab.com/api/v4")
 
+    def test_empty_api_url_falls_back_to_gitlab_cloud(self):
+        """An empty or whitespace-only api_url must fall back to gitlab.com."""
+        for value in ["", "   ", None]:
+            with self.subTest(api_url=value):
+                config = {"private_token": "dummy_token", "api_url": value}
+                client = Client(config)
+                self.assertEqual(client.base_url, "https://gitlab.com/api/v4")
+
     def test_custom_api_url_sets_base_url(self):
         """When api_url is set the client targets the on-prem instance."""
         config = {