-
Notifications
You must be signed in to change notification settings - Fork 1
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
As an application developer I want to use CLI tests to test the interface of my program such that incompatible options or invalid inputs can be tested automatically. #3
Comments
What is the design of the test class? Which functionality is expected? |
Task 1 (Change cmake to fetch and build gtest within configuration step using FETCH_CONTENT) is independent and not related to the User Story above. I created a separate issue for this: #47 |
The Definition of Done here makes no sense (API doc for CLI test?, we do not have coverage test yet, unit tests and microbenchmarks are independent). I will ignore many of them. |
That won't always be the case. But you did it right to cross through what is not needed. It is merely a reminder on commonly agreed goals for a story. In the future we might want to differentiate between items with a DoD and without. But first let's gather some intel to make an informed decision. |
This is finished since https://github.com/joergi-w/app-template/pull/5 and https://github.com/joergi-w/app-template/pull/6 are merged. |
Acceptance criteria
make cli_test
in the build directory of the application and it builds the testsTasks
Change cmake to fetch and build gtest within configuration step usingFETCH_CONTENTtesting::test
and adds support for configuring the application CLI parameter and calling the application.Definition of Done
Unit tests passTest coverage = 100%Microbenchmarks added and/or affected microbenchmarks < 5% performance dropAPI documentation addedThe text was updated successfully, but these errors were encountered: