Untone
1b48675b92
Some checks failed
Deploy on push / deploy (push) Failing after 2m22s
### 🔄 Изменения - **SQLAlchemy KeyError** - исправление ошибки `KeyError: Reaction` при инициализации - **Исправлена ошибка SQLAlchemy**: Устранена проблема `InvalidRequestError: When initializing mapper Mapper[Shout(shout)], expression Reaction failed to locate a name (Reaction)` ### 🧪 Тестирование - **Исправление тестов** - адаптация к новой структуре моделей - **RBAC инициализация** - добавление `rbac.initialize_rbac()` в `conftest.py` - **Создан тест для getSession**: Добавлен комплексный тест `test_getSession_cookies.py` с проверкой всех сценариев - **Покрытие edge cases**: Тесты проверяют работу с валидными/невалидными токенами, отсутствующими пользователями - **Мокирование зависимостей**: Использование unittest.mock для изоляции тестируемого кода ### 🔧 Рефакторинг - **Упрощена архитектура**: Убраны сложные конструкции с отложенными импортами, заменены на чистую архитектуру - **Перемещение моделей** - `Author` и связанные модели перенесены в `orm/author.py`: Вынесены базовые модели пользователей (`Author`, `AuthorFollower`, `AuthorBookmark`, `AuthorRating`) из `orm.author` в отдельный модуль - **Устранены циклические импорты**: Разорван цикл между `auth.core` → `orm.community` → `orm.author` через реструктуризацию архитектуры - **Создан модуль `utils/password.py`**: Класс `Password` вынесен в utils для избежания циклических зависимостей - **Оптимизированы импорты моделей**: Убран прямой импорт `Shout` из `orm/community.py`, заменен на строковые ссылки ### 🔧 Авторизация с cookies - **getSession теперь работает с cookies**: Мутация `getSession` теперь может получать токен из httpOnly cookies даже без заголовка Authorization - **Убрано требование авторизации**: `getSession` больше не требует декоратор `@login_required`, работает автономно - **Поддержка dual-авторизации**: Токен может быть получен как из заголовка Authorization, так и из cookie `session_token` - **Автоматическая установка cookies**: Middleware автоматически устанавливает httpOnly cookies при успешном `getSession` - **Обновлена GraphQL схема**: `SessionInfo` теперь содержит поля `success`, `error` и опциональные `token`, `author` - **Единообразная обработка токенов**: Все модули теперь используют централизованные функции для работы с токенами - **Улучшена обработка ошибок**: Добавлена детальная валидация токенов и пользователей в `getSession` - **Логирование операций**: Добавлены подробные логи для отслеживания процесса авторизации ### 📝 Документация - **Обновлена схема GraphQL**: `SessionInfo` тип теперь соответствует новому формату ответа - Обновлена документация RBAC - Обновлена документация авторизации с cookies
84 lines
3.1 KiB
Python
84 lines
3.1 KiB
Python
import time
|
||
|
||
from sqlalchemy import (
|
||
JSON,
|
||
Boolean,
|
||
ForeignKey,
|
||
Index,
|
||
Integer,
|
||
PrimaryKeyConstraint,
|
||
String,
|
||
)
|
||
from sqlalchemy.orm import Mapped, mapped_column
|
||
|
||
from orm.author import Author
|
||
from orm.base import BaseModel as Base
|
||
|
||
|
||
# Author уже импортирован в начале файла
|
||
def get_author_model():
|
||
"""Возвращает модель Author для использования в запросах"""
|
||
return Author
|
||
|
||
|
||
class TopicFollower(Base):
|
||
"""
|
||
Связь между топиком и его подписчиком.
|
||
|
||
Attributes:
|
||
follower (int): ID подписчика
|
||
topic (int): ID топика
|
||
created_at (int): Время создания связи
|
||
auto (bool): Автоматическая подписка
|
||
"""
|
||
|
||
__tablename__ = "topic_followers"
|
||
|
||
follower: Mapped[int] = mapped_column(ForeignKey("author.id"))
|
||
topic: Mapped[int] = mapped_column(ForeignKey("topic.id"))
|
||
created_at: Mapped[int] = mapped_column(Integer, nullable=False, default=lambda: int(time.time()))
|
||
auto: Mapped[bool] = mapped_column(Boolean, nullable=False, default=False)
|
||
|
||
# Определяем индексы
|
||
__table_args__ = (
|
||
PrimaryKeyConstraint(topic, follower),
|
||
# Индекс для быстрого поиска всех подписчиков топика
|
||
Index("idx_topic_followers_topic", "topic"),
|
||
# Индекс для быстрого поиска всех топиков, на которые подписан автор
|
||
Index("idx_topic_followers_follower", "follower"),
|
||
)
|
||
|
||
|
||
class Topic(Base):
|
||
"""
|
||
Модель топика (темы) публикаций.
|
||
|
||
Attributes:
|
||
slug (str): Уникальный строковый идентификатор темы
|
||
title (str): Название темы
|
||
body (str): Описание темы
|
||
pic (str): URL изображения темы
|
||
community (int): ID сообщества
|
||
oid (str): Старый ID
|
||
parent_ids (list): IDs родительских тем
|
||
"""
|
||
|
||
__tablename__ = "topic"
|
||
|
||
id: Mapped[int] = mapped_column(Integer, primary_key=True, autoincrement=True)
|
||
slug: Mapped[str] = mapped_column(String, unique=True)
|
||
title: Mapped[str] = mapped_column(String, nullable=False, comment="Title")
|
||
body: Mapped[str | None] = mapped_column(String, nullable=True, comment="Body")
|
||
pic: Mapped[str | None] = mapped_column(String, nullable=True, comment="Picture")
|
||
community: Mapped[int] = mapped_column(ForeignKey("community.id"), default=1)
|
||
oid: Mapped[str | None] = mapped_column(String, nullable=True, comment="Old ID")
|
||
parent_ids: Mapped[list[int] | None] = mapped_column(JSON, nullable=True, comment="Parent Topic IDs")
|
||
|
||
# Определяем индексы
|
||
__table_args__ = (
|
||
# Индекс для быстрого поиска по slug
|
||
Index("idx_topic_slug", "slug"),
|
||
# Индекс для быстрого поиска по сообществу
|
||
Index("idx_topic_community", "community"),
|
||
)
|