Development #65806
holidays: régression dans les tests
0%
Description
Entre les builds 2891 et 2890.
@with_httmock(ics_data)
def test_holidays_ics(app):
connector = Holidays.objects.create(slug='test', zone='a', holidays=[])
resp = app.get('/holidays/test/holidays.ics')
calendar = vobject.readOne(resp.text)
assert not 'vevent' in calendar.contents
connector.holidays.append('winter_holidays')
connector.save()
resp = app.get('/holidays/test/holidays.ics')
calendar = vobject.readOne(resp.text)
> first_event = calendar.contents['vevent'][0]
E KeyError: 'vevent'
tests/test_holidays.py:40: KeyError
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
Ok c'est pété parce que dans les fichiers d'origine les descriptions ont changé : 'Vacances de la Toussaint' est devenu 'Vacances de la Toussaint - Zone A'.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-tests-fix-holidays-request-mock-65806.patch 0001-tests-fix-holidays-request-mock-65806.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Le bug ici c'est que le mock est foireux, on ne devrait pas aller chercher les fichiers sur internet.
Mais le connecteur est effectivement cassé, et en plus d'avoir ajouté le suffixe « - Zone A » partout il y a
- DESCRIPTION:Vacances d'Été (Enseignants) + DESCRIPTION:Vacances d'Été (prérentrée Enseignants ) - Zone A
On ignorais de toute façon cet évènement mais l'espace avant la parenthèse me fait dire que ça a été fait à l'arrache et que ça sera peut-être revu bientôt, je vais faire un autre ticket et attendre un peu pour corriger.
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
Je n'étais pas fan de ce connecteur dès le départ, mais même avec, est-ce qu'on ne gagnerait pas à republier ces fichiers retravaillés sur une URL à nous ? (genre https://www.entrouvert.org/calendars/holidays-french-zone-a.ics), généré à partir d'un script qui chouinerait par mail si il ne trouve plus ses noms de vacances et laisserait la dernière version qui marche en place (et au pire on pourrait maintenir le fichier à la main, une fois par an et pas s'emmerder avec opendatasoft qui peut faire faillite demain ou changer les URLs ou être racheté par Microsoft ou whatever).
C'est dans la même veine que l'idée d'héberger notre propre instance de la BAN un de ces jours.
Mis à jour par Benjamin Dauvergne il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Je valide ce patch histoire que les tests passent, tu veux un autre ticket pour la correction au connecteur ?
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit ab3aa9b6b4886f300b15f5bcdbbf1a201c627f78
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue May 31 14:19:00 2022 +0200
tests: fix holidays request mock (#65806)Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: fix holidays request mock (#65806)