Skip to content

Commit

Permalink
✅ Couple of fix
Browse files Browse the repository at this point in the history
  • Loading branch information
@VioMrqs
VioMrqs committed Nov 14, 2024
1 parent d6f293c commit 7d86d91
Show file tree
Hide file tree
Showing 6 changed files with 18 additions and 23 deletions.
2 changes: 1 addition & 1 deletion packages/specifications/src/notification/notification.world.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ export class NotificationWorld {
const notif = this.#notifications.find(
(notif) =>
notif.recipients.find((r) => r.email === email) &&
(!sujet || notif.messageSubject === sujet),
(!sujet || notif.messageSubject.match(new RegExp(sujet))),
);
assert(notif, 'Pas de notification');
return notif;
Expand Down
8 changes: 3 additions & 5 deletions packages/specifications/src/notification/stepDefinitions/notification.then.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,11 +8,9 @@ function vérifierEmailEnvoyé(this: PotentielWorld, email: string, data: DataTa
const notif = this.notificationWorld.récupérerNotification(email, exemple.sujet);

for (const [key, value] of Object.entries(exemple)) {
if (key === 'sujet') {
expect(notif.messageSubject).to.equal(value);
} else {
expect(notif.variables[key]).to.match(new RegExp(value));
}
expect(key === 'sujet' ? notif.messageSubject : notif.variables[key]).to.match(
new RegExp(value),
);
}
}

Expand Down
14 changes: 6 additions & 8 deletions .../projet/lauréat/achèvement/attestationConformité/transmettreAttestationConformité.feature
View file
Original file line number Diff line number Diff line change
Expand Up @@ -24,16 +24,14 @@ Fonctionnalité: Transmettre une attestation de conformité
| date | 2024-01-05 |
| mis à jour par | porteur |
Et un email a été envoyé à la dreal avec :
| sujet | Potentiel - Une attestation de conformité a été transmise pour le projet Du boulodrome de Marseille dans le département departementProjet |
| nom_projet | Du boulodrome de Marseille |
| departement_projet | departementProjet |
| url | https://potentiel.beta.gouv.fr/projet/.*/details.html |
| sujet | Potentiel - Une attestation de conformité a été transmise pour le projet Du boulodrome de Marseille dans le département(.*) |
| nom_projet | Du boulodrome de Marseille |
| url | https://potentiel.beta.gouv.fr/projet/.*/details.html |

Et un email a été envoyé au porteur avec :
| sujet | Potentiel - Mise à jour de la date d'achèvement du projet Du boulodrome de Marseille dans le département departementProjet |
| nom_projet | Du boulodrome de Marseille |
| departement_projet | departementProjet |
| url | https://potentiel.beta.gouv.fr/projet/.*/details.html |
| sujet | Potentiel - Mise à jour de la date d'achèvement du projet Du boulodrome de Marseille dans le département(.*) |
| nom_projet | Du boulodrome de Marseille |
| url | https://potentiel.beta.gouv.fr/projet/.*/details.html |

Scénario: Une tâche du type "échoir les garanties financières" est annulée quand une attestation de conformité est transmise
Etant donné des garanties financières actuelles pour le projet "Du boulodrome de Marseille" avec :
Expand Down
8 changes: 3 additions & 5 deletions ...projet/lauréat/garantiesFinancières/actuelles/échoirGarantiesFinancièresActuelles.feature
View file
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,9 @@ Fonctionnalité: Échoir les garanties financières actuelles d'un projet
| à échoir le | 2024-07-18 |
Alors les garanties financières actuelles du projet "Du boulodrome de Marseille" sont échues
Et un email a été envoyé au porteur avec :
| sujet | Potentiel - Date d'échéance dépassée pour les garanties financières du projet Du boulodrome de Marseille dans le département departementProjet |
| nom_projet | Du boulodrome de Marseille |
| departement_projet | departementProjet |
| region_projet | regionProjet |
| date_echeance | 17/07/2024 |
| sujet | Potentiel - Date d'échéance dépassée pour les garanties financières du projet Du boulodrome de Marseille dans le département(.*) |
| nom_projet | Du boulodrome de Marseille |
| date_echeance | 17/07/2024 |

@NotImplemented
Scénario: Impossible d'échoir les garanties financières actuelles d'un projet si il n'y a pas de garanties financières actuelles pour ce projet
Expand Down
1 change: 0 additions & 1 deletion packages/specifications/src/projet/lauréat/lauréat.world.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,6 @@ export class LauréatWorld {

this.#notifierLauréatFixture = new NotifierLauréatFixture();

// this is a problem
this.#identifiantProjet = IdentifiantProjet.convertirEnValueType(`PPE2 - Eolien#1##23`);
}
}
8 changes: 5 additions & 3 deletions packages/specifications/src/utilisateur/stepDefinitions/utilisateur.given.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,9 @@ EtantDonné(

await insérerUtilisateur(id, nom, email, role);

await associerUtilisateurÀSaDreal(id);
const { région } = this.candidatureWorld.importerCandidature.values.localitéValue;

await associerUtilisateurÀSaDreal(id, région);
},
);

Expand Down Expand Up @@ -142,7 +144,7 @@ async function associerProjetAuPorteur(userId: string, projets: { id: string }[]
);
}

async function associerUtilisateurÀSaDreal(userId: string) {
async function associerUtilisateurÀSaDreal(userId: string, régionProjet: string) {
await executeQuery(
`
insert into "userDreals" (
Expand All @@ -158,7 +160,7 @@ async function associerUtilisateurÀSaDreal(userId: string) {
$4
)
`,
'regionProjet',
régionProjet,
userId,
new Date().toISOString(),
new Date().toISOString(),
Expand Down

0 comments on commit 7d86d91

Please sign in to comment.