jefflau / jest-fetch-mock

Jest mock for fetch
MIT License
886 stars 117 forks source link

#66: Only polyfill promise when it is not present #67

Closed doug-wade closed 6 years ago

doug-wade commented 6 years ago

Addresses #66. I've verified that this resolves the regression, but I have not verified that Promise.finally works as intended. It is implemented in terms of Promise primitives, though, so it should work the same on the native Promise as on the polyfilled Promise.

jefflau commented 6 years ago

@allenfang i merged this as a quick fix to your promise.finally polyfill PR. Do you know if this will all work as in intended following these changes?