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

Code coverage results: add the shell script #154

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

audoan99
Copy link
Contributor

@audoan99 audoan99 commented Apr 6, 2023

The shell script support generates gcov and displays code coverage results.
Steps to run in the script:

  • Run tests
  • Generate ccov
  • Show the coverage data
    E.g.
    image

Signed-off-by: Au Doan Ngoc [email protected]

@audoan99
Copy link
Contributor Author

audoan99 commented Apr 6, 2023

Tested-by: Au Doan Ngoc [email protected]

@khangtb1
Copy link
Contributor

This depends on #150

README Outdated
Syntax: <your installation path>/bin/unittest*
Example: $HOME/install/bin/unittest-ivi-controller
3. (Optional) To display code coverage results, run shell script in unittest folder
Syntax: code_coverage.sh <your build path>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This require the build path to script, it means the COV flags are default in the Cmake of tests, I shouldn't expected this. the binary when running will dump the *.gcno files in somewhere, it may get permission errors if the user don't have allow to write the files

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this suggestion is good. I will apply it to the code.

The shell script support generates gcov and displays code coverage results.

Signed-off-by: Au Doan Ngoc <[email protected]>
To generates gcov and display code coverage results, need to
follow syntax in README.

Signed-off-by: Au Doan Ngoc <[email protected]>
@audoan99 audoan99 force-pushed the devel/code_coverage_results branch from f8196e3 to 37aabe0 Compare April 25, 2023 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants