Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Mar 26, 2025

The jsdom package does not currently support execution of ESM scripts. Attempting to use the karma jsdom launcher will cause test failures with potentially unclear error messages after the tests have been built and have started to execute. The karma application builder test runner will now issue a warning describing the problem and remove the jsdom launcher from the browsers option. The warning will be shown while analyzing the options and prior to the actual start of the test process.

@clydin clydin added the target: major This PR is targeted for the next major release label Mar 26, 2025
The jsdom package does not currently support execution of ESM scripts. Attempting
to use the karma jsdom launcher will cause test failures with potentially
unclear error messages after the tests have been built and have started to execute.
The karma application builder test runner will now issue a warning
describing the problem and remove the jsdom launcher from the `browsers` option.
The warning will be shown while analyzing the options and prior to the actual
start of the test process.
@clydin clydin force-pushed the karma/jsdom-warning branch from 7dd8efb to ab96179 Compare March 26, 2025 18:05
@clydin clydin marked this pull request as ready for review March 26, 2025 18:59
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Mar 27, 2025
@alan-agius4 alan-agius4 merged commit 52fbffc into angular:main Mar 27, 2025
30 checks passed
@clydin clydin deleted the karma/jsdom-warning branch March 27, 2025 10:55
@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 Apr 27, 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: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants