Skip to content

Commit

Permalink
Update CI workflow for C++ coverage generation and testing (#571)
Browse files Browse the repository at this point in the history
* Update CI workflow for C++ coverage generation and testing

* Avoid mkdir error in CI workflow by using -p

* Update CI workflow to include coverage flags Review-1

* Added lcov to environment.yml under myst_nb

* Apply coverage flag in build and adjust C++ testing

* added installation lcov

* Update .github/workflows/ci.yml

* Apply suggestions from code review

---------

Co-authored-by: ਗਗਨਦੀਪ ਸਿੰਘ (Gagandeep Singh) <[email protected]>
  • Loading branch information
nagajaideep and czgdp1807 authored Dec 6, 2024
1 parent 4688182 commit ed88315
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 2 deletions.
17 changes: 15 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,15 +33,28 @@ jobs:
run: |
pip install -r requirements.txt
pip install -r docs/requirements.txt
- name: Install lcov
run: |
sudo apt-get update
sudo apt-get install -y lcov
- name: Build package
run: |
python scripts/build/install.py
CXXFLAGS=--coverage CFLAGS=--coverage python scripts/build/install.py
# coverage tests
- name: Run tests
run: |
python -m pytest --doctest-modules --cov=./ --cov-report=xml -s
- name: Capture Coverage Data with lcov
run: |
lcov --capture --directory . --output-file coverage.info --no-external
- name: Generate HTML Coverage Report with genhtml
run: |
genhtml coverage.info --output-directory coverage_report
- name: Upload Coverage
uses: codecov/codecov-action@v3
with:
Expand Down
1 change: 1 addition & 0 deletions environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,4 @@ dependencies:
- sphinx==5.0
- sphinx-readable-theme==1.3.0
- myst_nb==0.17.2
- lcov

0 comments on commit ed88315

Please sign in to comment.