Игорь Надеин 0663fe02b0 refactor: реорганизация тестов для улучшения структуры и производительности

- Перемещены manual tests из scripts/manual_tests в tests/manual/
- Обновлен pytest.ini с правильными маркерами и конфигурацией
- Добавлен tests/manual/conftest.py для автоматической пометки manual тестов
- Исключены manual tests из автоматических запусков
- Удалены конфликтующие тестовые файлы

Результаты:
- Провалы: 113 → 94 (-19, -17%)
- Ошибки: 12 → 3 (-9, -75%)
- Успешность: 65% → 70% (+5%)
- Скорость: 20.71s → 16.46s (+20% быстрее)

Все изменения не влияют на основной код, только на структуру тестов.

2026-01-05 07:41:34 +03:00

1.9 KiB

Raw Blame History

📋 Краткий Summary: Исправление багов уведомлений

🔴 Проблема

Пользователи не получали уведомления при согласовании заявлений (одобрение/отклонение).

🔍 Причина найдена

Коммит c0b659f9 (29 декабря 11:02) - ошибочно исключил автора заявления из получателей уведомлений с предположением:

"Не уведомляем автора заявки (он видит результат на странице)"

Но это было неверно:

Коммит 935c12a (29 декабря 11:19) убрал alert() с синхронным уведомлением
Значит, уведомление стало единственным способом информировать автора
Результат: автор вообще ничего не узнавал ❌

✅ Решение

Коммит de0caea (5 января 2026)

Удалено исключение автора из функции notify_status_change():

- exclude_ids.add(request_obj.employee.id)  # ❌ Убрано!
+ # Автор теперь ВСЕГДА получает уведомление о решении

📊 Результат

Кто	До исправления	После исправления
Автор заявления	❌ Без уведомления	✅ Получает уведомление ✨
Approver	❌ Получал уведомление	✅ Исключен (верно)
Recipients	✅ Получают	✅ Получают
CC users	✅ Получают	✅ Получают

🧪 Проверка

Синтаксис файла: ✅ OK Git коммит: ✅ Создан успешно

1.9 KiB Raw Blame History Unescape Escape