Skip to content

[ci] add ut coverage check #150

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
zjswhhh wants to merge 2 commits into from
Closed

[ci] add ut coverage check #150

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6885e59
coverage
zjswhhh Mar 17, 2026
427179b
azp
zjswhhh Mar 17, 2026
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
7 changes: 6 additions & 1 deletion Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ test-release:
clean-release:
cargo clean --release

ci-all: | ci-format ci-build ci-doc ci-lint ci-test
ci-all: | ci-format ci-build ci-doc ci-lint ci-test ci-coverage

ci-format:
cargo fmt --check --all
Expand All @@ -80,3 +80,8 @@ ci-test:
cargo test --workspace --all-features
cargo clean
cargo test --release --workspace --all-features

ci-coverage:
rustup component add llvm-tools-preview
cargo install cargo-llvm-cov --locked || true
cargo llvm-cov --workspace --all-features --fail-under-lines 80
4 changes: 4 additions & 0 deletions azure-pipelines.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,6 +109,10 @@ stages:
make ci-test
displayName: 'Test'

- script: |
make ci-coverage
displayName: 'Coverage (fail under 80% lines)'

- publish: $(System.DefaultWorkingDirectory)/
artifact: sonic-dash-ha
displayName: 'Archive release binaries'
Loading