Skip to content

Commit

Permalink
clean up methode creation utilisateur
Browse files Browse the repository at this point in the history
  • Loading branch information
@WoWo79
WoWo79 committed Oct 13, 2024
1 parent 47b4dd0 commit 5de150e
Show file tree
Hide file tree
Showing 12 changed files with 10 additions and 167 deletions.
19 changes: 7 additions & 12 deletions src/domain/utilisateur/utilisateur.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -117,18 +117,13 @@ export class Utilisateur extends UtilisateurData {
}

public static createNewUtilisateur(
nom: string,
prenom: string,
email: string,
annee_naissance: number,
code_postal: string,
commune: string,
is_magic_link: boolean,
source_inscription: SourceInscription,
): Utilisateur {
return new Utilisateur({
nom: nom,
prenom: prenom,
nom: null,
prenom: null,
email: email,
id: undefined,
code_departement: null,
Expand Down Expand Up @@ -163,8 +158,8 @@ export class Utilisateur extends UtilisateurData {
dpe: null,
plus_de_15_ans: null,
chauffage: null,
code_postal: code_postal,
commune: commune,
code_postal: null,
commune: null,
nombre_adultes: null,
nombre_enfants: null,
proprietaire: null,
Expand All @@ -175,14 +170,14 @@ export class Utilisateur extends UtilisateurData {
force_connexion: false,
derniere_activite: new Date(),
missions: new MissionsUtilisateur(),
annee_naissance: annee_naissance,
annee_naissance: null,
db_version: 0,
bilbiotheque_services: new BibliothequeServices(),
is_magic_link_user: is_magic_link,
rank: null,
rank_commune: null,
code_postal_classement: code_postal,
commune_classement: commune,
code_postal_classement: null,
commune_classement: null,
points_classement: 0,
status: UtilisateurStatus.default,
couverture_aides_ok: false,
Expand Down
5 changes: 0 additions & 5 deletions src/usecase/inscription.usecase.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -46,12 +46,7 @@ export class InscriptionUsecase {
Utilisateur.checkEmailFormat(utilisateurInput.email);

const utilisateurToCreate = Utilisateur.createNewUtilisateur(
null,
null,
utilisateurInput.email,
null,
null,
null,
false,
utilisateurInput.source_inscription || SourceInscription.inconnue,
);
Expand Down
5 changes: 0 additions & 5 deletions src/usecase/magicLink.usecase.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -78,12 +78,7 @@ export class MagicLinkUsecase {

if (!utilisateur) {
utilisateur = Utilisateur.createNewUtilisateur(
'NOM',
'PRENOM',
email,
null,
null,
null,
true,
SourceInscription.inconnue,
);
Expand Down
35 changes: 0 additions & 35 deletions test/integration/personalisation.int-spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,7 @@ describe('Personalisation', () => {
it('perso : ne touche pas une chaine quelconque', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'W',
'George',
'g@www.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand All @@ -46,12 +41,7 @@ describe('Personalisation', () => {
it('perso : ne bug pas sur undefined , null, et autres types', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'W',
'George',
'g@www.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand All @@ -67,12 +57,7 @@ describe('Personalisation', () => {
it('perso : remplace COMMUNE OK', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'W',
'George',
'g@www.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand All @@ -93,12 +78,7 @@ describe('Personalisation', () => {
it('perso : remplace COMMUNE dans un sous objet', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'W',
'George',
'g@www.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand All @@ -118,12 +98,7 @@ describe('Personalisation', () => {
it('perso : remplace COMMUNE dans une liste de objets', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'W',
'George',
'g@www.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand All @@ -147,12 +122,7 @@ describe('Personalisation', () => {
it('perso : remplace COMMUNE dans une liste de strings', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'W',
'George',
'g@www.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand All @@ -170,12 +140,7 @@ describe('Personalisation', () => {
it('perso : préserve les dates', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'W',
'George',
'g@www.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand Down
5 changes: 0 additions & 5 deletions test/integration/repository/emailTemplate.repository.int-spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,7 @@ describe('EmailTemplateRepository', () => {
it('template email code inscription', async () => {
// GIVEN
const utilisateur = Utilisateur.createNewUtilisateur(
'Clooney',
'George',
'g@c.com',
1967,
'91120',
'PALAISEAU',
false,
SourceInscription.web,
);
Expand Down
5 changes: 0 additions & 5 deletions test/integration/repository/utilisateur.repository.int-spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -69,12 +69,7 @@ describe('UtilisateurRepository', () => {
it('creation et lecture , versionning des donnes json ', async () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'pierre',
'paul',
'w@w.com',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand Down
5 changes: 0 additions & 5 deletions test/unit/domain/gamification/gamification.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,7 @@ describe('Gamification', () => {
// GIVEN
const gamification = new Gamification();
const user = Utilisateur.createNewUtilisateur(
'a',
'b',
'c',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand Down
5 changes: 0 additions & 5 deletions test/unit/domain/mission/mission.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -62,12 +62,7 @@ describe('Missions', () => {
it('validateDefi : terminaison de mission', () => {
// GIVEN
const utilisateur = Utilisateur.createNewUtilisateur(
'C',
'George',
'mail@www.com',
12345,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand Down
45 changes: 0 additions & 45 deletions test/unit/domain/notifications/notificationHistory.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -59,12 +59,7 @@ describe('NotificationHistory', () => {
process.env.NOTIFICATIONS_MAIL_ACTIVES = 'welcome,late_onboarding';

const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand All @@ -85,12 +80,7 @@ describe('NotificationHistory', () => {
// GIVEN
process.env.NOTIFICATIONS_MAIL_ACTIVES = '';
const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand All @@ -111,12 +101,7 @@ describe('NotificationHistory', () => {
// GIVEN
process.env.NOTIFICATIONS_MAIL_ACTIVES = 'welcome,late_onboarding';
const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand All @@ -139,12 +124,7 @@ describe('NotificationHistory', () => {
// GIVEN
process.env.NOTIFICATIONS_MAIL_ACTIVES = 'welcome,late_onboarding';
const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand Down Expand Up @@ -173,12 +153,7 @@ describe('NotificationHistory', () => {
// GIVEN
process.env.NOTIFICATIONS_MAIL_ACTIVES = 'welcome,late_onboarding';
const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand All @@ -200,12 +175,7 @@ describe('NotificationHistory', () => {
// GIVEN
process.env.NOTIFICATIONS_MAIL_ACTIVES = 'welcome,late_onboarding';
const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand All @@ -231,12 +201,7 @@ describe('NotificationHistory', () => {
// GIVEN
process.env.NOTIFICATIONS_MAIL_ACTIVES = 'welcome,late_onboarding';
const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand All @@ -260,12 +225,7 @@ describe('NotificationHistory', () => {
// GIVEN
process.env.NOTIFICATIONS_MAIL_ACTIVES = 'welcome,late_onboarding';
const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand Down Expand Up @@ -330,12 +290,7 @@ describe('NotificationHistory', () => {
});

const utilisateur = Utilisateur.createNewUtilisateur(
'yo',
'prenom',
'toto@dev.com',
1979,
'91120',
'PALAISEAU',
false,
SourceInscription.mobile,
);
Expand Down
10 changes: 0 additions & 10 deletions test/unit/domain/object_store/Gamification_vN.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -25,12 +25,7 @@ describe('Gamification vN ', () => {
it('serialise <=> deserialise v0 OK', () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'a',
'b',
'c',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand All @@ -47,12 +42,7 @@ describe('Gamification vN ', () => {
it('serialise <=> upgade <=> deserialise v0 OK', () => {
// GIVEN
const user = Utilisateur.createNewUtilisateur(
'a',
'b',
'c',
1234,
'91120',
'PALAISEAU',
false,
SourceInscription.inconnue,
);
Expand Down
Loading

0 comments on commit 5de150e

Please sign in to comment.