Skip to content

Commit

Permalink
chore: fix ntsc fix authz order tb [DET-8885] (#667)
Browse files Browse the repository at this point in the history
  • Loading branch information
eecsliu authored and rb-determined-ai committed Nov 2, 2023
1 parent db593a3 commit 5a0ba07
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions e2e_tests/tests/cluster/test_rbac_ntsc.py
Original file line number Diff line number Diff line change
Expand Up @@ -375,11 +375,10 @@ def can_access_logs(creds: authentication.Credentials, ntsc_id: str) -> bool:
session, typ, created_id, 1
) # user 2 should not be able to set priority or know it exists.
assert e.value.status_code == 404, f"user 2 should not be able to set priority {typ}"
if typ != "tensorboard":
with pytest.raises(errors.ForbiddenException):
launch_ntsc(session, workspaces[0].id, typ, experiment_id)
with pytest.raises(errors.ForbiddenException):
launch_ntsc(session, workspaces[1].id, typ, experiment_id)
with pytest.raises(errors.ForbiddenException):
launch_ntsc(session, workspaces[0].id, typ, experiment_id)
with pytest.raises(errors.ForbiddenException):
launch_ntsc(session, workspaces[1].id, typ, experiment_id)

# test visibility
created_id2 = launch_ntsc(
Expand Down

0 comments on commit 5a0ba07

Please sign in to comment.