orhun/binsider

Improve the test suite

Aperta

#5 aperta il 11 set 2024

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Rust (105 fork)auto 404
effort: mediumenhancementhelp wanted

Metriche repository

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

Descrizione

Is your feature request related to a problem? Please describe.

binsider is not tested thoroughly right now. See the codecov report: https://app.codecov.io/gh/orhun/binsider

Describe the solution you'd like

Add unit and integration tests and assert the state of the TUI. See the following project's test suite for inspiration:

Describe alternatives you've considered

None.

Additional context

There are some tests in place which should be skipped due to weird behavior on some platforms. For example, app::tests reads the debug binary but in CI it fails due to "resource temporarily unavailable". See https://github.com/orhun/binsider/actions/runs/10800912100/job/29959939240

Guida contributor

Direzione di ricerca
Esamina la struttura dei test esistente, comprendi le asserzioni dello stato TUI utilizzate nei progetti di riferimento (systeroid, taskwarrior tui) e aggiungi test unitari e di integrazione per le funzionalità principali.
Tech stack
rust
Dominio
tooling
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
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80