Skip to content

Commit 9c459e9

Browse files
Merge pull request #1131 from adamtheturtle/target-name-exists-capture
In TargetNameExist, capture the target name which already exists
2 parents 3ffa30a + 44881b6 commit 9c459e9

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

src/vws/exceptions.py

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,8 @@
11
"""
22
Custom exceptions for Vuforia errors.
33
"""
4+
5+
import json
46
from urllib.parse import urlparse
57

68
import requests
@@ -285,6 +287,15 @@ def response(self) -> Response:
285287
"""
286288
return self._response
287289

290+
@property
291+
def target_name(self) -> str:
292+
"""
293+
The target name which already exists.
294+
"""
295+
response_body = self.response.request.body or b''
296+
request_json = json.loads(response_body)
297+
return str(request_json['name'])
298+
288299

289300
class ImageTooLarge(Exception):
290301
"""

tests/test_exceptions.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -149,6 +149,7 @@ def test_target_name_exist(
149149
)
150150

151151
assert exc.value.response.status_code == codes.FORBIDDEN
152+
assert exc.value.target_name == 'x'
152153

153154

154155
def test_project_inactive(

0 commit comments

Comments
 (0)