Skip to content

Commit

Permalink
[CORRECTION] N'affiche pas d'infos utilisateur si inexistantes
Browse files Browse the repository at this point in the history
Depuis le dernier commit, on ne vérifiait plus la présent du JWT session
FC+ dans les infos utilisateurs (vu qu'il n'y était plus)… mais cette
vérification permettait de s'assurer qu'on n'affichait pas d'infos
utilisateur si le prénom et nom d'usage était inexistants (cas où
l'utilisateur revient sur la page du site vitrine en appuyant sur le
bouton « retour arrière » du navigateur).

Ce commit corrige la régression.

Co-authored-by: Fabien Lamarque <[email protected]>
  • Loading branch information
egaillot and Fabinout committed Jul 16, 2024
1 parent d946f26 commit 1a83b28
Show file tree
Hide file tree
Showing 3 changed files with 19 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/erreurs.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
class ErreurAbsenceReponseDestinataire extends Error {}
class ErreurAucunMessageDomibusRecu extends Error {}
class ErreurDestinataireInexistant extends Error {}
class ErreurDonneeManquante extends Error {}
class ErreurEchecAuthentification extends Error {}
class ErreurInstructionSOAPInconnue extends Error {}
class ErreurJetonInvalide extends Error {}
Expand All @@ -10,6 +11,7 @@ module.exports = {
ErreurAbsenceReponseDestinataire,
ErreurAucunMessageDomibusRecu,
ErreurDestinataireInexistant,
ErreurDonneeManquante,
ErreurEchecAuthentification,
ErreurInstructionSOAPInconnue,
ErreurJetonInvalide,
Expand Down
6 changes: 6 additions & 0 deletions src/modeles/utilisateur.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,11 @@
const { ErreurDonneeManquante } = require('../erreurs');

class Utilisateur {
constructor(donnees) {
if (typeof donnees.prenom === 'undefined' && typeof donnees.nomUsage === 'undefined') {
throw new ErreurDonneeManquante("Prénom et nom d'usage obligatoires pour instancier un utilisateur");
}

this.prenom = donnees.prenom;
this.nomUsage = donnees.nomUsage;
}
Expand Down
11 changes: 11 additions & 0 deletions test/modeles/utilisateur.spec.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,19 @@
const { ErreurDonneeManquante } = require('../../src/erreurs');
const Utilisateur = require('../../src/modeles/utilisateur');

describe("L'utilisateur courant", () => {
it("sait s'afficher", () => {
const utilisateur = new Utilisateur({ prenom: 'Juliette', nomUsage: 'Haucourt' });
expect(utilisateur.afficheToi()).toBe('Juliette Haucourt');
});

it("vérifie qu'il est initialisé avec un nom d'usage et un prénom", () => {
expect.assertions(1);

try {
new Utilisateur({});
} catch (e) {
expect(e).toBeInstanceOf(ErreurDonneeManquante);
}
});
});

0 comments on commit 1a83b28

Please sign in to comment.