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

Vérifie la présence des comptes essentiels au démarrage de l'application #6369

Merged
merged 4 commits into from
Aug 15, 2022

Conversation

Arnaud-D
Copy link
Contributor

Fix #4872.

Reprise de #4873.

  • Ajoute une vérification de la présence des comptes essentiels (bot + auteur externe + anonyme) au démarrage de l'application
  • Simplifie la récupération des comptes bots dans tout le code
  • Déplace les bots dans un fichier de fixtures dédié.

Contrôle qualité

Vérifier le déclenchement de l'erreur :

  • Supprimer la base de donnée (locale)
  • Appliquer les migrations (sans les fixtures)
  • Vérifier qu'on a une exception qui correspond à l'absence d'un compte

Ensuite, vérifier l'absence de l'erreur :

  • générer les fixtures (ça importe les comptes essentiels)
  • vérifier que ça démarre comme il faut

@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django C-DevelopmentEnv Amélioration de l'environnement de dev labels Jul 31, 2022
@Arnaud-D Arnaud-D changed the title Vérifie que la présence des comptes essentiels au démarrage de l'application Vérifie la présence des comptes essentiels au démarrage de l'application Jul 31, 2022
@coveralls
Copy link

coveralls commented Jul 31, 2022

Coverage Status

Coverage increased (+0.02%) to 88.206% when pulling bc21658 on Arnaud-D:fix-4872 into f491ee2 on zestedesavoir:dev.

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.

Rapport de QA

OK ✔️

Juste le petit changement demandé et je merge (au passage, un rebase serait le bienvenu !).

zds/utils/misc.py Outdated Show resolved Hide resolved
@philippemilink philippemilink merged commit 06b2853 into zestedesavoir:dev Aug 15, 2022
@Arnaud-D Arnaud-D deleted the fix-4872 branch September 5, 2022 12:12
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
None yet
Development

Successfully merging this pull request may close these issues.

Le compte du bot est-il essentiel au fonctionnement du site ?
3 participants