Skip to content

test(@angular/build): add spec for the application clearScreen option - #34498

Closed
MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:test/application-clear-screen
Closed

MeAkib wants to merge 1 commit into
angular:mainfrom
MeAkib:test/application-clear-screen

Conversation

@MeAkib

@MeAkib MeAkib commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

The clearScreen option had no test. Run the application builder in watch mode and check that console.clear is called before a rebuild only when the option is enabled.

The `clearScreen` option had no test. Run the application builder in watch mode and check that
`console.clear` is called before a rebuild only when the option is enabled.

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

Copy link
Copy Markdown

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 introduces a new test suite for the clearScreen option in the application builder. The tests verify that the console screen is cleared appropriately during a rebuild in watch mode based on the configuration. There are no review comments, and I have no feedback to provide.

@alan-agius4

alan-agius4 commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

Thanks for the contribution! Closing this PR as we prefer to avoid adding full watch-mode builder integration tests (and spying on global console.clear) for a simple conditional check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants