-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat(jest-circus, jest-jasmine2): Add support for async function in setupFilesAfterEnv #14749
Conversation
✅ Deploy Preview for jestjs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
@vkml can you sign the CLA? 🙂 |
@SimenB done |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Adds the corresponding support for CJS async function export of
setupFiles
tosetupFilesAfterEnv
(for supported test runners). #10962.Summary
Previously, support for CJS async functions was implemented solely for
setupFiles
. This PR aims to extend the same functionality tosetupFilesAfterEnv
.ESM natively supports top-level-await, making the addition of this feature to ESM redunant. However, CJS is still widely used, so I felt the need to expand the feature support to
setupFilesAfterEnv
.Test plan