Commentaire : git diff origin/develop -- reducers/authentication.test.js

1. initialState extrait dans une constante.

Ancienne version : objet codé en dur dans le test it('initial state') sous la forme { loading: false, error: null, loggedIn: false }.

Nouvelle version : constante initialState, qui en plus correspond exactement à initialState défini dans le réducteur [reducers/authentication.js:3].

2. loading: false → loading: true.

Ancien init : loading: false — l'utilisateur n'est pas connecté, le spinner ne tourne pas.

Nouvel init : loading: true — au chargement de l'application, loadUserStatus() est appelé immédiatement pour vérifier la session. L'état initial devient « recherche de l'utilisateur ». Cette modification vient du réducteur commit 8b3b44be7 (2020-03-18).

3. loggedIn: false → user: null. CHANGEMENT MAJEUR DE LA STRUCTURE DE L'ÉTAT (STATE SHAPE).

• loggedIn: true après connexion → désormais user: { name: 'test', ... }
• loggedIn: false avant connexion → désormais user: null
• Les composants vérifient user !== null au lieu de loggedIn === true
• Correspond à la migration connect() → useSelector [react-redux hooks]

unknown action → unknown action returns current state. Description explicite : « le réducteur ne connaît pas cette action → il retourne l'état actuel sans modification ». L'ancien test ne précisait pas ce qu'il vérifiait.

Modification de l'état d'entrée : l'ancien test vérifiait { loading: true, loggedIn: false }, le nouveau vérifie { loading: false, user: { name: 'test' } }. Un état d'utilisateur connecté a été choisi pour s'assurer que user n'est pas réinitialisé lors d'une action inconnue — ce qui garantit la pureté de la fonction [Redux reducers].

Formatage : l'ancien appel était sur 3 lignes, le nouveau tient sur une seule ligne. Uniquement du style.

Object.freeze(state) [MDN] — protection contre la mutation : si le réducteur modifiait l'état transmis (violant le principe de pureté), le test échouerait avec une erreur.

describe('handles USER_LOGIN_BEGIN') → describe('USER_LOGIN_BEGIN'). Le mot handles a été retiré — dans le nouveau style, chaque describe nomme l'action littéralement. [nouveau fichier]^⚡

2 des 3 tests supprimés.

« fresh state » — vérifiait la transition de { loading: false, error: null, ... } vers { loading: true, error: null, ... }. [ligne 34]

« error state » — vérifiait la transition de { loading: false, error: exampleError, ... } vers { loading: true, error: null, ... }. [ligne 53]

Les deux tests vérifiaient la même chose : error est réinitialisé à null, loading passe à true. La différence résidait dans l'error initial (null vs chaîne). C'était redondant — un seul test couvre les deux cas.

« loggedIn state » — entrée : { loading: false, error: null, loggedIn: false } — identique à « fresh state ». Probablement une coquille (il aurait dû y avoir loggedIn: true). [ligne 72]

Un seul nouveau test : 'resets to loading with null user' [nouveau fichier]^⚡

• Entrée : { loading: false, error: 'some error', user: { name: 'old' } }
• Sortie : { loading: true, error: null, user: null }
• Vérifie que user est réinitialisé à null (plus strict que loggedIn: false — nettoyage complet)
• error est effacé, loading passe à true — comme dans les anciens tests
• Le remplacement de loggedIn: false par user: null est la différence clé de la structure de l'état

Ancien test « loading state » — SANS PAYLOAD.

Appelait { type: 'USER_LOGIN_SUCCESS' } sans troisième argument (payload). L'ancien réducteur supposément ne faisait que définir loggedIn: true et loading: false — les données utilisateur n'étaient pas conservées.

Suppression du test « weird state » : { loading: false, error: exampleError, loggedIn: true } — combinaison absurde : une erreur existe alors que l'utilisateur est connecté. [ligne 115]

Nouveau test — avec un payload réel :

Pourquoi exactement ces champs : le réducteur dans authentication.js:18-27 attend un payload avec quatre champs — chacun est écrit directement dans l'état :

L'ancien test ne transmettait aucun payload — il ne vérifiait donc aucun de ces champs. Le nouveau test valide la correspondance complète.

loggedIn: true → user: { name: 'testuser' }. Les composants vérifient state.user !== null au lieu de state.loggedIn === true — dans le cadre de la migration vers les react-redux hooks.

Changements analogues : handles USER_LOGIN_FAILURE → USER_LOGIN_FAILURE, suppression de « weird state », un seul test au lieu de deux.

Nouvelle description : 'sets error and clears user/loading' — explicite sur le comportement : l'erreur est conservée, l'utilisateur est réinitialisé, le spinner est retiré.

loggedIn: false → user: null. Le champ user passe à null en cas d'erreur. C'est plus strict que loggedIn: false : l'ancienne approche conservait les données utilisateur dans l'état (elle indiquait simplement qu'il n'était plus connecté), la nouvelle les supprime complètement. Correspond à case USER_LOGIN_FAILURE: user: null.

Formatage du payload : { error: exampleError } sur une seule ligne au lieu de trois. Uniquement du style.

Supprimé en entier (2 tests, ~40 lignes).

Il vérifiait que USER_LOGOUT réinitialisait { loggedIn: true } en { loggedIn: false } et effaçait l'erreur.

Raisons de la suppression :

1. L'action USER_LOGOUT n'existe plus — elle n'est plus exportée depuis authentication.js (seuls USER_LOGIN_BEGIN, USER_LOGIN_FAILURE, USER_LOGIN_SUCCESS sont importés).
2. USER_LOGOUT est absent du switch du réducteur [seulement 3 cases].
3. La fonctionnalité de déconnexion est couverte par USER_LOGIN_BEGIN — les deux réinitialisent l'état.
4. Le test importait USER_LOGOUT depuis une exportation inexistante — le test était cassé.

Historique : USER_LOGOUT a été supprimé des sources dans le commit 7c60c2fbb (2019-07-13, « implemented logout check ») — des actions/authentication.js et du réducteur. Mais le test n'a pas été mis à jour — il a continué à importer et tester USER_LOGOUT pendant encore 7 ans. Dans le nouveau réducteur, la déconnexion n'est plus qu'un simple USER_LOGIN_BEGIN (réinitialisation vers l'état de chargement).

Réducteur Redux

Fonction pure : (state, action) → nouvel état. Aucun effet de bord, aucune mutation. Object.freeze(state) dans le test garantit ce principe : si le réducteur modifie accidentellement l'état transmis, le test échouera. [Documentation Redux]

Structure de l'état (State Shape) : loggedIn → user

Ancien état : { loading, error, loggedIn }. Nouveau : { loading, error, user }. loggedIn: boolean indiquait simplement « oui/non ». user: object|null fournit l'objet utilisateur complet (id, nom, locale, ...) — les composants via useSelector(state => state.authentication.user) récupèrent directement toutes les données.

Object.freeze

Rend l'objet immuable. Dans les tests de réducteur : Object.freeze(state); reducer(state, action); — si le réducteur tente de modifier l'état (au lieu d'en créer un nouveau), JavaScript lèvera une erreur en mode strict. C'est une vérification de la pureté de la fonction. [MDN]