Repository navigation
Collect managed coverage across CI test workers - #2147
Draft
Tyrie Vella (tyrielv) wants to merge 5 commits into
Draft
Tyrie Vella (tyrielv) wants to merge 5 commits into
Tyrie Vella (tyrielv) wants to merge 5 commits into
Conversation
Build an isolated instrumented installer for x64 shard 0 and collect coverage before Windows services start. Keep ordinary installers and test selections unchanged for the other workers. Assisted-by: GPT-6.1 Sol Signed-off-by: Tyrie Vella <tyrielv@gmail.com>
Tyrie Vella (tyrielv)
marked this pull request as ready for review
November 4, 2026 15:26
Contributor
Author
|
Do not complete - I marked this ready for review so we could verify #2153 is a problem. |
Johannes Schindelin (dscho)
approved these changes
Nov 4, 2026
Tyrie Vella (tyrielv)
marked this pull request as draft
November 4, 2026 15:36
Grant LocalSystem access through the Visual Studio collector. Verify service startup coverage before the selected functional shard runs. Assisted-by: GPT-6.1 Sol Signed-off-by: Tyrie Vella <tyrielv@gmail.com>
Instrument non-release payloads in the existing builds. Collect unit, functional, and upgrade evidence without a second test suite, and preserve source identity and separate unit-copy attribution. Assisted-by: GPT-6.1 Sol Signed-off-by: Tyrie Vella <tyrielv@gmail.com>
Tyrie Vella (tyrielv)
changed the title
Strip argument quotes before the batch script compares or assigns configuration and architecture. Preserve the existing defaults and test command. Assisted-by: GPT-6.1 Sol Signed-off-by: Tyrie Vella <tyrielv@gmail.com>
Resolve local relative paths without weakening source-path checks. Accept empty optional scenario fields and valid NUnit skipped roots. Compare PR coverage against the tested merge base. Assisted-by: GPT-6.1 Sol Signed-off-by: Tyrie Vella <tyrielv@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem and Context
CI does not publish full managed-source coverage for the deployed NativeAOT product.
Unit coverage can measure linked source inside a test assembly instead of the product executable.
The functional suite already uses twelve workers per architecture.
A separate coverage suite duplicates that work.
This change collects coverage during the existing test runs.
Changes
Coverage Scope
The reports measure managed source in the existing CI test suite.
Native C++ remains unmeasured.
The changed-line report identifies unsupported native files and unmeasured C# lines.
This change adds no coverage threshold.
The release pipeline does not enable instrumentation.
Workers that consume instrumented installers start a collector before the product runs.