fix(@schematics/angular): do not add changeDetection in default template

[cexbrayat]

PR Checklist

Please check to confirm your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Documentation content changes
• Other... Please describe:

What is the current behavior?

As the new default is Eager, the CLI was adding changeDetection: ChangeDetectionStrategy.Eager in every component

Does this PR introduce a breaking change?

• Yes
• No

Other information

[gemini-code-assist]

Code Review

This pull request fixes an issue where the default change detection strategy (Eager) was explicitly added to generated components. The component template is updated to avoid adding the changeDetection property when the strategy is 'Eager'. A new test case is included to verify this behavior. The changes appear correct and align with the goal of not specifying the default change detection strategy in the generated code.

[alan-agius4]

This is intended as ChangeDetectionStrategy.Eager needs to be explicitly set. We should not add ChangeDetectionStrategy.OnPush which is now the default instead.

Maybe we should also update the schematics default to OnPush to align with the FW default.

[cexbrayat]

@alan-agius4 Thanks for your feedback, I'll close this one and opened #32753 instead which switches the default to OnPush and don't add it to the decorator when that's the case.