Комментарий: git diff origin/develop -- reducers/authentication.test.js

1. initialState вынесен в константу.

Было: объект захардкожен в тесте it('initial state') как { loading: false, error: null, loggedIn: false }.

Стало: константа initialState, которая к тому же совпадает с initialState из самого редьюсера [reducers/authentication.js:3].

2. loading: false → loading: true.

Старый init: loading: false — пользователь не логинился, спиннер не крутится.

Новый init: loading: true — при загрузке приложения сразу идёт loadUserStatus(), который проверяет сессию. Стартовое состояние — «ищем пользователя». Это изменение редьюсера commit 8b3b44be7 (2020-03-18).

3. loggedIn: false → user: null. ГЛАВНОЕ ИЗМЕНЕНИЕ STATE SHAPE.

• loggedIn: true после логина → теперь user: { name: 'test', ... }
• loggedIn: false до логина → теперь user: null
• Компоненты проверяют user !== null вместо loggedIn === true
• Соответствует миграции connect() → useSelector [react-redux hooks]

unknown action → unknown action returns current state. Описание явное: «редиректор не знает такой action → возвращает текущее состояние без изменений». Старый тест не говорил что именно проверяет.

Изменение входного state: старый тест проверял { loading: true, loggedIn: false }, новый — { loading: false, user: { name: 'test' } }. Выбран state залогиненного пользователя, чтобы проверить что user не зануляется при неизвестном action — гарантия чистой функции [Redux reducers].

Форматирование: старый вызов разбит на 3 строки, новый — одной строкой. Только стиль.

Object.freeze(state) [MDN] — защита от мутации: если reducer изменит переданный state (нарушит чистоту), тест упадёт с ошибкой.

describe('handles USER_LOGIN_BEGIN') → describe('USER_LOGIN_BEGIN'). Убрано слово handles — в новом стиле все describe называют action дословно. [новый файл]^⚡

Удалено 2 из 3 тестов.

«fresh state» — проверял переход из { loading: false, error: null, ... } → { loading: true, error: null, ... }. [строка 34]

«error state» — проверял переход из { loading: false, error: exampleError, ... } → { loading: true, error: null, ... }. [строка 53]

Оба теста проверяют одно и то же: error зануляется, loading становится true. Разница — в исходном error (null vs строка). Избыточно — один тест покрывает оба случая.

«loggedIn state» — вход: { loading: false, error: null, loggedIn: false } — полностью совпадает с «fresh state». Возможно, опечатка (должно было быть loggedIn: true). [строка 72]

Один новый тест: 'resets to loading with null user' [новый файл]^⚡

• Вход: { loading: false, error: 'some error', user: { name: 'old' } }
• Выход: { loading: true, error: null, user: null }
• Проверяет что user зануляется (жёстче чем loggedIn: false — полная очистка)
• error зануляется, loading становится true — как и в старых тестах
• Замена loggedIn: false на user: null — ключевое отличие state shape

Старый тест «loading state» — БЕЗ PAYLOAD.

Вызывал { type: 'USER_LOGIN_SUCCESS' } без третьего аргумента (payload). Старый редьюсер якобы просто ставил loggedIn: true и loading: false — пользовательские данные не сохранялись.

Удалён «weird state»: { loading: false, error: exampleError, loggedIn: true } — бессмысленная комбинация: error есть при loggedIn = true. [строка 115]

Новый тест — с реальным payload:

Почему именно эти поля: редьюсер в authentication.js:18-27 ожидает payload с четырьмя полями — каждое пишется в state напрямую:

Старый тест не передавал payload вообще — а значит не проверял ни одного из этих полей. Новый тест проверяет полное соответствие.

loggedIn: true → user: { name: 'testuser' }. Компоненты проверяют state.user !== null вместо state.loggedIn === true — часть миграции на react-redux hooks.

Аналогичные изменения: handles USER_LOGIN_FAILURE → USER_LOGIN_FAILURE, удалён «weird state», один тест вместо двух.

Новое описание: 'sets error and clears user/loading' — явно говорит что происходит: ошибка сохраняется, пользователь сбрасывается, спиннер убирается.

loggedIn: false → user: null. Поле user становится null при ошибке. Это жёстче чем loggedIn: false — старый подход оставлял пользовательские данные в state (просто помечал что не залогинен), новый — полностью очищает. Соответствует case USER_LOGIN_FAILURE: user: null.

Форматирование payload: { error: exampleError } на одной строке вместо трёх. Только стиль.

Удалён целиком (2 теста, ~40 строк).

Проверял что USER_LOGOUT сбрасывает { loggedIn: true } в { loggedIn: false } и зануляет ошибку.

Почему удалён:

1. Action USER_LOGOUT больше не существует — не экспортится из authentication.js (импортит только USER_LOGIN_BEGIN, USER_LOGIN_FAILURE, USER_LOGIN_SUCCESS).
2. USER_LOGOUT нет в switch редьюсера [только 3 case].
3. Функциональность логаута перекрывается USER_LOGIN_BEGIN — оба сбрасывают состояние.
4. Тест импортировал USER_LOGOUT из несуществующего экспорта — тест был сломан.

Исторически: USER_LOGOUT был удалён из исходников commit 7c60c2fbb (2019-07-13, «implemented logout check») — из actions/authentication.js и из редьюсера. Но тест не обновили — он продолжал импортировать и тестировать USER_LOGOUT ещё 7 лет. В новом редьюсере логаут — это просто USER_LOGIN_BEGIN (сброс к loading).

Redux reducer

Чистая функция: (state, action) → new state. Никаких side effects, никаких мутаций. Object.freeze(state) в тесте гарантирует это: если редьюсер случайно мутирует переданный state, тест упадёт. [Redux docs]

State shape: loggedIn → user

Старый state: { loading, error, loggedIn }. Новый: { loading, error, user }. loggedIn: boolean говорил только «да/нет». user: object|null даёт полный объект пользователя (id, name, locale, ...) — компоненты через useSelector(state => state.authentication.user) получают сразу все данные.

Object.freeze

Делает объект неизменяемым. В тесте редьюсера: Object.freeze(state); reducer(state, action); — если редьюсер попытается изменить state (вместо создания нового), JavaScript бросит ошибку в strict mode. Это проверка чистоты функции. [MDN]