chore: fix failing tests (#1224)

Co-authored-by: Flora Thiebaut <flora.thiebaut@sdsc.ethz.ch>

Author: olevski

test/bases/renku_data_services/data_api/conftest.py

@@ -5,11 +5,9 @@
 from copy import deepcopy
 from datetime import timedelta
 from typing import Any, Protocol
-from unittest.mock import MagicMock
 
 import pytest
 import pytest_asyncio
-import sentry_sdk
 from authzed.api.v1 import Relationship, RelationshipUpdate, SubjectReference, WriteRelationshipsRequest
 from httpx import Response
 from sanic import Sanic
@@ -864,11 +862,3 @@ async def amalthea_session_k8s_watcher(cluster, amalthea_installation, app_manag
     yield
     with contextlib.suppress(TimeoutError):
         await watcher.stop(timeout=timedelta(seconds=1))
-
-
-@pytest.fixture
-def dummy_sentry():
-    client = sentry_sdk.Client(dsn="https://dummy@sentry.dsn/42", transport=MagicMock(), traces_sample_rate=0.0)
-
-    with sentry_sdk.hub.Hub(client):
-        yield

test/bases/renku_data_services/data_api/test_error_handler.py

@@ -1,5 +1,4 @@
 import inspect
-import re
 from collections.abc import Awaitable, Callable
 from typing import Any
 
@@ -117,24 +116,3 @@ def test_error_handler(err: Exception | Callable, expected_response: dict[str, A
     _, res = test_client.get("/")
     assert res.status_code == expected_status_code
     assert res.json == expected_response
-
-
-def test_sentry_trace_id_is_included_in_errors(dummy_sentry):
-    app = Sanic("test-error-handler")
-    app.error_handler = CustomErrorHandler(apispec)
-    app.get("/")(_trigger_error(Exception("test")))  # type: ignore[unused-coroutine]
-    test_client = SanicTestClient(app)
-
-    _, response = test_client.get("/")
-
-    assert response.status_code == 500
-    # trace_id is included in the error response and is a UUID
-    assert "trace_id" in response.json["error"]
-    assert re.match(r"^[0-9a-fA-F]{32}$", response.json["error"]["trace_id"])
-
-    # Set a specific trace_id
-    trace_id = "4bf92f3577b34da6a3ce929d0e0e4736"
-    _, response = test_client.get("/", headers={"sentry-trace": f"{trace_id}-0123456789abcdef-0"})
-
-    assert response.status_code == 500
-    assert response.json["error"]["trace_id"] == trace_id