Skip to content

fix(@angular/build): ensure Vitest setup files are executed in order#31789

Merged
clydin merged 1 commit intoangular:mainfrom
clydin:unit-test/setupfile-order
Nov 13, 2025
Merged

fix(@angular/build): ensure Vitest setup files are executed in order#31789
clydin merged 1 commit intoangular:mainfrom
clydin:unit-test/setupfile-order

Conversation

@clydin
Copy link
Member

@clydin clydin commented Nov 13, 2025

This commit configures the Vitest runner to execute setup files in the order they are defined. This is crucial for ensuring that polyfills and the Angular TestBed are initialized in the correct sequence before any tests are run, preventing potential issues with the testing environment.

@clydin clydin added the target: rc This PR is targeted for the next release-candidate label Nov 13, 2025
@clydin clydin force-pushed the unit-test/setupfile-order branch 6 times, most recently from 848db0f to 10b8268 Compare November 13, 2025 12:55
@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 13:08
@alan-agius4 alan-agius4 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
This commit configures the Vitest runner to execute setup files in the order they are defined. This is crucial for ensuring that polyfills and the Angular TestBed are initialized in the correct sequence before any tests are run, preventing potential issues with the testing environment.
@clydin clydin force-pushed the unit-test/setupfile-order branch from 10b8268 to 664fc03 Compare November 13, 2025 13:36
@clydin clydin merged commit 1ef24a7 into angular:main Nov 13, 2025
31 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/setupfile-order branch November 13, 2025 13:51
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

rc.2 Cannot configure the test module when the test module has already been instantiated

2 participants