Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Nov 13, 2025

This commit removes the explicit isolate option from the Vitest configuration within the Angular CLI builder.

Previously, the isolate option was explicitly set to false for DOM emulation and browser mode. However, due to observed non-deterministic failures and the need for consistent isolation behavior across all environments, it's more robust to allow Vitest to manage test isolation by its own default mechanisms. This change ensures that Vitest's internal defaults for isolation are respected, which are generally designed for optimal reliability.

This commit removes the explicit `isolate` option from the Vitest configuration within the Angular CLI builder.

Previously, the `isolate` option was explicitly set to `false` for DOM emulation and browser mode. However, due to observed non-deterministic failures and the need for consistent isolation behavior across all environments, it's more robust to allow Vitest to manage test isolation by its own default mechanisms. This change ensures that Vitest's internal defaults for isolation are respected, which are generally designed for optimal reliability.
@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Nov 13, 2025
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Nov 13, 2025
@clydin clydin requested a review from alan-agius4 November 13, 2025 19:17
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Nov 13, 2025
@clydin clydin merged commit 138649e into angular:main Nov 13, 2025
35 checks passed
@clydin
Copy link
Member Author

clydin commented Nov 13, 2025

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

@clydin clydin deleted the unit-test/remove-explicit-isolate branch November 13, 2025 19:27
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: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants