fix(UserSession): throw ArcGISAuthError instead of Error when unable …

…to refresh a token AFFECTS PACKAGES: @esri/arcgis-rest-auth ISSUES CLOSED: #56
Esri · Feb 1, 2018 · 8854765 · 8854765
1 parent f12822c
commit 8854765
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/packages/arcgis-rest-auth/src/UserSession.ts b/packages/arcgis-rest-auth/src/UserSession.ts
@@ -522,7 +522,7 @@ export class UserSession implements IAuthenticationManager {
       return this.refreshWithRefreshToken();
     }
 
-    return Promise.reject(new Error("Unable to refresh token."));
+    return Promise.reject(new ArcGISAuthError("Unable to refresh token."));
   }
 
   /**

diff --git a/packages/arcgis-rest-auth/test/UserSession.test.ts b/packages/arcgis-rest-auth/test/UserSession.test.ts
@@ -1,5 +1,9 @@
 import { UserSession, IFetchTokenResponse } from "../src/index";
-import { ArcGISRequestError, ErrorTypes } from "@esri/arcgis-rest-request";
+import {
+  ArcGISRequestError,
+  ArcGISAuthError,
+  ErrorTypes
+} from "@esri/arcgis-rest-request";
 import * as fetchMock from "fetch-mock";
 import { YESTERDAY, TOMORROW } from "./utils";
 
@@ -350,6 +354,8 @@ describe("UserSession", () => {
       });
 
       session.refreshSession().catch(e => {
+        expect(e instanceof ArcGISAuthError).toBeTruthy();
+        expect(e.name).toBe("ArcGISAuthError");
         expect(e.message).toBe("Unable to refresh token.");
         done();
       });