Actions
Anomalie #1612
fermé
Mistakes in reminders' unit tests
Début:
09/01/2022
Echéance:
% réalisé:
100%
Temps estimé:
Version utilisée:
Vote:
Description
After a first rereading :
- https://bugs.galette.eu/projects/galette/repository/62/revisions/master/entry/tests/Galette/Repository/tests/units/Reminders.php#L141 : should not subtract a day but add it to test the day before the first day of impendings
- https://bugs.galette.eu/projects/galette/repository/62/revisions/master/entry/tests/Galette/Repository/tests/units/Reminders.php#L160 : no reminder should normally be sent in this case since we test the day before the first day of impendings
- https://bugs.galette.eu/projects/galette/repository/62/revisions/master/entry/tests/Galette/Repository/tests/units/Reminders.php#L171 : one additional day should not be subtracted to test the very first day of impendings
- https://bugs.galette.eu/projects/galette/repository/62/revisions/master/entry/tests/Galette/Repository/tests/units/Reminders.php#L293 : for a contribution that expires the same day it is created, the member should not be up to date. And reminders for close and late deadlines are not checked; no reminder should normally be sent in this case
Actions
#1
Mis à jour par Guillaume AGNIERAY il y a presque 3 ans
- Lié à Anomalie #1590: Incohérence rappels échéances proches et en retard ajouté
Actions
#2
Mis à jour par Guillaume AGNIERAY il y a presque 3 ans
Corresponding MR : https://framagit.org/galette/galette/-/merge_requests/14
First, these changes show a mistake in code here : https://bugs.galette.eu/projects/galette/repository/62/revisions/master/entry/galette/lib/Galette/Entity/Adherent.php#L1002
Should be fixed with : return $ech > $now;
Actions
#3
Mis à jour par Guillaume AGNIERAY il y a presque 3 ans
- Lié à Anomalie #1614: due date reminder emails ajouté
Actions
#4
Mis à jour par Johan Cwiklinski il y a presque 3 ans
Guillaume Agnieray a écrit (#note-2):
First, these changes show a mistake in code here : https://bugs.galette.eu/projects/galette/repository/62/revisions/master/entry/galette/lib/Galette/Entity/Adherent.php#L1002
Should be fixed with :return $ech > $now;
This was the only fail; and the isUp2Date() method is not tested directly outside of reminders tests... :)
Actions
#5
Mis à jour par Guillaume AGNIERAY il y a presque 3 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit 0a1744af68aff5be8b46b4826ba414ee048492e4.
Actions
#6
Mis à jour par Johan Cwiklinski il y a plus d'un an
- Statut changé de Résolu à Fermé
- Assigné à mis à Guillaume AGNIERAY
- Version cible mis à 1.0.0
Actions