Skip to content

Integration coverage xpc helpers#1551

Open
noah-thor wants to merge 1 commit into
apple:mainfrom
noah-thor:integration-coverage-xpc-helpers
Open

Integration coverage xpc helpers#1551
noah-thor wants to merge 1 commit into
apple:mainfrom
noah-thor:integration-coverage-xpc-helpers

Conversation

@noah-thor

Copy link
Copy Markdown
Contributor

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update

Motivation and Context

This fixes the LLVM coverage data not properly being emitted for XPC
services. It requires piping the LLVM_PROFILE_FILE environment
variable through to all the services and plugins. The variable itself
also required the "%c" formatter to ensure that it continuously emits
coverage data, otherwise when XPC services are killed via "bootout" they
do not emit coverage.

Testing

  • Tested locally
  • Added/updated tests
  • Added/updated docs

@github-actions

github-actions Bot commented May 13, 2026

Copy link
Copy Markdown

Code Coverage

Tier Line Coverage
Unit 22.69%
Integration 68.59%
Combined 76.94%

@noah-thor noah-thor marked this pull request as ready for review May 18, 2026 17:25
@noah-thor noah-thor force-pushed the integration-coverage-xpc-helpers branch 2 times, most recently from 997dc38 to b4a4631 Compare June 3, 2026 17:38
@noah-thor noah-thor marked this pull request as draft June 3, 2026 17:40
This fixes the LLVM coverage data not properly being emitted for XPC
services. It requires piping the `LLVM_PROFILE_FILE` environment
variable through to all the services and plugins. The variable itself
also required the "%c" formatter to ensure that it continuously emits
coverage data, otherwise when XPC services are killed via "bootout" they
do not emit coverage.
@noah-thor noah-thor force-pushed the integration-coverage-xpc-helpers branch from b4a4631 to e6f832d Compare June 3, 2026 17:42
@noah-thor noah-thor marked this pull request as ready for review June 3, 2026 18:35
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