Merge pull request #36 from Covata/delete-secret

Delete secret

src/main/python/covata/delta/api/requestsclient.py

@@ -137,6 +137,15 @@ def get_secret(self, requestor_id, secret_id):
             secret["encryptionDetails"][k] = b64decode(v)
         return secret
 
+    def delete_secret(self, requestor_id, secret_id):
+        response = requests.delete(
+            url="{base_url}{resource}/{secret_id}".format(
+                base_url=self.DELTA_URL,
+                resource=self.RESOURCE_SECRETS,
+                secret_id=secret_id),
+            auth=self.signer(requestor_id))
+        response.raise_for_status()
+
     def update_secret_metadata(self,
                                requestor_id, secret_id, metadata, version):
         response = requests.put(

src/main/python/covata/delta/interfaces.py

@@ -98,6 +98,15 @@ def share_secret(self, requestor_id, content, encryption_details,
         :rtype: dict[str, str]
         """
 
+    @abstractmethod
+    def delete_secret(self, requestor_id, secret_id):
+        """
+        Deletes the secret with the given secret id.
+
+        :param str requestor_id: the authenticating identity id
+        :param str secret_id: the secret id to be deleted
+        """
+
     @abstractmethod
     def get_secret(self, requestor_id, secret_id):
         """

src/unittest/python/test_requests_api_client.py

@@ -256,6 +256,25 @@ def test_get_secret(api_client, mock_signer):
     assert response == expected_json
 
 
+@responses.activate
+def test_delete_secret(api_client, mock_signer):
+    requestor_id = "requestor_id"
+    secret_id = "secret_id"
+
+    responses.add(responses.DELETE,
+                  "{base_path}{resource}/{secret_id}".format(
+                      base_path=DeltaApiClient.DELTA_URL,
+                      resource=DeltaApiClient.RESOURCE_SECRETS,
+                      secret_id=secret_id),
+                  status=204)
+
+    api_client.delete_secret(requestor_id, secret_id)
+
+    mock_signer.assert_called_once_with(requestor_id)
+
+    assert len(responses.calls) == 1
+
+
 @responses.activate
 def test_get_secret_metadata(api_client, mock_signer):
     response_json = dict(metadata_key="metadata value")