Skip to content

fix(core): TestBed.tick should ensure test components are synchronized#61382

Closed
atscott wants to merge 1 commit intoangular:mainfrom
atscott:testbedtick
Closed

fix(core): TestBed.tick should ensure test components are synchronized#61382
atscott wants to merge 1 commit intoangular:mainfrom
atscott:testbedtick

Conversation

@atscott
Copy link
Copy Markdown
Contributor

@atscott atscott commented May 16, 2025

This ensures that TestBed.tick updates any components created with TestBed.createComponent, regardless of whether autoDetectChanges is on.

@atscott atscott added the target: rc This PR is targeted for the next release-candidate label May 16, 2025
@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label May 16, 2025
@ngbot ngbot bot added this to the Backlog milestone May 16, 2025
@atscott atscott marked this pull request as ready for review May 19, 2025 17:37
@atscott atscott modified the milestones: Backlog, v20 candidates May 19, 2025
@pkozlowski-opensource pkozlowski-opensource added the action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews label May 21, 2025
@pkozlowski-opensource
Copy link
Copy Markdown
Member

This will need cleanup as there is a test reporting size increase. It is not clear to me if this is linked to this change or we've just accumulated changes over time :-/ (and our infra is not helping here...)

@atscott
Copy link
Copy Markdown
Contributor Author

atscott commented May 21, 2025

@pkozlowski-opensource PTAL. Size tracking was unrelated and updated in https://github.com/angular/angular/pull/61261/files

This ensures that `TestBed.tick` updates any components created with
`TestBed.createComponent`, regardless of whether autoDetectChanges is
on.
@atscott atscott added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels May 21, 2025
thePunderWoman pushed a commit that referenced this pull request May 21, 2025
#61382)

This ensures that `TestBed.tick` updates any components created with
`TestBed.createComponent`, regardless of whether autoDetectChanges is
on.

PR Close #61382
@thePunderWoman
Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit 971981e.

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

@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Jun 21, 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: core Issues related to the framework runtime target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants