| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
There was a problem hiding this comment.
Neat 👍
Sorry, something went wrong.
|
🎉 This PR is included in version 3.2.8 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Purpose
By relying upon a 100ms delay before aborting, it is possible for the request to succeed before being canceled.
Since these 2 tests merely ensures that cancellation via AbortController works, adding a delay isn't needed regardless.
Changes
Removes delay between request and canceling.
Additional information
Undici runs node-fetch's test suite (more or less) and this test is flaky (https://github.com/nodejs/undici/runs/5153562802?check_suite_focus=true for example).
Example code with the issue fixed: