search

betagouv / betaGouvBot

Automated assistant for beta.gouv.fr administrative tasks on slack.
https://beta.gouv.fr
MIT License
4 stars 10 forks source link

Fix production bug #100

Closed Morendil closed 7 years ago

bonjourmauko commented 7 years ago

Et le test ?

Morendil commented 7 years ago

Notre couverture de test est actuellement incomplète, elle n'exerce pas le casexecute == true. Cette PR fait un tout petit peu plus qu'un hotfix: elle extrait la variable locale all_actions de façon à ce que webhook_spec puisse détecter le problème. Le refactoring a augmenté la couverture de tests, pour moi c'est suffisant dans l'immédiat.

Pour faire mieux il faudrait extraire le contrat ("executer les actions si et seulement si le secret passé via HTTP correspond à celui stocké dans l'environnement") et rédiger un test de contrat.