Skip to content

Conversation

@alexlamsl
Copy link
Contributor

@alexlamsl alexlamsl commented Dec 31, 2022

  • wait for async hooks to complete before running tests
  • add support for done(err) callbacks in hooks

fixes #1688

@Jarred-Sumner
Copy link
Collaborator

In a future PR we should support the done callback too

@alexlamsl alexlamsl changed the title [jest] wait for beforeAll() to complete [WIP] [jest] wait for beforeAll() to complete Dec 31, 2022
@alexlamsl alexlamsl changed the title [WIP] [jest] wait for beforeAll() to complete [jest] wait for beforeAll() to complete Dec 31, 2022
@alexlamsl alexlamsl changed the title [jest] wait for beforeAll() to complete [jest] fix and improve hooks Dec 31, 2022
@alexlamsl
Copy link
Contributor Author

In a future PR we should support the done callback too

rolled into this PR 😉

@alexlamsl alexlamsl changed the title [jest] fix and improve hooks [WIP] [jest] fix and improve hooks Dec 31, 2022
- wait for async hooks to complete before running tests
- add support for `done(err)` callbacks in hooks

fixes oven-sh#1688
@alexlamsl alexlamsl changed the title [WIP] [jest] fix and improve hooks [jest] fix and improve hooks Dec 31, 2022
@Jarred-Sumner Jarred-Sumner merged commit 9b3db96 into oven-sh:main Dec 31, 2022
@Jarred-Sumner
Copy link
Collaborator

great

@alexlamsl alexlamsl deleted the issue-1688 branch December 31, 2022 11:32
Jarred-Sumner pushed a commit that referenced this pull request Jan 6, 2023
- wait for async hooks to complete before running tests
- add support for `done(err)` callbacks in hooks

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test suite beforeAll is not executed

2 participants