Skip to content

fix(@schematics/angular): fix browserMode option mapping in refactor-jasmine-vitest#33314

Merged
clydin merged 1 commit into
angular:mainfrom
clydin:fix-jasmine-vitest-refactor-options
Jun 5, 2026
Merged

fix(@schematics/angular): fix browserMode option mapping in refactor-jasmine-vitest#33314
clydin merged 1 commit into
angular:mainfrom
clydin:fix-jasmine-vitest-refactor-options

Conversation

@clydin
Copy link
Copy Markdown
Member

@clydin clydin commented Jun 5, 2026

The jasmine-vitest refactor schematic previously mapped the browserMode option to options.browerMode (with a spelling typo). Because of the index signature [property: string]: any in the generated options Schema type, the compiler did not emit any error, which resulted in the option silently resolving to undefined at runtime and disabling browser-mode transformations.

…jasmine-vitest

The jasmine-vitest refactor schematic previously mapped the `browserMode` option to `options.browerMode` (with a spelling typo). Because of the index signature `[property: string]: any` in the generated options Schema type, the compiler did not emit any error, which resulted in the option silently resolving to `undefined` at runtime and disabling browser-mode transformations.
@clydin clydin added the target: patch This PR is targeted for the next patch release label Jun 5, 2026
Copy link
Copy Markdown

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

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 fixes a typo in the jasmine-vitest schematic where options.browerMode was incorrectly used instead of options.browserMode. It also adds corresponding unit tests to verify the behavior of toHaveClass transformations based on the browserMode option. There are no review comments, so I have no feedback to provide.

@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Jun 5, 2026
@clydin clydin merged commit 21cb628 into angular:main Jun 5, 2026
41 checks passed
@clydin
Copy link
Copy Markdown
Member Author

clydin commented Jun 5, 2026

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

@clydin clydin deleted the fix-jasmine-vitest-refactor-options branch June 5, 2026 14:58
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: @schematics/angular target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants