zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Passe à Django 4.2 #6577

Closed Arnaud-D closed 8 months ago

Arnaud-D commented 10 months ago

Cette PR fait les dernières modifications pour passer à Django 4.2.

Contrôle qualité

Tester un peu tout le site et en particulier les pages associées aux vues qui ont été modifiées.

Arnaud-D commented 9 months ago

J'ai pu retravailler sur les tests qui ne passent pas :

Bonne et mauvaise nouvelle à la fois : en local, ils se comportent mal aussi sur la dernière version de dev, à savoir ils réussissent en isolation mais ratent associés à certains autres tests. Ce n'est donc pas lié à mes modifications. C'est probablement une différence d'environnement entre ma machine et la CI, combinée avec des tests qui interfèrent. Je n'ai pas exploré plus que ça. La principale différence est la version de Python (3.9 sur la CI, 3.10 sur ma machine), mais j'ai du mal à voir la différence que ça ferait.

Une commande minimale qui rate (identifiée pour le deuxième test mentionné ci-dessus) :

python manage.py test zds.member.tests.views.tests_profile.MemberTests.test_list_members zds.member.tests.views.tests_moderation.IpListingsTests --settings zds.settings.test

Sachant que dans test_list_members, la ligne incriminée est weird = ProfileFactory() : en la commentant, plus de souci sur l'autre test.

En comparaison, cette commande réussit :

python manage.py test zds.member.tests.views.tests_moderation.IpListingsTests --settings zds.settings.test
coveralls commented 9 months ago

Coverage Status

coverage: 88.661% (-0.003%) from 88.664% when pulling 7c8e067ec84e0d8e08ef791ff8866ed2c7ac02aa on Arnaud-D:django4.2 into 57a4c11f9e8610166a8ec487beea0c9f12694b3e on zestedesavoir:dev.

Arnaud-D commented 9 months ago

Les tests passent !