orhun/binsider

Improve the test suite

Offen

#5 geöffnet am 11.09.2024

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Rust (105 Forks)auto 404
effort: mediumenhancementhelp wanted

Repository-Metriken

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

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die vorhandene Teststruktur, verstehe die TUI Zustandsassertionen, die in den Referenzprojekten (systeroid, taskwarrior tui) verwendet werden, und füge Unit und Integrationstests für die Kernfunktionalität hinzu.
Tech Stack
rust
Domain
tooling
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
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80