ankidroid/Anki-Android

[Cleanup]: Add Tests to the code (`@NeedsTest`)

Open

#13.283 geöffnet am 16. Feb. 2023

Auf GitHub ansehen
 (37 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Kotlin (2.789 Forks)batch import
Good First IssueKeep OpenTests

Repository-Metriken

Stars
 (11.132 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 9T 12h) (93 gemergte PRs in 30 T)

Beschreibung

[!NOTE] This issue does not need to be assigned to you before you work on it

A number of paths of the code have been identified as requiring automated testing.

Typically this is regression testing: if a bug has been found, then it's useful to have a unit test which would have failed due to the bug.

  1. Find a usage of @NeedsTest: https://github.com/ankidroid/Anki-Android/blob/main/AnkiDroid/src/main/java/com/ichi2/annotations/NeedsTest.kt a. Ensure that the method(s) you are testing are in use.
  2. Post the test(s) that you're handling on this issue, so people don't duplicate work
  3. Write a unit test (/test, not /androidTest)
  4. Ensure that re-introducing the bug or issue would break the test
  5. Submit a Pull Request

Contributor Guide

Research-Richtung
Finde eine Methode, die mit @NeedsTest annotiert und in Verwendung ist, schreibe einen Unit Test unter /test, der fehlschlagen würde, wenn der Fehler wieder auftritt, und reiche einen PR ein.
Tech Stack
kotlin
Domain
mobile
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.
Klar
Voraussetzungen
GitKotlinUnit Testing
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80