fix(redux-requests-fetch): fix handling of request abortion with nati…

…ve `AbortController`

Add `isNativeAbortError` guard function & tests
Rethrow `'REQUEST_ABORTED'` when a native `AbortError` is emitted

packages/redux-requests-fetch/src/fetch-api-driver.js

@@ -1,5 +1,7 @@
 import isAbsoluteUrl from 'axios/lib/helpers/isAbsoluteURL';
 
+import { isNativeAbortError } from './helpers';
+
 const responseTypes = ['arraybuffer', 'blob', 'formData', 'json', 'text', null];
 
 const getData = (response, responseType) => {
@@ -50,6 +52,12 @@ export const createDriver = (
         throw response;
       },
     );
+  }).catch(error => {
+    if (isNativeAbortError(error)) {
+      throw 'REQUEST_ABORTED';
+    }
+
+    throw error;
   });
 
   responsePromise.cancel = () => abortSource.abort();

packages/redux-requests-fetch/src/helpers.js

@@ -0,0 +1,7 @@
+// a real browser will throw an instance of `DomException`, polyfills an instance of `AbortError`
+// https://developer.mozilla.org/en-US/docs/Web/API/AbortController#examples
+// old browsers would set the exception `code` property equal to DOMException.ABORT_ERR
+// https://developer.mozilla.org/en-US/docs/Web/API/DOMException/code
+export const isNativeAbortError = err =>
+  ['DOMException', 'AbortError'].includes(err.constructor.name) &&
+  (err.name === 'AbortError' || err.code === DOMException.ABORT_ERR);

packages/redux-requests-fetch/src/helpers.spec.js

@@ -0,0 +1,18 @@
+import { isNativeAbortError } from './helpers';
+
+describe('helpers', () => {
+  describe('isNativeAbortError', () => {
+
+    it('properly identifies a native `AbortError`', () => {
+      expect(isNativeAbortError(new DOMException('request aborted', 'AbortError'))).toBe(true);
+    });
+
+    it('does not identify another Error', () => {
+      expect(isNativeAbortError(new Error())).toBe(false);
+    });
+
+    it('does not identify a string', () => {
+      expect(isNativeAbortError('REQUEST_ABORTED')).toBe(false);
+    });
+  });
+});