From 4d30a64bc9d327c70b58340b001c7083ac4977a2 Mon Sep 17 00:00:00 2001
From: Ilya Kreymer <ikreymer@users.noreply.github.com>
Date: Mon, 29 May 2023 18:19:04 -0700
Subject: [PATCH] collection delete: (#896)

set delete endpoint to use DELETE verb, fix for #869
---
 backend/btrixcloud/colls.py      | 4 ++--
 backend/test/test_collections.py | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/backend/btrixcloud/colls.py b/backend/btrixcloud/colls.py
index 9b869e60..7d6ae5f2 100644
--- a/backend/btrixcloud/colls.py
+++ b/backend/btrixcloud/colls.py
@@ -478,8 +478,8 @@ def init_collections_api(app, mdb, crawls, orgs, crawl_manager):
             coll_id, crawlList.crawlIds, org
         )
 
-    @app.get(
-        "/orgs/{oid}/collections/{coll_id}/delete",
+    @app.delete(
+        "/orgs/{oid}/collections/{coll_id}",
         tags=["collections"],
     )
     async def delete_collection(
diff --git a/backend/test/test_collections.py b/backend/test/test_collections.py
index 83e2f5dd..8abf5e40 100644
--- a/backend/test/test_collections.py
+++ b/backend/test/test_collections.py
@@ -357,8 +357,8 @@ def test_filter_sort_collections(
 
 def test_delete_collection(crawler_auth_headers, default_org_id, crawler_crawl_id):
     # Delete second collection
-    r = requests.get(
-        f"{API_PREFIX}/orgs/{default_org_id}/collections/{_second_coll_id}/delete",
+    r = requests.delete(
+        f"{API_PREFIX}/orgs/{default_org_id}/collections/{_second_coll_id}",
         headers=crawler_auth_headers,
     )
     assert r.status_code == 200