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.

Sign up for GitHub

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

Jump to bottom

Auto tests for CLI #183

Closed
wonder-sk opened this issue Aug 4, 2022 · 1 comment · Fixed by #195
Closed

Auto tests for CLI #183

wonder-sk opened this issue Aug 4, 2022 · 1 comment · Fixed by #195
Assignees
@PeterPetrik
Labels
infrastructure Related to project infrastructure (continuous integration, build system)

Comments

@wonder-sk
Copy link
Contributor

Currently CLI of geodiff is not covered by automatic testing, apart from running geodiff version command (see geodiff_cli_test in geodiff/tests/CMakeLists.txt).

It would be good to cover all the commands and check that their outputs are correct.
@wonder-sk wonder-sk added the infrastructure Related to project infrastructure (continuous integration, build system) label Aug 4, 2022
@PeterPetrik
Copy link
Contributor

not sure if cli is not out of code-coverage report, if yes, put it back

@PeterPetrik PeterPetrik self-assigned this Jan 17, 2023
@PeterPetrik PeterPetrik mentioned this issue Jan 18, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PeterPetrik PeterPetrik

Labels
infrastructure Related to project infrastructure (continuous integration, build system)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add tests for geodiff-cli MerginMaps/geodiff
2 participants
@wonder-sk @PeterPetrik