Skip to content

fix(@angular/build): suggest the istanbul provider in the unit-test V8 browser coverage error - #34484

Merged
clydin merged 1 commit into
angular:mainfrom
MeAkib:fix/unit-test-v8-coverage-browser-hint
Oct 22, 2026
Merged

clydin merged 1 commit into
angular:mainfrom
MeAkib:fix/unit-test-v8-coverage-browser-hint

Conversation

@MeAkib

@MeAkib MeAkib commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

When code coverage uses the V8 provider with a non-Chromium browser, the unit-test builder told users to disable coverage or remove the browser. The error predates Istanbul coverage support in the Vitest runner, so it never mentioned the option that keeps both. Suggest the istanbul provider first. The browser coverage e2e test now asserts the new hint.

…8 browser coverage error

When code coverage uses the V8 provider with a non-Chromium browser, the unit-test builder told
users to disable coverage or remove the browser. The error predates Istanbul coverage support in
the Vitest runner, so it never mentioned the option that keeps both. Suggest the `istanbul` provider
first. The browser coverage e2e test now asserts the new hint.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the validation error message shown when code coverage is enabled but the configured browsers do not support the V8 coverage provider, suggesting the use of the 'istanbul' coverage provider instead. It also adds a corresponding end-to-end test assertion to verify this updated message. There are no review comments to address, and the changes look correct.

@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Oct 22, 2026
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Oct 22, 2026
@clydin
clydin merged commit eb38ef3 into angular:main Oct 22, 2026
42 checks passed
@clydin

clydin commented Oct 22, 2026

Copy link
Copy Markdown
Member

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: @angular/build target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants