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

test: Increase test coverage #60

Open
kurosakishigure opened this issue Dec 9, 2024 · 1 comment
Open

test: Increase test coverage #60

kurosakishigure opened this issue Dec 9, 2024 · 1 comment
Labels
test Add or improve test(issue or PR)

Comments

@kurosakishigure
Copy link
Owner

kurosakishigure commented Dec 9, 2024

Test Description

Increase the scope of unit tests. Missing test cases can be found on Coveralls.

Rationale

Test coverage is low.

Additional Information

We can refer to the code marked as lacking tests on Coveralls and add corresponding test cases.

@kurosakishigure kurosakishigure added the enhancement New feature(issue or PR) label Dec 9, 2024
@kurosakishigure
Copy link
Owner Author

kurosakishigure commented Dec 9, 2024

The file that has the greatest impact on coverage is handle.rs.

The key lines are:

let ans: &str = Select::new("A katharsis.config.toml already exists in the current directory. Do you want to overwrite it?", options).prompt()?;

When testing, we should think about how to simulate user input🤔.

Tips: We are considering adding a -y parameter.

@kurosakishigure kurosakishigure added test Add or improve test(issue or PR) and removed enhancement New feature(issue or PR) labels Dec 18, 2024
@kurosakishigure kurosakishigure changed the title Increase test coverage test: Increase test coverage Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Add or improve test(issue or PR)
Projects
None yet
Development

No branches or pull requests

1 participant