Skip to content

Commit 0b26af2

Browse files
committed
fixes test
1 parent 978018e commit 0b26af2

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

services/web/server/src/simcore_service_webserver/security/_authz_policy.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,8 +50,9 @@ def _handle_exceptions_as_503():
5050
except DatabaseError as err:
5151
_logger.exception(
5252
**create_troubleshooting_log_kwargs(
53-
"Auth unavailable due to database error",
53+
MSG_AUTH_NOT_AVAILABLE + ": Auth unavailable due to database error.",
5454
error=err,
55+
error_context={"origin": str(err.orig) if err.orig else None},
5556
tip="Check database connection",
5657
)
5758
)

services/web/server/tests/unit/isolated/test_security__authz.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
import pytest
1818
from aiocache.base import BaseCache
1919
from aiohttp import web
20-
from psycopg2 import DatabaseError
2120
from pytest_mock import MockerFixture
2221
from simcore_service_webserver.projects.models import ProjectDict
2322
from simcore_service_webserver.security._authz_access_model import (
@@ -30,6 +29,7 @@
3029
)
3130
from simcore_service_webserver.security._authz_policy import AuthorizationPolicy
3231
from simcore_service_webserver.security._authz_repository import ActiveUserIdAndRole
32+
from sqlalchemy.exc import DatabaseError
3333

3434

3535
@pytest.fixture
@@ -277,7 +277,9 @@ async def _fake_db(engine, email):
277277
assert engine == "FAKE-ENGINE"
278278

279279
if "db-failure" in email:
280-
raise DatabaseError
280+
raise DatabaseError(
281+
statement="SELECT 1", params=None, orig=Exception("fake db error")
282+
)
281283

282284
# inactive user or not found
283285
return copy.deepcopy(users_db.get(email))

0 commit comments

Comments
 (0)