Skip to content

refactor(animations): add unit test for leaving animated child#44489

Closed
dario-piotrowicz wants to merge 1 commit intoangular:masterfrom
dario-piotrowicz:no-consume-animateChild-unit-test
Closed

refactor(animations): add unit test for leaving animated child#44489
dario-piotrowicz wants to merge 1 commit intoangular:masterfrom
dario-piotrowicz:no-consume-animateChild-unit-test

Conversation

@dario-piotrowicz
Copy link
Contributor

@dario-piotrowicz dario-piotrowicz commented Dec 15, 2021

add a new unit test to make sure that leaving elements queried via
a parent queried via animateChild are correctly removed

note: this tests the fix introduced in PR #44357

PR Checklist

Please check if your PR fulfills the following requirements:

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
  • angular.io application / infrastructure changes
  • Other... Please describe:

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@pullapprove pullapprove bot requested a review from jessicajaniuk December 15, 2021 13:47
add a new unit test to make sure that leaving elements queried via
a parent queried via animateChild are correctly removed

note: this tests the fix introduced in PR angular#44357
@dario-piotrowicz dario-piotrowicz force-pushed the no-consume-animateChild-unit-test branch from 42b1896 to a438bc3 Compare December 15, 2021 13:54
@ngbot ngbot bot added this to the Backlog milestone Jan 4, 2022
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🍪

Thanks, Dario!

reviewed-for: fw-animations, fw-core

@pullapprove pullapprove bot requested a review from dylhunn January 4, 2022 21:46
@jessicajaniuk jessicajaniuk removed the request for review from dylhunn January 4, 2022 21:47
@jessicajaniuk jessicajaniuk added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Jan 4, 2022
@dylhunn
Copy link
Contributor

dylhunn commented Jan 4, 2022

This PR was merged into the repository by commit f2b858d.

@dylhunn dylhunn closed this in f2b858d Jan 4, 2022
dylhunn pushed a commit that referenced this pull request Jan 4, 2022
add a new unit test to make sure that leaving elements queried via
a parent queried via animateChild are correctly removed

note: this tests the fix introduced in PR #44357

PR Close #44489
@angular-automatic-lock-bot
Copy link

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 Feb 4, 2022
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: animations target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants