[build] add code coverage to provider_integration workflow by zimmy87 · Pull Request #352 · Azure/azihsm-sdk

zimmy87 · 2026-04-30T01:05:58Z

No description provided.

…k in provider_integration workflow

Copilot

Pull request overview

This PR updates the repo’s xtask and CI workflow so the provider_integration GitHub Actions job runs integration tests under cargo llvm-cov and emits a coverage summary via xtask precheck --coverage-report.

Changes:

Extend xtask coverage to accept nextest-like arguments (features/package/profile/filterset/exclude) and forward them to cargo llvm-cov nextest.
Update xtask precheck to pass coverage configuration (and to run multiple coverage invocations in the default “SDK” path).
Switch provider_integration workflow steps from --nextest to --coverage, and add a coverage summary step.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File	Description
`xtask/src/precheck.rs`	Routes `precheck --coverage` into parameterized `Coverage` runs (default SDK + resiliency runs, or user-provided package/features).
`xtask/src/coverage.rs`	Adds CLI flags to `Coverage` and builds argument lists for `cargo llvm-cov nextest`.
`.github/workflows/rust.yml`	Runs provider integration test packages with coverage and adds a coverage summary report step.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

…g sdk-cov.json Agent-Logs-Url: https://github.com/Azure/azihsm-sdk/sessions/36ef3ca2-4175-4870-bd8a-2cfb4384dfc3 Co-authored-by: zimmy87 <5205889+zimmy87@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

…h default precheck runs Agent-Logs-Url: https://github.com/Azure/azihsm-sdk/sessions/c036d7d2-3887-40af-8d85-48d245439a70 Co-authored-by: zimmy87 <5205889+zimmy87@users.noreply.github.com>

…flow

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

…LAGS

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

…ub.com/Azure/azihsm-sdk into user/v-davidz/provider_integration_cov

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

…factor to eliminate duplicate test parameter definitions

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

…ub.com/Azure/azihsm-sdk into user/v-davidz/provider_integration_cov

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

add parameters to coverage xtask and call it in place of nextest xtas…

ae92d55

…k in provider_integration workflow

Copilot AI review requested due to automatic review settings April 30, 2026 01:05

zimmy87 linked an issue Apr 30, 2026 that may be closed by this pull request

[task] add code coverage to provider_integration #348

Open

Copilot started reviewing on behalf of zimmy87 April 30, 2026 01:06 View session

Copilot AI reviewed Apr 30, 2026

View reviewed changes

Comment thread xtask/src/precheck.rs Outdated

Comment thread .github/workflows/rust.yml Outdated

Comment thread xtask/src/coverage.rs Outdated

Comment thread xtask/src/precheck.rs Outdated

zimmy87 added 2 commits April 30, 2026 11:35

fmt fixes

4b7a914

Merge branch 'main' into user/v-davidz/provider_integration_cov

3ddc7ca

Copilot started work on behalf of zimmy87 April 30, 2026 20:37 View session

Update xtask/src/coverage.rs

aad1538

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings April 30, 2026 20:38

Copilot started reviewing on behalf of zimmy87 April 30, 2026 20:38 View session

fix: run coverage-report after each coverage step to avoid overwritin…

798d93c

…g sdk-cov.json Agent-Logs-Url: https://github.com/Azure/azihsm-sdk/sessions/36ef3ca2-4175-4870-bd8a-2cfb4384dfc3 Co-authored-by: zimmy87 <5205889+zimmy87@users.noreply.github.com>

Copilot finished work on behalf of zimmy87 April 30, 2026 20:41

Copilot started work on behalf of zimmy87 April 30, 2026 20:41 View session

Copilot AI reviewed Apr 30, 2026

View reviewed changes

Comment thread xtask/src/precheck.rs Outdated

fix: use no_report flag to generate combined coverage report from bot…

2492ab1

…h default precheck runs Agent-Logs-Url: https://github.com/Azure/azihsm-sdk/sessions/c036d7d2-3887-40af-8d85-48d245439a70 Co-authored-by: zimmy87 <5205889+zimmy87@users.noreply.github.com>

Copilot finished work on behalf of zimmy87 April 30, 2026 20:46

remove superfluous coverage-report steps in provider_integration work…

61771e4

…flow

Copilot AI review requested due to automatic review settings May 1, 2026 18:32

Copilot started reviewing on behalf of zimmy87 May 1, 2026 18:33 View session

Copilot AI reviewed May 1, 2026

View reviewed changes

zimmy87 added 2 commits May 1, 2026 11:59

fix attempt #2 for missing coverage data

d38828b

fix attempt #3 for missing coverage data

991f069

Copilot AI review requested due to automatic review settings May 1, 2026 19:26

Copilot started reviewing on behalf of zimmy87 May 1, 2026 19:27 View session

Copilot AI reviewed May 1, 2026

View reviewed changes

Comment thread xtask/src/precheck.rs Outdated

Comment thread .github/workflows/rust.yml Outdated

Comment thread xtask/src/coverage.rs Outdated

Comment thread xtask/src/coverage_report.rs

zimmy87 added 3 commits May 5, 2026 11:40

set env vars to instrument ossl provider & add provider to LLVM_COV_F…

3c7014e

…LAGS

Merge branch 'main' into user/v-davidz/provider_integration_cov

311d585

instrument native lib in provider_integration workflow properly

5915132

Copilot AI review requested due to automatic review settings May 18, 2026 18:56

Copilot AI review requested due to automatic review settings May 20, 2026 01:40

Merge branch 'main' into user/v-davidz/provider_integration_cov

44e5f6c

Copilot started reviewing on behalf of zimmy87 May 20, 2026 01:43 View session

Copilot AI reviewed May 20, 2026

View reviewed changes

Comment thread xtask/src/coverage.rs

Comment thread xtask/src/precheck.rs Outdated

Comment thread xtask/src/coverage_report.rs

Comment thread xtask/README.md

Comment thread xtask/README.md Outdated

zimmy87 added 2 commits May 19, 2026 19:00

copilot feedback

59d395a

Merge branch 'user/v-davidz/provider_integration_cov' of https://gith…

c60cd53

…ub.com/Azure/azihsm-sdk into user/v-davidz/provider_integration_cov

zimmy87 marked this pull request as ready for review May 20, 2026 02:02

Copilot AI review requested due to automatic review settings May 20, 2026 02:02

Copilot started reviewing on behalf of zimmy87 May 20, 2026 02:04 View session

Copilot AI reviewed May 20, 2026

View reviewed changes

Comment thread xtask/src/coverage.rs

zimmy87 requested a review from rajesh-gali May 20, 2026 03:44

rajesh-gali reviewed May 20, 2026

View reviewed changes

Comment thread xtask/src/precheck.rs

rajesh-gali requested changes May 20, 2026

View reviewed changes

zimmy87 added 6 commits May 22, 2026 11:39

update --coverage stage to run mock+resiliency+integration tests & re…

cc92a5c

…factor to eliminate duplicate test parameter definitions

compile errors & fmt fixes

542e24e

clippy+fmt fixes

01b70b0

add --skip-integration flag to precheck

6d3aae3

use --skip-integration flag in workflows

e0e689a

Merge branch 'main' into user/v-davidz/provider_integration_cov

f0b368f

Copilot AI review requested due to automatic review settings May 22, 2026 20:29

Copilot started reviewing on behalf of zimmy87 May 22, 2026 20:29 View session

Copilot AI reviewed May 22, 2026

View reviewed changes

Comment thread xtask/src/precheck.rs Outdated

Comment thread xtask/src/precheck.rs

Comment thread xtask/src/coverage_report.rs Outdated

Comment thread xtask/README.md Outdated

zimmy87 and others added 3 commits May 22, 2026 14:21

fix build_windows clippy failure

0ac0754

Merge branch 'user/v-davidz/provider_integration_cov' of https://gith…

a452c5a

…ub.com/Azure/azihsm-sdk into user/v-davidz/provider_integration_cov

Apply suggestions from code review

abe7440

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings May 22, 2026 21:28

Copilot started reviewing on behalf of zimmy87 May 22, 2026 21:28 View session

Merge branch 'main' into user/v-davidz/provider_integration_cov

be9935e

Copilot AI reviewed May 22, 2026

View reviewed changes

Comment thread xtask/src/coverage_report.rs

Comment thread xtask/src/coverage_report.rs

zimmy87 requested a review from rajesh-gali May 22, 2026 22:20

Conversation

zimmy87 commented Apr 30, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants