orhun/binsider

Improve the test suite

Open

#5 opened on Sep 11, 2024

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (105 forks)auto 404
effort: mediumenhancementhelp wanted

Repository metrics

Stars
 (4,333 stars)
PR merge metrics
 (PR metrics pending)

Description

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 direction
Examine the existing test structure, understand the TUI state assertions used in reference projects (systeroid, taskwarrior tui), and add unit/integration tests for core functionality.
Tech stack
rust
Domain
tooling
Issue type
Test
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Rust
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80