Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(@angular/build): also disable outputMode in vitest unit-tests #30414

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 29, 2025

Conversation

clydin
Copy link
Member

@clydin clydin commented May 29, 2025

Application defined entry points (browser/server) are disabled when using the experimental unit-test builder. This allows the unit test files themselves to become the entry points for testing purposes. Previously, the outputMode option was not also disabled. This lead to errors if it was present in the build target configuration used for unit-test due to the outputMode option requiring the server option.

Closes #30410

Application defined entry points (`browser`/`server`) are disabled when
using the experimental unit-test builder. This allows the unit test files
themselves to become the entry points for testing purposes. Previously,
the `outputMode` option was not also disabled. This lead to errors if it
was present in the build target configuration used for `unit-test` due to
the `outputMode` option requiring the `server` option.
@clydin clydin added the target: patch This PR is targeted for the next patch release label May 29, 2025
@clydin clydin requested a review from alan-agius4 May 29, 2025 13:28
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label May 29, 2025
@clydin clydin removed the request for review from alan-agius4 May 29, 2025 14:18
@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 May 29, 2025
@jkrems jkrems merged commit 020527e into angular:main May 29, 2025
34 checks passed
@jkrems
Copy link
Contributor

jkrems commented May 29, 2025

The changes were merged into the following branches: main, 20.0.x

@clydin clydin deleted the unit-test/disable-output-mode branch May 29, 2025 16:06
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.

unit-test builder doesn't work with server
2 participants