SE-PRG/azmitool

unit testing - step 2

Aperta

#151 aperta il 16 mag 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C# (5 fork)auto 404
exceptions-handlinggood first issuetesting

Metriche repository

Star
 (2 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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

Guida contributor

Direzione di ricerca
Esamina i test unitari esistenti nel progetto. Segui la checklist: aggiungi test negativi, mocka il metodo GetToken nei test IDCheck, unisci test simili in Theory, allinea i test al pattern AAA e migliora i nomi dei test. Inizia comprendendo la struttura attuale dei test e le convenzioni di xUnit.
Tech stack
csharpazure
Dominio
backendsecurity
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Unit testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70