StackStorm/st2

Add tests for all linux pack actions

Open

#4 999 ouverte le 20 juil. 2020

Voir sur GitHub
 (3 commentaires) (1 réaction) (0 assignés)Python (740 forks)batch import
bugcomplexity:easygood first issuepackstests

Métriques du dépôt

Stars
 (5 794 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

The contrib linux pack has a few broken actions (check_loadavg, check_processes, dig: #4993, diag_loadavg, netstat: #4947, possibly more) at least on CentOS 8. It's really disheartening to spin up StackStorm and immediately run into broken actions.

We should add tests for those actions to the pack.

The tests from the core pack can be used as a template for writing tests for the linux pack.

I will try to add tests for the dig action in my PR #4993, to serve as a concrete example, but adding tests would be a useful way to introduce somebody to developing StackStorm that would have an immediate impact.

Guide contributeur

Direction de recherche
Examinez les tests existants dans contrib/core/tests comme modèle. Étudiez les actions défectueuses (check loadavg, check processes, dig, diag loadavg, netstat) et leur comportement actuel sur CentOS 8. La PR #4993 fournit un exemple pour l'action dig. Rédigez des tests similaires pour chaque action, en vous concentrant sur la vérification des sorties attendues et la gestion des erreurs.
Stack technique
pythonshell
Domaine
testingbackend
Type d'issue
Test
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Python testing basicsStackStorm pack structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70