fix tests

Author: mconflitti-pbc

tests/posit/connect/external/test_connect_api.py

@@ -45,7 +45,7 @@ def test_provider(self):
         )
         viewer_client = provider.get_client()
         assert viewer_client is not None
-        assert viewer_client.cfg.url == "http://connect.example/"
+        assert viewer_client.cfg.url == "http://connect.example/__api__"
         assert viewer_client.cfg.api_key == "viewer-api-key"
 
     @responses.activate
@@ -64,13 +64,16 @@ def test_provider_with_url_override(self):
             user_session_token="cit",
             url_override="http://connect2.example/",
         )
+        provider._client._ctx.version = None
         viewer_client = provider.get_client()
         assert viewer_client is not None
-        assert viewer_client.cfg.url == "http://connect2.example/"
+        assert viewer_client.cfg.url == "http://connect2.example/__api__"
         assert viewer_client.cfg.api_key == "viewer-api-key"
 
     @responses.activate
-    @patch.dict("os.environ", {"RSTUDIO_PRODUCT": "CONNECT"})
+    @patch.dict(
+        "os.environ", {"RSTUDIO_PRODUCT": "CONNECT", "CONNECT_SERVER": "http://connect.example/"}
+    )
     def test_provider_with_client_override(self):
         register_mocks()
 
@@ -82,7 +85,7 @@ def test_provider_with_client_override(self):
         )
         viewer_client = provider.get_client()
         assert viewer_client is not None
-        assert viewer_client.cfg.url == "http://connect.example/"
+        assert viewer_client.cfg.url == "http://connect.example/__api__"
         assert viewer_client.cfg.api_key == "viewer-api-key"
 
     @patch.dict(
@@ -93,8 +96,9 @@ def test_provider_fallback(self):
         provider = ViewerConnectClientProvider(
             user_session_token="cit",
         )
+        provider._client._ctx.version = None
         viewer_client = provider.get_client()
-        assert viewer_client.cfg.url == "https://connect.example/"
+        assert viewer_client.cfg.url == "https://connect.example/__api__"
         assert viewer_client.cfg.api_key == "12345"
 
     def test_provider_fallback_with_client_override(self):
@@ -106,5 +110,5 @@ def test_provider_fallback_with_client_override(self):
             client_override=client,
         )
         viewer_client = provider.get_client()
-        assert viewer_client.cfg.url == "https://connect.example/"
+        assert viewer_client.cfg.url == "https://connect.example/__api__"
         assert viewer_client.cfg.api_key == "12345"