SE-PRG/azmitool

unit testing - step 2

Offen

#151 geöffnet am 16.05.2020

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C# (5 Forks)auto 404
exceptions-handlinggood first issuetesting

Repository-Metriken

Stars
 (2 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandenen Unit Tests im Projekt. Befolgen Sie die Checkliste: Fügen Sie Negativtests hinzu, mocken Sie die GetToken Methode in IDCheck Tests, fassen Sie ähnliche Tests in Theory zusammen, richten Sie Tests nach dem AAA Muster aus und verbessern Sie die Testnamen. Beginnen Sie damit, die aktuelle Teststruktur und die xUnit Konventionen zu verstehen.
Tech Stack
csharpazure
Domain
backendsecurity
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C#Unit testing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70