search

jestjs / jest

Delightful JavaScript Testing.
https://jestjs.io
MIT License
44.12k stars 6.44k forks source link

fix: simplify promise finally action #15199

Closed Connormiha closed 2 months ago

Connormiha commented 2 months ago

Summary

then and catch have the some code and catch throws error next. Single finally has the some behavior.

Test plan

netlify[bot] commented 2 months ago

Deploy Preview for jestjs ready!

Built without sensitive environment variables

Name Link
Latest commit 20bd4d43e91af0b5e9cc95a5fdc104a10c45ccee
Latest deploy log https://app.netlify.com/sites/jestjs/deploys/66992ba1dfb6ab00082f19e6
Deploy Preview https://deploy-preview-15199--jestjs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

github-actions[bot] commented 1 month ago

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. Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.