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

Modifie l'utilisation d'assertQuerysetEqual dans un test (préparation pour Django 4) #6542

Merged

Conversation

Arnaud-D
Copy link
Contributor

Ce commit modifie l'utilisation d'assertQuerysetEqual dans un test en :

  • retirant la comparaison avec une liste de repr() (retiré dans Django 4.1)
  • la remplaçant par une comparaison directe avec un Queryset (ajouté dans Django 3.2)

Contrôle qualité

La CI devrait suffire.

@Arnaud-D Arnaud-D added the C-Back Concerne le back-end Django label Oct 14, 2023
… pour Django 4)

* retire la comparaison avec une liste de repr() (retiré dans Django 4.1)
* remplace par une comparaison directe avec un Queryset (ajouté dans Django 3.2)
@Arnaud-D Arnaud-D force-pushed the prepa-django4-fix-assertequal branch from 9c34241 to e228505 Compare October 14, 2023 06:35
@coveralls
Copy link

coveralls commented Oct 14, 2023

Coverage Status

coverage: 88.56% (-0.01%) from 88.571% when pulling e228505 on Arnaud-D:prepa-django4-fix-assertequal into f547a2f on zestedesavoir:dev.

@Arnaud-D Arnaud-D added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 14, 2023
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 ✔️

@philippemilink philippemilink merged commit 4d3fefb into zestedesavoir:dev Oct 14, 2023
12 checks passed
@Arnaud-D Arnaud-D deleted the prepa-django4-fix-assertequal branch October 15, 2023 12:47
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 hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants