Skip to content

Conversation

@JiaLiPassion
Copy link
Contributor

Related to #41434

Fix several flaky cases.

  1. should restore window.onerror in test cases.
  2. expect().toThrow() should pass a function.

@JiaLiPassion JiaLiPassion requested a review from alan-agius4 April 8, 2021 23:08
@google-cla google-cla bot added the cla: yes label Apr 8, 2021
@zarend zarend added the area: zones Issues related to zone.js label Apr 8, 2021
@ngbot ngbot bot added this to the Backlog milestone Apr 8, 2021
@JiaLiPassion JiaLiPassion marked this pull request as draft April 9, 2021 02:57
Related to angular#41434

Fix several flaky cases.

1. should restore `window.onerror` in test cases.
2. expect().toThrow() should pass a function.
@JiaLiPassion JiaLiPassion marked this pull request as ready for review December 18, 2021 00:10
@JiaLiPassion JiaLiPassion added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Jan 21, 2022
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit 25a83eb.

@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 24, 2022
josmar-crwdstffng pushed a commit to josmar-crwdstffng/angular that referenced this pull request Apr 8, 2022
Related to angular#41434

Fix several flaky cases.

1. should restore `window.onerror` in test cases.
2. expect().toThrow() should pass a function.

PR Close angular#41526
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: zones Issues related to zone.js cla: yes target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants