Skip to content

Commit

Permalink
update signature of create_user
Browse files Browse the repository at this point in the history
  • Loading branch information
sir-sigurd committed Jun 5, 2024
1 parent bad7c78 commit b6fe2aa
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions api/python/quilt3/adminql/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,10 @@ def get_users() -> List[GetUsersAdminUserList]:
return _get_client().get_users()

Check warning on line 31 in api/python/quilt3/adminql/__init__.py

View check run for this annotation

Codecov / codecov/patch/informational

api/python/quilt3/adminql/__init__.py#L31

Added line #L31 was not covered by tests


def create_user(input: UserInput) -> None:
_handle_errors(_get_client().create_user(input=input))
def create_user(name: str, email: str, role: str, extra_roles: Optional[List[str]] = None) -> None:
_handle_errors(

Check warning on line 35 in api/python/quilt3/adminql/__init__.py

View check run for this annotation

Codecov / codecov/patch/informational

api/python/quilt3/adminql/__init__.py#L35

Added line #L35 was not covered by tests
_get_client().create_user(input=UserInput(name=name, email=email, role=role, extra_roles=extra_roles))
)
return None

Check warning on line 38 in api/python/quilt3/adminql/__init__.py

View check run for this annotation

Codecov / codecov/patch/informational

api/python/quilt3/adminql/__init__.py#L38

Added line #L38 was not covered by tests


Expand All @@ -59,9 +61,7 @@ def set_role(
return None

Check warning on line 61 in api/python/quilt3/adminql/__init__.py

View check run for this annotation

Codecov / codecov/patch/informational

api/python/quilt3/adminql/__init__.py#L57-L61

Added lines #L57 - L61 were not covered by tests


def add_roles(
roles: List[str], name: str
) -> None:
def add_roles(roles: List[str], name: str) -> None:
result = _get_client().add_roles(roles=roles, name=name)
if result is None:
raise UserNotFoundError
Expand Down

0 comments on commit b6fe2aa

Please sign in to comment.