toulouse-maelis: do not crash linking RL1 with no birtdate (#80922) #341

nroche · 2023-09-05T20:21:44+02:00

nroche commented

2023-09-05 20:21:44 +02:00

On peut avoir des RL1 qui n'ont pas de date de naissance.
Ça a été corrigé au fils de l'eau dans le connecteur, mais pas encore sur l’appairage.
https://redmine.sigec.fr/issues/2758

Dans le test je prévois le cas où la date de naissance est invalide pour ne pas multiplier les tests,
mais le code prend également en charge le cas où la date remontée est nulle.
(par excès de zèle je me suis dit que maélis pourrais tout aussi bien nous remonter une mauvaise date).

On peut avoir des RL1 qui n'ont pas de date de naissance. Ça a été corrigé au fils de l'eau dans le connecteur, mais pas encore sur l’appairage. https://redmine.sigec.fr/issues/2758 Dans le test je prévois le cas où la date de naissance est invalide pour ne pas multiplier les tests, mais le code prend également en charge le cas où la date remontée est nulle. (par excès de zèle je me suis dit que maélis pourrais tout aussi bien nous remonter une mauvaise date).

fpeters commented

2023-09-05 20:29:30 +02:00

Dans le test je prévois le cas où la date de naissance est invalide pour ne pas multiplier les tests,
mais le code prend également en charge le cas où la date remontée est nulle.
(par excès de zèle je me suis dit que maélis pourrais tout aussi bien nous remonter une mauvaise date).

Mais donc tu fais un test sur justement le cas qui n'arrive pas en vrai ? Je trouverais plus logique de faire le test sur la situation réellement rencontrée (i.e. date nulle).

> Dans le test je prévois le cas où la date de naissance est invalide pour ne pas multiplier les tests, > mais le code prend également en charge le cas où la date remontée est nulle. > (par excès de zèle je me suis dit que maélis pourrais tout aussi bien nous remonter une mauvaise date). Mais donc tu fais un test sur justement le cas qui n'arrive pas en vrai ? Je trouverais plus logique de faire le test sur la situation réellement rencontrée (i.e. date nulle).

smihai requested changes 2023-09-05 21:24:54 +02:00

smihai left a comment

datebirth => dateBirth dans le message d'erreur pour avoir le nom de l'attribut précis.

nroche force-pushed wip/80922-parsifal-link-error-on-birthdate from ca6b7da264 to 4089c98044

2023-09-06 17:04:03 +02:00

Compare

nroche force-pushed wip/80922-parsifal-link-error-on-birthdate from 4089c98044 to 564affa590

2023-09-06 17:07:38 +02:00

Compare

nroche commented

2023-09-06 17:10:28 +02:00

Je trouverais plus logique de faire le test sur la situation réellement rencontrée (i.e. date nulle).

J'ai fait ça : comme dans le ticket lié, le test reçoit à présent une structure dépourvue du champ birthDate.

datebirth => dateBirth dans le message d'erreur pour avoir le nom de l'attribut précis.

J'ai fait ça aussi.

> Je trouverais plus logique de faire le test sur la situation réellement rencontrée (i.e. date nulle). J'ai fait ça : comme dans le ticket lié, le test reçoit à présent une structure dépourvue du champ birthDate. > datebirth => dateBirth dans le message d'erreur pour avoir le nom de l'attribut précis. J'ai fait ça aussi.

nroche requested review from smihai 2023-09-15 17:48:05 +02:00

smihai approved these changes 2023-09-18 11:27:41 +02:00

nroche force-pushed wip/80922-parsifal-link-error-on-birthdate from 564affa590 to fd1c591ab3

2023-09-18 16:32:37 +02:00

Compare

nroche merged commit fd1c591ab3 into main

2023-09-18 17:10:27 +02:00

nroche deleted branch wip/80922-parsifal-link-error-on-birthdate

2023-09-18 17:10:27 +02:00

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

3 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#341