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

Ajoute des tests et refactorise AddSuggestion #6190

Merged
merged 1 commit into from
Feb 18, 2022

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Oct 9, 2021

Dans la suite de #6188, cette PR :

  • rajoute des tests pour la vue AddSuggestion ;
  • corrige un bug (déjà hotfixé) qui permettait à des auteurs motivés de faire des modifs alors que c'est normalement destiné au staff.

Contrôle qualité

S'assurer de bien avoir Elastic Search d'installé (nécessaire pour ajouter des suggestions facilement).

  • se connecter avec un compte staff
  • créer des suggestions sur un contenu :
    • normalement (une ou plusieurs suggestions) et constater que ça fonctionne bien
    • envoyer des choses invalides (par exemple en bidouillant le formulaire) : vide, pas un id existant, un id qui n'est pas un nombre et observer ce qu'il se passe (il peut y avoir des 404 et des erreurs internes, je n'ai pas changé les comportements bruts de décoffrage actuels) ;
      *envoyer des suggestions interdites tels que des billets non-validés, des contenus non-publiés, des suggestions en double, suggérer le contenu lui-même et observer les erreurs associées.

Pour ce qui est du bug de passer outre l'autorisation staff, il suffit d'envoyer une requête en tant qu'auteur avec le bon formulaire (avec l'inspecteur prendre le formulaire staff, l'injecter sur une page en tant qu'auteur, en prenant soin de changer le token csrf et envoyer). Il faut donc tester que ce n'est plus possible.

@Arnaud-D Arnaud-D added C-Back Concerne le back-end Django hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest labels Oct 9, 2021
@coveralls
Copy link

coveralls commented Oct 9, 2021

Coverage Status

Coverage increased (+0.1%) to 87.162% when pulling 9591115 on Arnaud-D:it_is_suggested_to_test_more into 7b389ba on zestedesavoir:dev.

@artragis artragis merged commit 337fb40 into zestedesavoir:dev Feb 18, 2022
@Arnaud-D Arnaud-D deleted the it_is_suggested_to_test_more branch February 18, 2022 10:16
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