fix(domain/utilisateur): add better safeguard to getNombrePersonnesDa…

…nsLogement()
betagouv · Oct 31, 2024 · 8fa9055 · 8fa9055
1 parent b27823e
commit 8fa9055
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/src/domain/logement/logement.ts b/src/domain/logement/logement.ts
@@ -77,7 +77,7 @@ export class Logement {
     this.dpe = this.AorB(input.dpe, this.dpe);
   }
 
-  private undefinedToNull?(val): any {
+  private undefinedToNull?<T>(val: T | undefined): T | null {
     return val === undefined ? null : val;
   }
 

diff --git a/src/domain/utilisateur/utilisateur.ts b/src/domain/utilisateur/utilisateur.ts
@@ -263,11 +263,15 @@ export class Utilisateur extends UtilisateurData {
 
   /**
    * Returns the total number of people in the household, including adults and
-   * children.
+   * children (see {@link UtilisateurData.logement}).
+   *
+   * @ensures The result to be in the range [1, +∞[.
    */
-  public getNombrePersonnesDansLogement?() {
-    const total = this.logement.nombre_adultes + this.logement.nombre_enfants;
-    return total || 1;
+  public getNombrePersonnesDansLogement?(): number {
+    return Math.max(
+      this.logement.nombre_adultes + this.logement.nombre_enfants,
+      1,
+    );
   }
 
   public setPassword?(password: string) {