Skip to content

Commit

Permalink
fix: in UserFromSearch field ip_phone, other_ip_phone from str to int
Browse files Browse the repository at this point in the history
  • Loading branch information
Leonid Gorbunov committed Jan 27, 2025
1 parent ef104e4 commit 038ab40
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions pybotx/client/users_api/user_from_csv.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ class BotXAPIUserFromCSVResult(VerifiedPayloadBaseModel):
description: Optional[str] = Field(alias="Description")
phone: Optional[str] = Field(alias="Phone")
other_phone: Optional[str] = Field(alias="Other phone")
ip_phone: Optional[str] = Field(alias="IP phone")
other_ip_phone: Optional[str] = Field(alias="Other IP phone")
ip_phone: Optional[int] = Field(alias="IP phone")
other_ip_phone: Optional[int] = Field(alias="Other IP phone")
personnel_number: Optional[str] = Field(alias="Personnel number")

@validator(
Expand Down
4 changes: 2 additions & 2 deletions pybotx/models/users.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,6 +106,6 @@ class UserFromCSV:
description: Optional[str] = None
phone: Optional[str] = None
other_phone: Optional[str] = None
ip_phone: Optional[str] = None
other_ip_phone: Optional[str] = None
ip_phone: Optional[int] = None
other_ip_phone: Optional[int] = None
personnel_number: Optional[str] = None
6 changes: 3 additions & 3 deletions tests/client/users_api/test_users_as_csv.py
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ async def test__users_as_csv__succeed(
status_code=HTTPStatus.OK,
content=(
b"HUID,AD Login,Domain,AD E-mail,Name,Sync source,Active,Kind,Company,Department,Position,Manager,Manager HUID,Personnel number,Description,IP phone,Other IP phone,Phone,Other phone,Avatar,Office,Avatar preview\n"
b"dbc8934f-d0d7-4a9e-89df-d45c137a851c,test_user_17,cts.example.com,,test_user_17,ad,false,cts_user,Company,Department,Position,Manager John,13a6909c-bce1-4dbf-8359-efb7ef8e5b34,Some number,Description,Ip_phone,Other_ip_phone,Phone,Other_phone,Avatar,Office,Avatar_preview\n"
b"dbc8934f-d0d7-4a9e-89df-d45c137a851c,test_user_17,cts.example.com,,test_user_17,ad,false,cts_user,Company,Department,Position,Manager John,13a6909c-bce1-4dbf-8359-efb7ef8e5b34,Some number,Description,123,321,Phone,Other_phone,Avatar,Office,Avatar_preview\n"
b"13a6909c-bce1-4dbf-8359-efb7ef8e5b34,test_user_18,cts.example.com,,test_user_18,unsupported,true,cts_user,,,,,,,,,,,,,,"
),
),
Expand Down Expand Up @@ -107,8 +107,8 @@ async def test__users_as_csv__succeed(
manager="Manager John",
manager_huid=UUID("13a6909c-bce1-4dbf-8359-efb7ef8e5b34"),
description="Description",
ip_phone="Ip_phone",
other_ip_phone="Other_ip_phone",
ip_phone=123,
other_ip_phone=321,
phone="Phone",
other_phone="Other_phone",
avatar="Avatar",
Expand Down

0 comments on commit 038ab40

Please sign in to comment.