Skip to content

Commit f2f6e9e

Browse files
committed
Use keyword args for shared helper calls
1 parent bf93bda commit f2f6e9e

6 files changed

Lines changed: 32 additions & 14 deletions

File tree

src/vws/async_query.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -186,6 +186,9 @@ async def query(
186186

187187
result_code = json.loads(s=response.text)["result_code"]
188188
if result_code != "Success":
189-
raise_for_cloud_reco_result_code(result_code, response)
189+
raise_for_cloud_reco_result_code(
190+
result_code=result_code,
191+
response=response,
192+
)
190193

191-
return parse_query_results(response.text)
194+
return parse_query_results(text=response.text)

src/vws/async_vumark_service.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,5 +140,8 @@ async def generate_vumark_instance(
140140

141141
if response.status_code != HTTPStatus.OK:
142142
result_code = json.loads(s=response.text)["result_code"]
143-
raise_for_vumark_result_code(result_code, response)
143+
raise_for_vumark_result_code(
144+
result_code=result_code,
145+
response=response,
146+
)
144147
return response.content

src/vws/async_vws.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,10 @@ async def make_request(
143143
result_code = json.loads(s=response.text)["result_code"]
144144

145145
if result_code != expected_result_code:
146-
raise_for_vws_result_code(result_code, response)
146+
raise_for_vws_result_code(
147+
result_code=result_code,
148+
response=response,
149+
)
147150
return response
148151

149152
async def add_target(
@@ -254,7 +257,7 @@ async def get_target_record(self, target_id: str) -> TargetStatusAndRecord:
254257
content_type="application/json",
255258
)
256259

257-
return parse_target_record_response(response.text)
260+
return parse_target_record_response(text=response.text)
258261

259262
async def wait_for_target_processed(
260263
self,
@@ -388,7 +391,7 @@ async def get_target_summary_report(
388391
content_type="application/json",
389392
)
390393

391-
return parse_target_summary_response(response.text)
394+
return parse_target_summary_response(text=response.text)
392395

393396
async def get_database_summary_report(
394397
self,
@@ -422,7 +425,7 @@ async def get_database_summary_report(
422425
content_type="application/json",
423426
)
424427

425-
return parse_database_summary_response(response.text)
428+
return parse_database_summary_response(text=response.text)
426429

427430
async def delete_target(self, target_id: str) -> None:
428431
"""Delete a given target.

src/vws/query.py

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,9 @@ def query(
154154

155155
result_code = json.loads(s=response.text)["result_code"]
156156
if result_code != "Success":
157-
raise_for_cloud_reco_result_code(result_code, response)
157+
raise_for_cloud_reco_result_code(
158+
result_code=result_code,
159+
response=response,
160+
)
158161

159-
return parse_query_results(response.text)
162+
return parse_query_results(text=response.text)

src/vws/vumark_service.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,5 +121,8 @@ def generate_vumark_instance(
121121

122122
if response.status_code != HTTPStatus.OK:
123123
result_code = json.loads(s=response.text)["result_code"]
124-
raise_for_vumark_result_code(result_code, response)
124+
raise_for_vumark_result_code(
125+
result_code=result_code,
126+
response=response,
127+
)
125128
return response.content

src/vws/vws.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,10 @@ def make_request(
128128
result_code = json.loads(s=response.text)["result_code"]
129129

130130
if result_code != expected_result_code:
131-
raise_for_vws_result_code(result_code, response)
131+
raise_for_vws_result_code(
132+
result_code=result_code,
133+
response=response,
134+
)
132135
return response
133136

134137
def add_target(
@@ -239,7 +242,7 @@ def get_target_record(self, target_id: str) -> TargetStatusAndRecord:
239242
content_type="application/json",
240243
)
241244

242-
return parse_target_record_response(response.text)
245+
return parse_target_record_response(text=response.text)
243246

244247
def wait_for_target_processed(
245248
self,
@@ -362,7 +365,7 @@ def get_target_summary_report(self, target_id: str) -> TargetSummaryReport:
362365
content_type="application/json",
363366
)
364367

365-
return parse_target_summary_response(response.text)
368+
return parse_target_summary_response(text=response.text)
366369

367370
def get_database_summary_report(self) -> DatabaseSummaryReport:
368371
"""Get a summary report for the database.
@@ -394,7 +397,7 @@ def get_database_summary_report(self) -> DatabaseSummaryReport:
394397
content_type="application/json",
395398
)
396399

397-
return parse_database_summary_response(response.text)
400+
return parse_database_summary_response(text=response.text)
398401

399402
def delete_target(self, target_id: str) -> None:
400403
"""Delete a given target.

0 commit comments

Comments
 (0)