paypal/squbs

Clean up test cases for #175

Open

#182 geöffnet am 9. Feb. 2016

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)Scala (248 Forks)batch import
help wanted

Repository-Metriken

Stars
 (1.384 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

The test cases may still be a little hard to understand and some of the tests would always be passing. We need to ensure the test cases are valid.

Contributor Guide

Research-Richtung
Überprüfe zunächst Issue #175, um die zu testende Funktion zu verstehen. Finde dann die vorhandenen Testdateien im squbs Repository und identifiziere alle Tests, die immer bestehen oder schlecht strukturiert sind. Verbessere diese Tests, indem du aussagekräftige Assertions hinzufügst und die Testlogik klärst. Überprüfe die Kommentare im Issue auf spezifische Anleitungen der Maintainer.
Tech Stack
Keine
Domain
backendtesting
Issue Type
Test
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
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.
Meist klar
Voraussetzungen
ScalaAkka
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65