Skip to content

Commit 64a7e7d

Browse files
fix: type annotations
1 parent ff99d7d commit 64a7e7d

3 files changed

Lines changed: 11 additions & 5 deletions

File tree

app/common/infrastructure/sqlalchemy_criteria_translator.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
1-
__all__ = ("SqlalchemyCriteriaTranslator",)
1+
__all__ = (
2+
"SqlalchemyCriteriaTranslator",
3+
"Statement",
4+
)
25

36
import abc
47
import typing as t
@@ -7,7 +10,7 @@
710

811
from app.common.domain import Criteria
912

10-
type _Statement = sa.Select[t.Any] | sa.Update | sa.Delete
13+
type Statement = sa.Select[t.Any] | sa.Update | sa.Delete
1114
type _Criteria = Criteria[t.Any]
1215
type _Translator = "SqlalchemyCriteriaTranslator[t.Any]"
1316
type _Registry = dict[type[_Criteria], type[_Translator]]
@@ -38,7 +41,7 @@ def get_for_criteria(cls, criteria: _Criteria) -> _Translator:
3841
return translator_cls(criteria)
3942

4043
@abc.abstractmethod
41-
def translate(self, stmt: _Statement) -> _Statement:
44+
def translate(self, stmt: Statement) -> Statement:
4245
raise NotImplementedError()
4346

4447

app/users/api/dependencies/current_user_dependency.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
)
55

66
import typing as t
7+
import uuid
78

89
import fastapi
910

@@ -34,8 +35,9 @@ def get_current_user(
3435
status_code=fastapi.status.HTTP_401_UNAUTHORIZED, detail=e.message
3536
)
3637

38+
user_id = UserId(uuid.UUID(token_data.user_id))
3739
try:
38-
user = user_repository.find_or_fail(UserId(token_data.user_id))
40+
user = user_repository.find_or_fail(user_id)
3941
except ModelNotFoundException as e:
4042
raise fastapi.HTTPException(
4143
status_code=404,

app/users/infrastructure/sqlalchemy_user_translators.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
__all__ = ("SQLAlchemyUserEmailFilterTranslator",)
22

33
from app.common.infrastructure import SqlalchemyCriteriaTranslator
4+
from app.common.infrastructure import Statement
45
from app.users.domain import UserEmailFilter
56

67
from .sqlalchemy_user_model import SQLAlchemyUserModel
@@ -10,5 +11,5 @@ class SQLAlchemyUserEmailFilterTranslator(
1011
SqlalchemyCriteriaTranslator[UserEmailFilter],
1112
criteria=UserEmailFilter,
1213
):
13-
def translate(self, stmt):
14+
def translate(self, stmt: Statement) -> Statement:
1415
return stmt.where(SQLAlchemyUserModel.email == self._criteria.email)

0 commit comments

Comments
 (0)