Skip to content

Commit e379b4a

Browse files
Merge pull request #92 from vossen-adobe/#91-fix-value-error
Fix bad return on RequestError
2 parents 2d5f8b2 + f1f4060 commit e379b4a

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

tests/test_queries.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ def test_query_multiple_user_not_found():
104104
with mock.patch("umapi_client.connection.requests.Session.get") as mock_get:
105105
mock_get.return_value = MockResponse(404, text="404 Object not found")
106106
conn = Connection(**mock_connection_params)
107-
assert conn.query_multiple("user") == ([], True)
107+
assert conn.query_multiple("user") == ([], True, 0, 0, 0, 0)
108108

109109

110110
def test_query_multiple_user_paged():
@@ -230,7 +230,7 @@ def test_query_multiple_usergroup_not_found():
230230
with mock.patch("umapi_client.connection.requests.Session.get") as mock_get:
231231
mock_get.return_value = MockResponse(404, text="404 Object not found")
232232
conn = Connection(**mock_connection_params)
233-
assert conn.query_multiple("user-group") == ([], True)
233+
assert conn.query_multiple("user-group") == ([], True, 0, 0, 0, 0)
234234

235235

236236
def test_query_multiple_usergroup_paged():

umapi_client/connection.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,7 @@ def query_multiple(self, object_type, page=0, url_params=None, query_params=None
308308
if re.result.status_code == 404:
309309
if self.logger: self.logger.debug("Ran %s query: %s %s (0 found)",
310310
object_type, url_params, query_params)
311-
return [], True
311+
return [], True, 0, 0, 0, 0
312312
else:
313313
raise re
314314

0 commit comments

Comments
 (0)