Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Passe à Django 4.2 #6577

Merged
merged 7 commits into from
Mar 3, 2024
Merged

Passe à Django 4.2 #6577

merged 7 commits into from
Mar 3, 2024

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Jan 18, 2024

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 Arnaud-D added the C-Back Concerne le back-end Django label Jan 18, 2024
@Arnaud-D
Copy link
Contributor Author

Arnaud-D commented Feb 10, 2024

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

  • test_hat_detail (zds.member.tests.views.tests_hats.HatTests)
  • test_same_ipv4 (zds.member.tests.views.tests_moderation.IpListingsTests)

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

@Arnaud-D Arnaud-D force-pushed the django4.2 branch 2 times, most recently from c6eb3a3 to b907123 Compare February 24, 2024 18:39
@coveralls
Copy link

coveralls commented Feb 25, 2024

Coverage Status

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

@Arnaud-D
Copy link
Contributor Author

Les tests passent !

@Arnaud-D Arnaud-D marked this pull request as ready for review February 25, 2024 06:10
@Arnaud-D Arnaud-D force-pushed the django4.2 branch 2 times, most recently from 4a0789b to 7e1730c Compare March 2, 2024 16:22
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

J'ai testé les vues modifiées, tout est OK. J'ai ajouté un commit qui récupère les fichiers .js.map pour tous les fichiers JS, puisqu'on en a maintenant besoin lorsqu'on exécute manage.py collectstatic lors du déploiement de zds-site.

@philippemilink philippemilink merged commit a932053 into zestedesavoir:dev Mar 3, 2024
12 checks passed
@Arnaud-D Arnaud-D deleted the django4.2 branch March 3, 2024 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants