From 470468ee8b33920d03e63e0fb6e0bc9fc296b31b Mon Sep 17 00:00:00 2001
From: Benjamin Levesque <14175665+benjlevesque@users.noreply.github.com>
Date: Fri, 31 Jan 2025 14:03:34 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20PR=20review?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 packages/domain/common/src/valueTypes/email.valueType.ts      | 2 +-
 .../utilisateur/src/identifiantUtilisateur.valueType.ts       | 2 +-
 packages/specifications/src/candidature/candidature.world.ts  | 4 ++--
 .../specifications/src/utilisateur/fixtures/utilisateur.ts    | 4 +---
 4 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/packages/domain/common/src/valueTypes/email.valueType.ts b/packages/domain/common/src/valueTypes/email.valueType.ts
index 89fb3a33cf..e4e1daa8d1 100644
--- a/packages/domain/common/src/valueTypes/email.valueType.ts
+++ b/packages/domain/common/src/valueTypes/email.valueType.ts
@@ -22,7 +22,7 @@ export const bind = ({ email }: PlainType<ValueType>): ValueType => {
 
 export const convertirEnValueType = (value: string): ValueType => {
   return bind({
-    email: value?.toLowerCase(),
+    email: value.toLowerCase(),
   });
 };
 
diff --git a/packages/domain/utilisateur/src/identifiantUtilisateur.valueType.ts b/packages/domain/utilisateur/src/identifiantUtilisateur.valueType.ts
index 466260bd2b..e6be57d99c 100644
--- a/packages/domain/utilisateur/src/identifiantUtilisateur.valueType.ts
+++ b/packages/domain/utilisateur/src/identifiantUtilisateur.valueType.ts
@@ -34,7 +34,7 @@ export const bind = ({ email }: PlainType<ValueType>): ValueType => {
  */
 export const convertirEnValueType = (value: string): ValueType => {
   return bind({
-    email: value?.toLowerCase(),
+    email: value.toLowerCase(),
   });
 };
 
diff --git a/packages/specifications/src/candidature/candidature.world.ts b/packages/specifications/src/candidature/candidature.world.ts
index fc33e9a076..11a4e8d26c 100644
--- a/packages/specifications/src/candidature/candidature.world.ts
+++ b/packages/specifications/src/candidature/candidature.world.ts
@@ -1,5 +1,5 @@
 import { Candidature } from '@potentiel-domain/candidature';
-import { DateTime, Email, IdentifiantProjet } from '@potentiel-domain/common';
+import { DateTime, IdentifiantProjet } from '@potentiel-domain/common';
 import { DocumentProjet } from '@potentiel-domain/document';
 
 import { CorrigerCandidatureFixture } from './fixtures/corrigerCandidature.fixture';
@@ -91,7 +91,7 @@ export class CandidatureWorld {
       dateÉchéanceGf: expectedValues.dateÉchéanceGfValue
         ? DateTime.convertirEnValueType(expectedValues.dateÉchéanceGfValue)
         : undefined,
-      emailContact: Email.convertirEnValueType(expectedValues.emailContactValue.toLowerCase()),
+      emailContact: expectedValues.emailContactValue.toLowerCase(),
       evaluationCarboneSimplifiée: expectedValues.evaluationCarboneSimplifiéeValue,
       historiqueAbandon: Candidature.HistoriqueAbandon.convertirEnValueType(
         expectedValues.historiqueAbandonValue,
diff --git a/packages/specifications/src/utilisateur/fixtures/utilisateur.ts b/packages/specifications/src/utilisateur/fixtures/utilisateur.ts
index 91595ce066..288b40df08 100644
--- a/packages/specifications/src/utilisateur/fixtures/utilisateur.ts
+++ b/packages/specifications/src/utilisateur/fixtures/utilisateur.ts
@@ -1,7 +1,5 @@
 import { faker } from '@faker-js/faker';
 
-import { Email } from '@potentiel-domain/common';
-
 export interface Utilisateur {
   id: string;
   email: string;
@@ -41,7 +39,7 @@ export abstract class AbstractUtilisateur implements Utilisateur {
 
   protected créer(partial?: Partial<Readonly<Omit<Utilisateur, 'role>'>>>): Readonly<Utilisateur> {
     const utilisateur: Utilisateur = {
-      email: Email.convertirEnValueType(faker.internet.email()).formatter(),
+      email: faker.internet.email().toLowerCase(),
       nom: faker.person.fullName(),
       id: faker.string.uuid(),
       ...partial,