Change _fail to use AuthErrorCode.NETWORK_REQUEST_FAILED

Author: NhienLam

packages/auth/src/api/index.test.ts

@@ -308,26 +308,6 @@ describe('api/_performApiRequest', () => {
     });
   });
 
-  context('with non-Firebase Errors', () => {
-    afterEach(mockFetch.tearDown);
-
-    it('should handle non-FirebaseErrors', async () => {
-      mockFetch.setUpWithOverride(() => {
-        return new Promise<never>((_, reject) => reject(new Error('error')));
-      });
-      const promise = _performApiRequest<typeof request, never>(
-        auth,
-        HttpMethod.POST,
-        Endpoint.SIGN_UP,
-        request
-      );
-      await expect(promise).to.be.rejectedWith(
-        FirebaseError,
-        'auth/internal-error'
-      );
-    });
-  });
-
   context('with network issues', () => {
     afterEach(mockFetch.tearDown);
 
@@ -365,6 +345,24 @@ describe('api/_performApiRequest', () => {
       expect(clock.clearTimeout).to.have.been.called;
       clock.restore();
     });
+
+    it('should handle network failure', async () => {
+      mockFetch.setUpWithOverride(() => {
+        return new Promise<never>((_, reject) =>
+          reject(new Error('network error'))
+        );
+      });
+      const promise = _performApiRequest<typeof request, never>(
+        auth,
+        HttpMethod.POST,
+        Endpoint.SIGN_UP,
+        request
+      );
+      await expect(promise).to.be.rejectedWith(
+        FirebaseError,
+        'auth/network-request-failed'
+      );
+    });
   });
 
   context('edgcase error mapping', () => {
@@ -524,4 +522,4 @@ describe('api/_performApiRequest', () => {
         .and.not.have.property('tenantId');
     });
   });
-});
+});

packages/auth/src/api/index.ts

@@ -181,7 +181,7 @@ export async function _performFetchWithErrorHandling<V>(
     if (e instanceof FirebaseError) {
       throw e;
     }
-    _fail(auth, AuthErrorCode.INTERNAL_ERROR, { 'message': String(e) });
+    _fail(auth, AuthErrorCode.NETWORK_REQUEST_FAILED, { 'message': String(e) });
   }
 }