chore: добавить just-команды для тестов и убрать лишний redis extra

Что сделано:
- добавлены just-команды test, test-unit, test-cov, test-cov-html
- добавлен pytest-cov в dev-зависимости и обновлён lock-файл
- зависимость redis[asyncio] заменена на redis (asyncio уже встроен)
- подправлены type hints в test_auth_service.py для чистого Pylance

Author: RomanVetrov

Justfile

@@ -77,6 +77,22 @@ pre-commit-run:
 pre-commit-update:
   uv run pre-commit autoupdate
 
+# Запустить все тесты.
+test:
+  uv run pytest
+
+# Запустить только unit-тесты.
+test-unit:
+  uv run pytest tests/unit -q
+
+# Запустить тесты с покрытием в консоли.
+test-cov:
+  uv run pytest --cov=app --cov-report=term-missing
+
+# Запустить тесты с HTML-отчётом покрытия (htmlcov/index.html).
+test-cov-html:
+  uv run pytest --cov=app --cov-report=html
+
 # Проверить код линтером Ruff (без автоисправлений).
 lint:
   uv run ruff check .

pyproject.toml

@@ -13,7 +13,7 @@ dependencies = [
     "pydantic-settings>=2.13.1",
     "pyjwt[crypto]>=2.11.0",
     "python-multipart>=0.0.9",
-    "redis[asyncio]>=7.2.1",
+    "redis>=7.2.1",
     "sqlalchemy[asyncio]>=2.0.48",
     "uvicorn[standard]>=0.41.0",
     "structlog>=25.5.0",
@@ -33,6 +33,7 @@ dev = [
     "pre-commit>=4.5.1",
     "pytest>=9.0.2",
     "pytest-asyncio>=1.3.0",
+    "pytest-cov>=7.0.0",
     "ruff>=0.15.5",
 ]
 

tests/unit/services/test_auth_service.py

@@ -1,40 +1,46 @@
 from __future__ import annotations
 
-from types import SimpleNamespace
+from typing import Any, cast
 from unittest.mock import AsyncMock, Mock
 from uuid import uuid4
 
 import pytest
+from sqlalchemy.ext.asyncio import AsyncSession
 
+from app.models.user import User
 from app.services import auth as auth_service
 
 
 @pytest.fixture
-def fake_session() -> object:
-    return object()
+def fake_session() -> AsyncSession:
+    return cast(AsyncSession, AsyncMock(spec=AsyncSession))
 
 
 @pytest.fixture
-def active_user() -> SimpleNamespace:
-    return SimpleNamespace(
-        id=uuid4(),
-        email="user@example.com",
-        hashed_password="hashed-password",
-        is_active=True,
-    )
+def active_user() -> User:
+    user = User()
+    user.id = uuid4()
+    user.email = "user@example.com"
+    user.hashed_password = "hashed-password"
+    user.is_active = True
+    return user
 
 
 @pytest.mark.asyncio
 async def test_register_user_raises_when_email_already_exists(
     monkeypatch: pytest.MonkeyPatch,
-    fake_session: object,
-    active_user: SimpleNamespace,
+    fake_session: AsyncSession,
+    active_user: User,
 ) -> None:
     get_user_by_email = AsyncMock(return_value=active_user)
     create_user = AsyncMock()
 
-    monkeypatch.setattr(auth_service.user_repo, "get_user_by_email", get_user_by_email)
-    monkeypatch.setattr(auth_service.user_repo, "create_user", create_user)
+    monkeypatch.setattr(
+        auth_service.user_repo,
+        "get_user_by_email",
+        cast(Any, get_user_by_email),
+    )
+    monkeypatch.setattr(auth_service.user_repo, "create_user", cast(Any, create_user))
 
     with pytest.raises(auth_service.UserAlreadyExists):
         await auth_service.register_user(fake_session, "user@example.com", "password")
@@ -45,16 +51,20 @@ async def test_register_user_raises_when_email_already_exists(
 @pytest.mark.asyncio
 async def test_register_user_hashes_password_and_creates_user(
     monkeypatch: pytest.MonkeyPatch,
-    fake_session: object,
-    active_user: SimpleNamespace,
+    fake_session: AsyncSession,
+    active_user: User,
 ) -> None:
     get_user_by_email = AsyncMock(return_value=None)
     hash_password = AsyncMock(return_value="hashed-by-test")
     create_user = AsyncMock(return_value=active_user)
 
-    monkeypatch.setattr(auth_service.user_repo, "get_user_by_email", get_user_by_email)
-    monkeypatch.setattr(auth_service, "hash_password", hash_password)
-    monkeypatch.setattr(auth_service.user_repo, "create_user", create_user)
+    monkeypatch.setattr(
+        auth_service.user_repo,
+        "get_user_by_email",
+        cast(Any, get_user_by_email),
+    )
+    monkeypatch.setattr(auth_service, "hash_password", cast(Any, hash_password))
+    monkeypatch.setattr(auth_service.user_repo, "create_user", cast(Any, create_user))
 
     created = await auth_service.register_user(
         fake_session,
@@ -74,15 +84,19 @@ async def test_register_user_hashes_password_and_creates_user(
 @pytest.mark.asyncio
 async def test_authenticate_user_uses_dummy_hash_when_user_not_found(
     monkeypatch: pytest.MonkeyPatch,
-    fake_session: object,
+    fake_session: AsyncSession,
 ) -> None:
     get_user_by_email = AsyncMock(return_value=None)
     get_dummy_hash = Mock(return_value="dummy-hash")
     verify_password = AsyncMock(return_value=False)
 
-    monkeypatch.setattr(auth_service.user_repo, "get_user_by_email", get_user_by_email)
-    monkeypatch.setattr(auth_service, "get_dummy_hash", get_dummy_hash)
-    monkeypatch.setattr(auth_service, "verify_password", verify_password)
+    monkeypatch.setattr(
+        auth_service.user_repo,
+        "get_user_by_email",
+        cast(Any, get_user_by_email),
+    )
+    monkeypatch.setattr(auth_service, "get_dummy_hash", cast(Any, get_dummy_hash))
+    monkeypatch.setattr(auth_service, "verify_password", cast(Any, verify_password))
 
     authenticated = await auth_service.authenticate_user(
         fake_session,
@@ -100,15 +114,20 @@ async def test_authenticate_user_uses_dummy_hash_when_user_not_found(
 @pytest.mark.asyncio
 async def test_authenticate_active_user_raises_for_inactive_user(
     monkeypatch: pytest.MonkeyPatch,
+    fake_session: AsyncSession,
 ) -> None:
-    inactive_user = SimpleNamespace(is_active=False)
+    inactive_user = User()
+    inactive_user.id = uuid4()
+    inactive_user.email = "inactive@example.com"
+    inactive_user.hashed_password = "hashed-password"
+    inactive_user.is_active = False
     authenticate_user = AsyncMock(return_value=inactive_user)
 
-    monkeypatch.setattr(auth_service, "authenticate_user", authenticate_user)
+    monkeypatch.setattr(auth_service, "authenticate_user", cast(Any, authenticate_user))
 
     with pytest.raises(auth_service.UserInactive):
         await auth_service.authenticate_active_user(
-            object(),
+            fake_session,
             "user@example.com",
             "password",
         )
@@ -117,10 +136,12 @@ async def test_authenticate_active_user_raises_for_inactive_user(
 @pytest.mark.asyncio
 async def test_validate_refresh_subject_raises_when_user_not_found(
     monkeypatch: pytest.MonkeyPatch,
-    fake_session: object,
+    fake_session: AsyncSession,
 ) -> None:
     get_user_by_id = AsyncMock(return_value=None)
-    monkeypatch.setattr(auth_service.user_repo, "get_user_by_id", get_user_by_id)
+    monkeypatch.setattr(
+        auth_service.user_repo, "get_user_by_id", cast(Any, get_user_by_id)
+    )
 
     with pytest.raises(auth_service.UserNotFound):
         await auth_service.validate_refresh_subject(fake_session, uuid4())
@@ -129,11 +150,17 @@ async def test_validate_refresh_subject_raises_when_user_not_found(
 @pytest.mark.asyncio
 async def test_validate_refresh_subject_raises_when_user_inactive(
     monkeypatch: pytest.MonkeyPatch,
-    fake_session: object,
+    fake_session: AsyncSession,
 ) -> None:
-    inactive_user = SimpleNamespace(is_active=False)
+    inactive_user = User()
+    inactive_user.id = uuid4()
+    inactive_user.email = "inactive@example.com"
+    inactive_user.hashed_password = "hashed-password"
+    inactive_user.is_active = False
     get_user_by_id = AsyncMock(return_value=inactive_user)
-    monkeypatch.setattr(auth_service.user_repo, "get_user_by_id", get_user_by_id)
+    monkeypatch.setattr(
+        auth_service.user_repo, "get_user_by_id", cast(Any, get_user_by_id)
+    )
 
     with pytest.raises(auth_service.UserInactive):
         await auth_service.validate_refresh_subject(fake_session, uuid4())