Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: add integration tests #77

Open
tshakalekholoane opened this issue Aug 29, 2023 · 0 comments
Open

tests: add integration tests #77

tshakalekholoane opened this issue Aug 29, 2023 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@tshakalekholoane
Copy link
Collaborator

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

Improve coverage by adding integration tests. The cmd crate has test keys that can be used to achieve this.

Describe the solution you'd like

Perhaps using something like assert_cmd or clap's Command::debug_assert.

Describe alternatives you've considered

Writing unit tests that read a key and ensure that it is parsed correctly. This would probably be enough given the individual primitives already have tests in the relevant standards modules.

@tshakalekholoane tshakalekholoane added enhancement New feature or request good first issue Good for newcomers labels Aug 29, 2023
@tshakalekholoane tshakalekholoane moved this to 📋 Backlog in wardstone Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant