fix(tests): patch out live connectivity probe in 4 micro-tests (#292)

BaseServerClient.__init__ performs its own live connectivity probe
(check_connection()) before a test ever gets a chance to install a mock.
tests/micro-tests/test_delete_element_request_body.py already documents
and works around this (patch check_connection to a no-op for the
duration of client construction); these four files never picked up that
pattern, so they silently depended on a real, reachable Egeria platform
at localhost:9443 -- which happened to be true on a dev machine running
a live server, masking the bug, but fails outright in CI with no such
server:

    PyegeriaConnectionException: Client failed to connect to the Egeria
    platform using URL `https://localhost:9443/api/about`.

Confirmed against the v6.1.0 release workflow run:
https://github.com/odpi/egeria-python/actions/runs/32277343662

Files fixed:
- tests/micro-tests/test_get_guid_request.py
- tests/micro-tests/test_metadata_expert_count.py
- tests/micro-tests/test_automated_curation_engine_action.py
- tests/micro-tests/governance/test_governance_officer_governed_by_paths.py

Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>

Author: dwolfson

tests/micro-tests/governance/test_governance_officer_governed_by_paths.py

@@ -1,11 +1,19 @@
+from unittest.mock import patch
+
 import pytest
 
 from pyegeria.omvs.governance_officer import GovernanceOfficer
 
 
 @pytest.mark.asyncio
 async def test_detach_governed_by_uses_definition_path_segment():
-    client = GovernanceOfficer("view-server", "https://localhost:9443", "user", "pwd")
+    # BaseServerClient.__init__ does its own live connectivity probe
+    # (check_connection()) before any mock can be installed, so
+    # check_connection is patched out for the duration of client
+    # construction -- otherwise this file would silently depend on a
+    # reachable Egeria platform despite living in tests/micro-tests/.
+    with patch("pyegeria.core._base_server_client.BaseServerClient.check_connection", return_value=""):
+        client = GovernanceOfficer("view-server", "https://localhost:9443", "user", "pwd")
     captured = {}
 
     async def _fake_delete(url, body):

tests/micro-tests/test_automated_curation_engine_action.py

@@ -9,14 +9,20 @@
 actual @PostMapping route: .../governance-engines/{governanceEngineName}/engine-actions/
 initiate), so the method was unusable before the fix.
 """
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, patch
 
 from pyegeria.omvs.automated_curation import AutomatedCuration
 
 
 def _client():
-    return AutomatedCuration(view_server="vs", platform_url="https://localhost:9443",
-                              user_id="u", user_pwd="p")
+    # BaseServerClient.__init__ does its own live connectivity probe
+    # (check_connection()) before any mock can be installed, so
+    # check_connection is patched out for the duration of client
+    # construction -- otherwise this file would silently depend on a
+    # reachable Egeria platform despite living in tests/micro-tests/.
+    with patch("pyegeria.core._base_server_client.BaseServerClient.check_connection", return_value=""):
+        return AutomatedCuration(view_server="vs", platform_url="https://localhost:9443",
+                                  user_id="u", user_pwd="p")
 
 
 def _mock_request(capture, payload):

tests/micro-tests/test_get_guid_request.py

@@ -13,14 +13,20 @@
 shape was never checked, so get_linked_projects always returned
 NO_ELEMENTS_FOUND even when the server's raw response body had real data.
 """
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, patch
 
 from pyegeria.omvs.project_manager import ProjectManager
 
 
 def _client():
-    return ProjectManager(view_server="vs", platform_url="https://localhost:9443",
-                           user_id="u", user_pwd="p")
+    # BaseServerClient.__init__ does its own live connectivity probe
+    # (check_connection()) before any mock can be installed, so
+    # check_connection is patched out for the duration of client
+    # construction -- otherwise this file would silently depend on a
+    # reachable Egeria platform despite living in tests/micro-tests/.
+    with patch("pyegeria.core._base_server_client.BaseServerClient.check_connection", return_value=""):
+        return ProjectManager(view_server="vs", platform_url="https://localhost:9443",
+                               user_id="u", user_pwd="p")
 
 
 def _mock_request(payload):

tests/micro-tests/test_metadata_expert_count.py

@@ -7,14 +7,20 @@
 No live server: `_async_make_request` is mocked to return a CountResponse. Verifies
 the endpoint URL, POST method, body slimming, asOfTime pass-through, and count parsing.
 """
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, patch
 
 from pyegeria.omvs.metadata_expert import MetadataExpert
 
 
 def _client():
-    return MetadataExpert(view_server="vs", platform_url="https://localhost:9443",
-                          user_id="u", user_pwd="p")
+    # BaseServerClient.__init__ does its own live connectivity probe
+    # (check_connection()) before any mock can be installed, so
+    # check_connection is patched out for the duration of client
+    # construction -- otherwise this file would silently depend on a
+    # reachable Egeria platform despite living in tests/micro-tests/.
+    with patch("pyegeria.core._base_server_client.BaseServerClient.check_connection", return_value=""):
+        return MetadataExpert(view_server="vs", platform_url="https://localhost:9443",
+                              user_id="u", user_pwd="p")
 
 
 def _mock_request(capture, count_payload):