SE-PRG/azmitool

unit testing - step 2

Ouverte

#151 ouverte le 16 mai 2020

Voir sur GitHub
 (0 commentaire) (0 réaction) (0 personne assignée)C# (5 forks)auto 404
exceptions-handlinggood first issuetesting

Métriques du dépôt

Stars
 (2 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

This is continuing from issue #135 and PR #149

Next tasks in unit testing:

  • Add negative tests where possible
  • Add mock for GetToken in IDCheck tests
  • fix other TODO: issues
  • merge similar tests into Theory
  • align test cases according to AAA approach (arrange-act-assert) (three lines of code)
  • add more descriptive test names (see comment in PR #149 )

Pipeline related tasks

  • Confirm pipeline is integrated as PR validation
  • re-check code coverage in pipeline

Guide contributeur

Direction de recherche
Examinez les tests unitaires existants dans le projet. Suivez la liste de contrôle : ajoutez des tests négatifs, simulez la méthode GetToken dans les tests IDCheck, fusionnez les tests similaires en Theory, alignez les tests sur le modèle AAA et améliorez les noms de tests. Commencez par comprendre la structure actuelle des tests et les conventions xUnit.
Stack technique
csharpazure
Domaine
backendsecurity
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.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C#Unit testing
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70