Skip to content

added the tests for python and C++ and combined test report into unified test report #574

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

Closed
wants to merge 6 commits into from
Closed

added the tests for python and C++ and combined test report into unified test report #574

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4d42a41
added the tests for python and C++ and combined test report into unif…
nagajaideep Dec 21, 2024
5ea0d61
changed cobertura with codecov
nagajaideep Dec 21, 2024
6ed57eb
fixed the path in pytests
nagajaideep Dec 21, 2024
8d2c7ef
added c++ and python tests into single report in old code base
nagajaideep Dec 22, 2024
66dc46a
changed merging coverage report flow line-56
nagajaideep Dec 22, 2024
654f210
instead of manual combining using codecov for simple process
nagajaideep Dec 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 1 addition & 7 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,20 +53,14 @@ jobs:
python -m pip install gcovr
gcovr -r . --xml-pretty --output coverage-cpp.xml

- name: Merge Coverage Reports
run: |
python -m pip install coverage
coverage combine coverage-python.xml coverage-cpp.xml
coverage xml -o merged-coverage.xml

- name: Upload Coverage
uses: codecov/codecov-action@v3
with:
token: ${{ secrets.CODECOV_TOKEN }}
directory: ./coverage/reports/
env_vars: OS,PYTHON
fail_ci_if_error: false
files: ./merged-coverage.xml
files: ./coverage-python.xml,./coverage-cpp.xml
Copy link
Member

Choose a reason for hiding this comment

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

This works?

Copy link
Member

Choose a reason for hiding this comment

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

How to do the same thing locally? Can you show a screenshot?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have arrived at a preliminary solution for merging Python and C++ coverage reports using Codecov, based on their official documentation https://docs.codecov.com/docs/github-4a-merging-reports.

To further validate this solution, I plan to develop and test it locally by utilizing a Codecov API key. I will ensure comprehensive testing and provide you with an update on the results by the end of the week sir .

Copy link
Member

@czgdp1807 czgdp1807 Dec 27, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

Actually there should be a way to do it without Codecov. This is indeed tough to achieve. So if you want to take other issues while parallelly working on it, feel free to do so.

flags: unittests
name: codecov-umbrella
path_to_write_report: ./coverage/codecov_report.txt
Expand Down
Loading