update

Author: parthban-db

tests/integration/test_client.py

@@ -1,20 +1,4 @@
-def test_get_workspace_client(ucacct, env_or_skip):
-    # Need to switch to ucacct
-    workspace_id = env_or_skip("TEST_WORKSPACE_ID")
-    ws = ucacct.workspaces.get(workspace_id)
-    w = ucacct.get_workspace_client(ws)
-    assert w.current_user.me().active
-
-
-# TODO: need to decide if we want get_workspace_id on every client
+#TODO: need to decide if we want get_workspace_id on every client
 # def test_get_workspace_id(ucws, env_or_skip):
 #     ws_id = int(env_or_skip("THIS_WORKSPACE_ID"))
 #     assert ucws.get_workspace_id() == ws_id
-
-
-def test_creating_ws_client_from_ac_client_does_not_override_config(ucacct, env_or_skip):
-    ws_id = env_or_skip("TEST_WORKSPACE_ID")
-    ws = ucacct.workspaces.get(ws_id)
-    w = ucacct.get_workspace_client(ws)
-    me = w.current_user.me()
-    assert me.user_name is not None

tests/integration/test_files.py

@@ -249,7 +249,7 @@ def create_volume(w, catalog, schema, volume):
             name=volume,
             volume_type=VolumeType.MANAGED,
         )
-        return ResourceWithCleanup(lambda: w.volumes.delete(res.full_name))
+        return ResourceWithCleanup(lambda: vc.delete(res.full_name))
 
 
 def test_files_api_upload_download(ucws, files_api, random):