Skip to content

Commit 67d91b9

Browse files
committed
Test migrat acls for collection
1 parent 4907958 commit 67d91b9

File tree

1 file changed

+15
-4
lines changed

1 file changed

+15
-4
lines changed

tests/unit/test_cli.py

Lines changed: 15 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -429,10 +429,21 @@ def test_save_storage_and_principal_gcp(ws):
429429
principal_prefix_access(ws, ctx=ctx)
430430

431431

432-
def test_migrate_acls_calls_workspace_id(ws) -> None:
433-
ctx = WorkspaceContext(ws)
434-
migrate_acls(ws, ctx=ctx)
435-
ws.get_workspace_id.assert_called()
432+
@pytest.mark.parametrize("run_as_collection", [True, False])
433+
def test_migrate_acls_calls_workspace_id(
434+
run_as_collection,
435+
workspace_clients,
436+
acc_client,
437+
) -> None:
438+
if not run_as_collection:
439+
workspace_clients = [workspace_clients[0]]
440+
migrate_acls(
441+
workspace_clients[0],
442+
run_as_collection=run_as_collection,
443+
a=acc_client,
444+
)
445+
for workspace_client in workspace_clients:
446+
workspace_client.get_workspace_id.assert_called()
436447

437448

438449
def test_migrate_credentials_azure(ws, acc_client):

0 commit comments

Comments
 (0)