feat: add delete method to IndexResource with comprehensive tests (#79)

* feat: add delete method to IndexResource

* test: add unit tests for delete method in IndexResource

* test: add integration test for delete method in IndexResource

* fix: correct key for request endpoint in delete test

Changed from 'url' to 'endpoint' to match how BaseFakeClient records requests

* fix: formatting

Author: mathislucka

src/deepset_mcp/api/indexes/resource.py

@@ -94,3 +94,12 @@ async def update(
         raise_for_status(response)
 
         return Index.model_validate(response.json)
+
+    async def delete(self, index_name: str) -> None:
+        """Delete an index.
+
+        :param index_name: Name of the index to delete.
+        """
+        response = await self._client.request(f"/v1/workspaces/{self._workspace}/indexes/{index_name}", method="DELETE")
+
+        raise_for_status(response)

test/integration/test_integration_index_resource.py

@@ -316,3 +316,27 @@ async def test_get_nonexistent_index(
     # Trying to get a non-existent index should raise an exception
     with pytest.raises(ResourceNotFoundError):
         await index_resource.get(index_name=non_existent_name)
+
+
+@pytest.mark.asyncio
+async def test_delete_index(
+    index_resource: IndexResource,
+    valid_index_config: str,
+) -> None:
+    """Test deleting an index."""
+    index_name = "test-delete-index"
+
+    # Create an index to delete
+    config = json.loads(valid_index_config)
+    await index_resource.create(name=index_name, yaml_config=config["config_yaml"])
+
+    # Verify the index exists
+    index: Index = await index_resource.get(index_name=index_name)
+    assert index.name == index_name
+
+    # Delete the index
+    await index_resource.delete(index_name=index_name)
+
+    # Verify the index no longer exists
+    with pytest.raises(ResourceNotFoundError):
+        await index_resource.get(index_name=index_name)

test/unit/api/indexes/test_index_resource.py

@@ -251,3 +251,41 @@ async def test_update_index_invalid_config(
         resource = IndexResource(fake_client, workspace)
         with pytest.raises(BadRequestError):
             await resource.update(index_name="invalid-index", yaml_config="invalid: yaml")
+
+    async def test_delete_index_successful(self, fake_client: BaseFakeClient, workspace: str) -> None:
+        """Test deleting an index successfully."""
+        fake_client.responses[f"/v1/workspaces/{workspace}/indexes/test-index"] = TransportResponse(
+            status_code=204, json={}, text=""
+        )
+
+        resource = IndexResource(fake_client, workspace)
+        await resource.delete("test-index")
+
+        # Verify request
+        last_request = fake_client.requests[-1]
+        assert last_request["method"] == "DELETE"
+        assert last_request["endpoint"] == f"/v1/workspaces/{workspace}/indexes/test-index"
+
+    async def test_delete_nonexistent_index_raises_404(self, fake_client: BaseFakeClient, workspace: str) -> None:
+        """Test that deleting a nonexistent index raises ResourceNotFoundError."""
+        fake_client.responses[f"/v1/workspaces/{workspace}/indexes/nonexistent-index"] = TransportResponse(
+            status_code=404,
+            json={"detail": "Index not found"},
+            text=json.dumps({"detail": "Index not found"}),
+        )
+
+        resource = IndexResource(fake_client, workspace)
+        with pytest.raises(ResourceNotFoundError):
+            await resource.delete("nonexistent-index")
+
+    async def test_delete_server_error_raises_500(self, fake_client: BaseFakeClient, workspace: str) -> None:
+        """Test that server error during delete raises UnexpectedAPIError."""
+        fake_client.responses[f"/v1/workspaces/{workspace}/indexes/server-error-index"] = TransportResponse(
+            status_code=500,
+            json={"detail": "Internal server error"},
+            text=json.dumps({"detail": "Internal server error"}),
+        )
+
+        resource = IndexResource(fake_client, workspace)
+        with pytest.raises(UnexpectedAPIError):
+            await resource.delete("server-error-index")