Merge pull request #115 from jmeas/AbortError

Add AbortError name to faux-abort errors

Author: jamesplease

src/fetch.js

@@ -107,9 +107,11 @@ export class Fetch extends React.Component {
   // configured, then we need to "cancel" the previous one.
   cancelExistingRequest = reason => {
     if (this.state.fetching && !this.hasHandledNetworkResponse) {
+      const abortError = new Error(reason);
+      abortError.name = 'AbortError';
       this.onResponseReceived({
         ...this.responseReceivedInfo,
-        error: new Error(reason),
+        error: abortError,
         hittingNetwork: true
       });
     }

test/index.test.js

@@ -982,7 +982,7 @@ describe('request cancellation', () => {
   });
 
   test('it should cancel when a double request is initiated via `doFetch`', () => {
-    expect.assertions(2);
+    expect.assertions(3);
     jest.useFakeTimers();
     const afterFetchMock = jest.fn();
 
@@ -1006,6 +1006,10 @@ describe('request cancellation', () => {
       'error.message',
       'New fetch initiated'
     );
+    expect(afterFetchMock.mock.calls[0][0]).toHaveProperty(
+      'error.name',
+      'AbortError'
+    );
   });
 
   test('it should cancel when a double request is initiated via prop changes', () => {
@@ -1026,6 +1030,10 @@ describe('request cancellation', () => {
       'error.message',
       'New fetch initiated'
     );
+    expect(afterFetchMock.mock.calls[0][0]).toHaveProperty(
+      'error.name',
+      'AbortError'
+    );
   });
 
   test('it should cancel when the component unmounts', () => {
@@ -1042,6 +1050,10 @@ describe('request cancellation', () => {
       'error.message',
       'Component unmounted'
     );
+    expect(afterFetchMock.mock.calls[0][0]).toHaveProperty(
+      'error.name',
+      'AbortError'
+    );
     expect(afterFetchMock).toBeCalledWith(
       expect.objectContaining({
         url: '/test/hangs',