Use helper for making requests

Author: adamtheturtle

src/vws/vws.py

@@ -130,6 +130,32 @@ def __init__(
         self._server_secret_key = server_secret_key.encode()
         self._base_vws_url = base_vws_url
 
+    def _make_request(
+        self,
+        method: str,
+        content: bytes,
+        request_path: str,
+        expected_result_code: str,
+    ) -> Response:
+        """
+        XXX
+        """
+        response = _target_api_request(
+            server_access_key=self._server_access_key,
+            server_secret_key=self._server_secret_key,
+            method=method,
+            content=content,
+            request_path=request_path,
+            base_vws_url=self._base_vws_url,
+        )
+
+        _raise_for_result_code(
+            response=response,
+            expected_result_code=expected_result_code,
+        )
+
+        return response
+
     def add_target(
         self,
         name: str,
@@ -173,17 +199,10 @@ def add_target(
 
         content = bytes(json.dumps(data), encoding='utf-8')
 
-        response = _target_api_request(
-            server_access_key=self._server_access_key,
-            server_secret_key=self._server_secret_key,
+        response = self._make_request(
             method='POST',
             content=content,
             request_path='/targets',
-            base_vws_url=self._base_vws_url,
-        )
-
-        _raise_for_result_code(
-            response=response,
             expected_result_code='TargetCreated',
         )
 
@@ -202,19 +221,13 @@ def get_target_record(self, target_id: str) -> Dict[str, Union[str, int]]:
         Returns:
             Response details of a target from Vuforia.
         """
-        response = _target_api_request(
-            server_access_key=self._server_access_key,
-            server_secret_key=self._server_secret_key,
+        response = self._make_request(
             method='GET',
             content=b'',
             request_path=f'/targets/{target_id}',
-            base_vws_url=self._base_vws_url,
-        )
-
-        _raise_for_result_code(
-            response=response,
             expected_result_code='Success',
         )
+
         return dict(response.json()['target_record'])
 
     @timeout_decorator.timeout(seconds=60 * 5)
@@ -250,19 +263,13 @@ def list_targets(self) -> List[str]:
         Returns:
             The IDs of all targets in the database.
         """
-        response = _target_api_request(
-            server_access_key=self._server_access_key,
-            server_secret_key=self._server_secret_key,
+        response = self._make_request(
             method='GET',
             content=b'',
             request_path='/targets',
-            base_vws_url=self._base_vws_url,
-        )
-
-        _raise_for_result_code(
-            response=response,
             expected_result_code='Success',
         )
+
         return list(response.json()['results'])
 
     def get_target_summary_report(
@@ -281,19 +288,13 @@ def get_target_summary_report(
         Returns:
             Details of the target.
         """
-        response = _target_api_request(
-            server_access_key=self._server_access_key,
-            server_secret_key=self._server_secret_key,
+        response = self._make_request(
             method='GET',
             content=b'',
             request_path=f'/summary/{target_id}',
-            base_vws_url=self._base_vws_url,
-        )
-
-        _raise_for_result_code(
-            response=response,
             expected_result_code='Success',
         )
+
         return dict(response.json())
 
     def get_database_summary_report(self) -> Dict[str, Union[str, int]]:
@@ -306,17 +307,10 @@ def get_database_summary_report(self) -> Dict[str, Union[str, int]]:
         Returns:
             Details of the database.
         """
-        response = _target_api_request(
-            server_access_key=self._server_access_key,
-            server_secret_key=self._server_secret_key,
+        response = self._make_request(
             method='GET',
             content=b'',
             request_path='/summary',
-            base_vws_url=self._base_vws_url,
-        )
-
-        _raise_for_result_code(
-            response=response,
             expected_result_code='Success',
         )
 
@@ -332,16 +326,9 @@ def delete_target(self, target_id: str) -> None:
         Args:
             target_id: The ID of the target to delete.
         """
-        response = _target_api_request(
-            server_access_key=self._server_access_key,
-            server_secret_key=self._server_secret_key,
+        response = self._make_request(
             method='DELETE',
             content=b'',
             request_path=f'/targets/{target_id}',
-            base_vws_url=self._base_vws_url,
-        )
-
-        _raise_for_result_code(
-            response=response,
             expected_result_code='Success',
         )