Skip to content

Commit ea072fa

Browse files
authored
Remove parameters that are not in the API docs (#383)
1 parent 940073e commit ea072fa

4 files changed

Lines changed: 6 additions & 8 deletions

File tree

rocketchat_API/APISections/dm.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,10 +78,10 @@ def dm_files(self, room_id=None, user_name=None, **kwargs):
7878
return self.call_api_get("dm.files", username=user_name, kwargs=kwargs)
7979
raise RocketMissingParamException(ROOM_ID_OR_USERNAME_REQUIRED)
8080

81-
def dm_counters(self, room_id, user_name=None):
81+
def dm_counters(self, room_id, user_id=None):
8282
"""Gets counters of direct messages."""
83-
if user_name:
84-
return self.call_api_get("dm.counters", roomId=room_id, username=user_name)
83+
if user_id:
84+
return self.call_api_get("dm.counters", roomId=room_id, userId=user_id)
8585
return self.call_api_get("dm.counters", roomId=room_id)
8686

8787
def dm_delete(self, room_id, **kwargs):

rocketchat_API/APISections/teams.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ def teams_autocomplete(self, name, **kwargs):
173173

174174
@paginated("rooms")
175175
def teams_list_rooms_of_user(
176-
self, team_id=None, team_name=None, user_id=None, can_add=False, **kwargs
176+
self, team_id=None, team_name=None, user_id=None, **kwargs
177177
):
178178
"""
179179
Lists only the team's rooms that a particular user has joined.
@@ -184,15 +184,13 @@ def teams_list_rooms_of_user(
184184
"teams.listRoomsOfUser",
185185
teamId=team_id,
186186
userId=user_id,
187-
canAdd=can_add,
188187
kwargs=kwargs,
189188
)
190189
if team_name:
191190
return self.call_api_get(
192191
"teams.listRoomsOfUser",
193192
teamName=team_name,
194193
userId=user_id,
195-
canAdd=can_add,
196194
kwargs=kwargs,
197195
)
198196
raise RocketMissingParamException(TEAM_ID_OR_TEAM_NAME_REQUIRED)

rocketchat_API/APISections/users.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def users_set_active_status(self, user_id, active_status, **kwargs):
129129

130130
def users_forgot_password(self, email, **kwargs):
131131
"""Send email to reset your password."""
132-
return self.call_api_post("users.forgotPassword", email=email, data=kwargs)
132+
return self.call_api_post("users.forgotPassword", email=email, kwargs=kwargs)
133133

134134
def users_get_preferences(self, **kwargs):
135135
"""Gets all preferences of user."""

tests/test_dms.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,7 @@ def test_dm_counters(logged_rocket, recipient_user):
146146

147147
dm_counters = logged_rocket.dm_counters(
148148
room_id=dm_create.get("room").get("_id"),
149-
user_name=logged_rocket.me().get("_id"),
149+
user_id=logged_rocket.me().get("_id"),
150150
)
151151
assert all(
152152
key in dm_counters

0 commit comments

Comments
 (0)