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

Supprime le fichier .svgz généré pendant un test #6463

Merged

Conversation

philippemilink
Copy link
Member

Lors de l'exécution du test zds.gallery.api.tests.ImageListAPITest.test_post_fail_add_image_not_an_image, un fichier est généré pour les besoins du test. Ce fichier n'était pas supprimé à la fin du test et restait donc dans les sources de zds-site. Cette PR supprime ce fichier à la fin du test.

Contrôle qualité

Avant de passer sur le code de la PR, exécuter le test en question :

python manage.py test zds.gallery.api.tests.ImageListAPITest.test_post_fail_add_image_not_an_image

Constater qu'un fichier .svgz est apparu à la racine des sources du projet (ça se voit bien avec git status).

Supprimer manuellement ce fichier, passer sur le code de la PR, ré-exécuter le test : il n'y a pas de fichier .svgz.

Pour s'assurer que le fichier est également supprimé si une exception se produit dans le test, on peut ajouter print(1/0) juste avant la ligne du finally pour générer exprès une erreur.

@philippemilink philippemilink added C-Back Concerne le back-end Django C-DevelopmentEnv Amélioration de l'environnement de dev labels Feb 19, 2023
@coveralls
Copy link

coveralls commented Feb 19, 2023

Coverage Status

Coverage: 88.385%. Remained the same when pulling 5fbdf40 on philippemilink:supprime-fichier-test into 21413cc on zestedesavoir:dev.

@Migwel
Copy link
Contributor

Migwel commented May 14, 2023

QA OK

J'ai lancé le test et le fichier est bien supprimé une fois le test terminé. J'ai ensuite ajouté une exception dans le test pour le faire échouer et dans ce cas aussi, le fichier a bien été supprimé

@philippemilink philippemilink enabled auto-merge (squash) May 14, 2023 11:55
@philippemilink
Copy link
Member Author

Merci @Migwel pour la QA. Tu peux approuver la PR, stp ? (onglet Files changed, bouton Review changes en haut à droite, Approve et Submit review) C'est nécessaire pour pouvoir merger sans que j'ai à abuser de mes super-pouvoirs :)

@philippemilink philippemilink merged commit 1fe24f2 into zestedesavoir:dev May 14, 2023
@philippemilink philippemilink deleted the supprime-fichier-test branch May 14, 2023 13:13
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 C-DevelopmentEnv Amélioration de l'environnement de dev
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants