antonmi/espec

Add option for suppress logs in test cases.

Ouverte

#243 ouverte le 11 oct. 2017

Voir sur GitHub
 (2 commentaires) (0 réaction) (0 personne assignée)Elixir (67 forks)batch import
enhancementhelp wanted

Métriques du dépôt

Stars
 (814 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

It would awesome to have a functionality similar to the ExUnit tags. Especially for the suppression of the logs generated in the tests.

@tag :capture_log
it "should forward the error result and stop" do
  #...
do

It would be useful for cases like this one, where ran three tests successfully but the first one is showing a ugly log.

.2017-10-11 14:46:57.783 nonode@nohost pid=<0.30.0> : Process #PID<0.576.0> raised an exception
** (RuntimeError) crash_test
    spec/orders_service/fsm/recipient_spec.exs:683: anonymous fn/0 in (....)
..

This functionality already exists and is not documented correctly? or could be added in a near future? A function capture_log already exists, but the purpose is different.

Guide contributeur

Direction de recherche
Étudiez comment ExUnit implémente `@tag :capture log` pour la suppression des journaux (voir hexdocs.pm/ex unit). Dans espec, localisez le lanceur de tests et les mécanismes de capture des journaux, probablement dans `lib/espec/runner.ex` ou les fichiers associés. Vérifiez la fonction `capture log` existante et déterminez si elle peut être réutilisée. Envisagez d'ajouter une option pour supprimer les journaux par cas de test via une balise similaire à ExUnit.
Stack technique
Aucun
Domaine
testing
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
ElixirESpecExUnit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70