Update code to pass tests

Author: Paweł Mudlaff

gradient/api_sdk/repositories/jobs.py

@@ -1,5 +1,6 @@
 from gradient import config
 from gradient.api_sdk import serializers
+from gradient.api_sdk.clients import http_client
 from .common import ListResources, CreateResource, BaseRepository, GetResource, DeleteResource, StopResource
 from ..serializers import JobSchema, LogRowSchema
 
@@ -150,20 +151,22 @@ def _get_request_params(self, kwargs):
         return params
 
 
-class DeleteJobArtifacts(GetBaseJobApiUrlMixin, BaseRepository):
+class DeleteJobArtifacts(GetBaseJobApiUrlMixin, DeleteResource):
     VALIDATION_ERROR_MESSAGE = "Failed to delete resource"
 
     def get_request_url(self, **kwargs):
         return "/jobs/{}/artifactsDestroy".format(kwargs.get("id"))
 
-    # def delete(self, id_, **kwargs):
-    #     url = self.get_request_url(id_=id_)
-    #
-    #     params = self._get_request_params(kwargs)
-    #
-    #     client = self._get_client()
-    #     response = client.post(url, json=kwargs.get("json"), params=params)
-    #     self._validate_response(response)
+    def _send(self, url, use_vpc=False, **kwargs):
+        client = self._get_client(use_vpc=use_vpc)
+        params_data = self._get_request_params(kwargs)
+        response = self._send_request(client, url, params_data=params_data)
+        gradient_response = http_client.GradientResponse.interpret_response(response)
+        return gradient_response
+
+    def _send_request(self, client, url, params_data=None):
+        response = client.post(url, params=params_data)
+        return response
 
     def _get_request_params(self, kwargs):
         filters = dict()

tests/functional/test_jobs.py

@@ -311,7 +311,7 @@ def test_should_send_valid_post_request_when_destroying_artifacts_with_files_spe
                                               "some_key"])
 
         assert result.exit_code == 0, result.exc_info
-        post_patched.assert_called_with("{}/jobs/{}/artifactsDestroy/".format(self.URL, job_id),
+        post_patched.assert_called_with("{}/jobs/{}/artifactsDestroy".format(self.URL, job_id),
                                         files=None,
                                         headers=self.EXPECTED_HEADERS_WITH_CHANGED_API_KEY,
                                         json=None,
@@ -324,7 +324,7 @@ def test_should_send_valid_post_request_when_destroying_artifacts_without_files_
         job_id = "some_job_id"
         result = self.runner.invoke(cli.cli, ["jobs", "artifacts", "destroy", job_id, "--apiKey", "some_key"])
 
-        post_patched.assert_called_with("{}/jobs/{}/artifactsDestroy/".format(self.URL, job_id),
+        post_patched.assert_called_with("{}/jobs/{}/artifactsDestroy".format(self.URL, job_id),
                                         files=None,
                                         headers=self.EXPECTED_HEADERS_WITH_CHANGED_API_KEY,
                                         json=None,
@@ -340,7 +340,7 @@ def test_should_read_options_from_yaml_file(self, post_patched, jobs_artifacts_d
         result = self.runner.invoke(cli.cli, command)
 
         assert result.exit_code == 0, result.exc_info
-        post_patched.assert_called_with("{}/jobs/some_id/artifactsDestroy/".format(self.URL),
+        post_patched.assert_called_with("{}/jobs/some_id/artifactsDestroy".format(self.URL),
                                         files=None,
                                         headers=self.EXPECTED_HEADERS_WITH_CHANGED_API_KEY,
                                         json=None,