ankidroid/Anki-Android

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

Open

#13.283 aperta il 16 feb 2023

Vedi su GitHub
 (37 commenti) (0 reazioni) (0 assegnatari)Kotlin (2789 fork)batch import
Good First IssueKeep OpenTests

Metriche repository

Star
 (11.132 star)
Metriche merge PR
 (Merge medio 9g 12h) (93 PR mergiate in 30 g)

Descrizione

[!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

Guida contributor

Direzione di ricerca
Trova un metodo annotato con @NeedsTest che è in uso, scrivi un test unitario sotto /test che fallirebbe se il bug si ripresentasse e invia una PR.
Tech stack
kotlin
Dominio
mobile
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
GitKotlinUnit Testing
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80