fix(ingestion): fix ES SSL context and SAS client test assertions

Author: RinZ27

ingestion/src/metadata/ingestion/source/search/elasticsearch/connection.py

@@ -140,7 +140,7 @@ def get_ssl_context(
         )
         return ssl_context  # noqa: RET504
 
-    return ssl.create_default_context()
+    return None
 
 
 def get_connection(connection: ElasticsearchConnection) -> Elasticsearch:

ingestion/tests/unit/topology/database/test_sas_client.py

@@ -104,7 +104,7 @@ def test_api_error_wrapping(self, mock_rest, mock_request):
 
         with pytest.raises(APIError) as exc:
             client.get_instance("123")
-        assert exc.value.args[0] == "Unauthorized access"
+        assert "Unauthorized access" in str(exc.value)
 
     @patch("metadata.ingestion.source.database.sas.client.requests.request")
     @patch("metadata.ingestion.source.database.sas.client.TrackedREST")
@@ -122,4 +122,4 @@ def test_get_views_error_sanitization(self, mock_rest, mock_request):
         with pytest.raises(APIError) as exc:
             client.get_views("query")
         # Should show the sanitized message instead of the raw error
-        assert exc.value.args[0] == "Error fetching views from SAS"
+        assert "Error fetching views from SAS" in str(exc.value)

openmetadata-spec/src/main/resources/json/schema/entity/services/connections/database/sasConnection.json

@@ -20,15 +20,18 @@
       "default": "SAS"
     },
     "username": {
+      "title": "Username",
       "description": "Username to connect to SAS Viya.",
       "type": "string"
     },
     "password": {
+      "title": "Password",
       "description": "Password to connect to SAS Viya",
       "type": "string",
       "format": "password"
     },
     "serverHost": {
+      "title": "Server Host",
       "description": "Hostname of SAS Viya deployment.",
       "type": "string",
       "format": "uri"