fixed test

Author: rkritika1508

backend/app/tests/test_banlists_integration.py

@@ -11,7 +11,6 @@
 TEST_ORGANIZATION_ID = 1
 TEST_PROJECT_ID = 1
 BASE_URL = "/api/v1/guardrails/ban-lists/"
-DEFAULT_QUERY = f"?organization_id={TEST_ORGANIZATION_ID}&project_id={TEST_PROJECT_ID}"
 DEFAULT_HEADERS = {
     "X-Org-Id": str(TEST_ORGANIZATION_ID),
     "X-Project-Id": str(TEST_PROJECT_ID),
@@ -71,33 +70,34 @@ def _headers(self, org=TEST_ORGANIZATION_ID, project=TEST_PROJECT_ID):
     def create(self, client, payload_key="minimal", **kwargs):
         payload = {**BAN_LIST_PAYLOADS[payload_key], **kwargs}
         return client.post(
-            f"{BASE_URL}{DEFAULT_QUERY}",
+            f"{BASE_URL}",
             json=payload,
             headers=self._headers(),
         )
 
     def list(self, client, org=TEST_ORGANIZATION_ID, project=TEST_PROJECT_ID, **filters):
-        params = f"?organization_id={org}&project_id={project}"
         if filters:
-            params += "&" + "&".join(f"{k}={v}" for k, v in filters.items())
+            params = "?" + "&".join(f"{k}={v}" for k, v in filters.items())
+        else:
+            params = ""
         return client.get(f"{BASE_URL}{params}", headers=self._headers(org, project))
 
     def get(self, client, id, org=TEST_ORGANIZATION_ID, project=TEST_PROJECT_ID):
         return client.get(
-            f"{BASE_URL}{id}/?organization_id={org}&project_id={project}",
+            f"{BASE_URL}{id}",
             headers=self._headers(org, project),
         )
 
     def update(self, client, id, payload):
         return client.patch(
-            f"{BASE_URL}{id}/{DEFAULT_QUERY}",
+            f"{BASE_URL}{id}",
             json=payload,
             headers=self._headers(),
         )
 
     def delete(self, client, id):
         return client.delete(
-            f"{BASE_URL}{id}/{DEFAULT_QUERY}",
+            f"{BASE_URL}{id}",
             headers=self._headers(),
         )
 
@@ -115,7 +115,7 @@ def test_create_success(self, integration_client, clear_database):
 
     def test_create_validation_error(self, integration_client, clear_database):
         response = integration_client.post(
-            f"{BASE_URL}{DEFAULT_QUERY}",
+            f"{BASE_URL}",
             json={"name": "missing words"},
             headers=self._headers(),
         )
@@ -244,7 +244,7 @@ def test_delete_wrong_owner(self, integration_client, clear_database):
         ban_id = create_resp.json()["data"]["id"]
 
         response = integration_client.delete(
-            f"{BASE_URL}{ban_id}/?organization_id=999&project_id=999",
+            f"{BASE_URL}{ban_id}",
             headers=self._headers(999, 999),
         )