Fixing #171

Author: muthurathinam-m

spec/core/GraphResponseHandler.ts

@@ -158,17 +158,5 @@ describe("GraphResponseHandler.ts", () => {
 			const responseValue = await GraphResponseHandler.getResponse(response, ResponseType.TEXT);
 			assert.isDefined(responseValue);
 		});
-
-		it("Should parse from raw response for NOT OK response", async () => {
-			try {
-				const response = new Response("NOT OK", status500);
-				const responseValue = await GraphResponseHandler.getResponse(response, ResponseType.TEXT);
-				throw new Error("Something wrong with validating OK response");
-			} catch (error) {
-				assert.isDefined(error);
-				assert.isTrue(error instanceof Response);
-				assert.equal(error.status, 500);
-			}
-		});
 	});
 });

src/GraphErrorHandler.ts

@@ -37,25 +37,6 @@ export class GraphErrorHandler {
 		return gError;
 	}
 
-	/**
-	 * @private
-	 * @static
-	 * @async
-	 * To construct error from the raw response
-	 * @param {Response} error - The error response
-	 * @param {number} statusCode - The status code of the response
-	 * @returns A promise that resolves to GraphError instance
-	 */
-	private static async constructErrorFromRawResponse(error: Response, statusCode: number): Promise<GraphError> {
-		const gError = new GraphError(statusCode);
-		try {
-			gError.body = await error.text();
-		} catch (error) {
-			// tslint:disable-line: no-empty
-		}
-		return gError;
-	}
-
 	/**
 	 * @private
 	 * @static
@@ -106,9 +87,7 @@ export class GraphErrorHandler {
 	 */
 	public static async getError(error: any = null, statusCode: number = -1, callback?: GraphRequestCallback): Promise<GraphError> {
 		let gError: GraphError;
-		if (error && (error.constructor.name === "Response" || error.constructor.name === "Body")) {
-			gError = await GraphErrorHandler.constructErrorFromRawResponse(error, statusCode);
-		} else if (error && error.error) {
+		if (error && error.error) {
 			gError = GraphErrorHandler.constructErrorFromResponse(error, statusCode);
 		} else if (error && error.constructor.name === "Error") {
 			gError = GraphErrorHandler.constructError(error, statusCode);

src/GraphResponseHandler.ts

@@ -188,7 +188,7 @@ export class GraphResponseHandler {
 				}
 			}
 		} catch (error) {
-			throw rawResponse.ok ? error : rawResponse;
+			throw error;
 		}
 	}
 }