use a dummy base url for tests

Author: Geoffrey Koros

test/common/content/BatchRequestContent.ts

@@ -10,16 +10,15 @@ import "isomorphic-fetch";
 import { assert } from "chai";
 
 import { BatchRequestContent, BatchRequestStep } from "../../../src/content/BatchRequestContent";
-import { randomString } from "../../test-helper";
-import { GRAPH_BASE_URL } from "../../../src/Constants";
+import { DUMMY_BASE_URL, randomString } from "../../test-helper";
 
 const folderName = randomString();
 const folderDetails = {
 	name: folderName,
 	folder: {},
 };
 
-const createFolderRequest = new Request(GRAPH_BASE_URL + "/me/drive/root/children", {
+const createFolderRequest = new Request(DUMMY_BASE_URL + "/me/drive/root/children", {
 	method: "POST",
 	headers: {
 		"Content-type": "application/json",

test/common/core/HTTPClient.ts

@@ -11,7 +11,7 @@ import { HTTPClient } from "../../../src/HTTPClient";
 import { Context } from "../../../src/IContext";
 import { FetchOptions } from "../../../src/IFetchOptions";
 import { DummyHTTPMessageHandler } from "../../DummyHTTPMessageHandler";
-import { GRAPH_BASE_URL } from "../../../src/Constants";
+import { DUMMY_BASE_URL } from "../../test-helper";
 
 describe("HTTPClient.ts", () => {
 	const httpMessageHandler: DummyHTTPMessageHandler = new DummyHTTPMessageHandler();
@@ -64,7 +64,7 @@ describe("HTTPClient.ts", () => {
 		});
 
 		it("Should execute for context object with Request instance", async () => {
-			const request: Request = new Request(GRAPH_BASE_URL + "dummy_url", {
+			const request: Request = new Request(DUMMY_BASE_URL + "/dummy_url", {
 				method: "GET",
 			});
 			const context: Context = {

test/common/middleware/AuthenticationHandler.ts

@@ -12,6 +12,7 @@ import { GRAPH_BASE_URL } from "../../../src/Constants";
 import { Context } from "../../../src/IContext";
 import { AuthenticationHandler } from "../../../src/middleware/AuthenticationHandler";
 import { DummyAuthenticationProvider } from "../../DummyAuthenticationProvider";
+import { DUMMY_BASE_URL } from "../../test-helper";
 
 const dummyAuthProvider = new DummyAuthenticationProvider();
 const authHandler = new AuthenticationHandler(dummyAuthProvider);
@@ -26,7 +27,7 @@ describe("AuthenticationHandler.ts", async () => {
 	});
 	describe("Auth Headers", () => {
 		it("Should delete Auth header when Request object is passed with non Graph URL", async () => {
-			const request = new Request(GRAPH_BASE_URL + "test_url");
+			const request = new Request(DUMMY_BASE_URL + "/test_url");
 			const context: Context = {
 				request,
 				options: {

test/common/middleware/MiddlewareUtil.ts

@@ -9,7 +9,7 @@ import { assert } from "chai";
 
 import { FetchOptions } from "../../../src/IFetchOptions";
 import { appendRequestHeader, generateUUID, getRequestHeader, setRequestHeader } from "../../../src/middleware/MiddlewareUtil";
-import { GRAPH_BASE_URL } from "../../../src/Constants";
+import { DUMMY_BASE_URL } from "../../test-helper";
 
 describe("MiddlewareUtil.ts", async () => {
 	function getMultipleValuesForHeader(headers: Headers, key: string): string[] {
@@ -24,7 +24,7 @@ describe("MiddlewareUtil.ts", async () => {
 	describe("getRequestHeader", () => {
 		const key = "Content-Type";
 		const value = "application/json";
-		const url = GRAPH_BASE_URL + "/dummy_url";
+		const url = DUMMY_BASE_URL + "/dummy_url";
 		it("Should get header from request object", () => {
 			const request: Request = new Request(url, {
 				method: "test",
@@ -77,7 +77,7 @@ describe("MiddlewareUtil.ts", async () => {
 	describe("setRequestHeader", () => {
 		const key = "Content-Type";
 		const value = "application/json";
-		const url = GRAPH_BASE_URL + "dummy_url";
+		const url = DUMMY_BASE_URL + "/dummy_url";
 		it("Should set header in request object", () => {
 			const request: Request = new Request(url, {
 				method: "test",
@@ -180,7 +180,7 @@ describe("MiddlewareUtil.ts", async () => {
 		const key = "Content-Type";
 		const value = "application/json";
 		const firstValue = "text/html";
-		const url = GRAPH_BASE_URL + "dummy_url";
+		const url = DUMMY_BASE_URL + "/dummy_url";
 		it("Should set header in request object if the header is not present", () => {
 			const request: Request = new Request(url, {
 				method: "test",

test/common/middleware/TelemetryHandler.ts

@@ -14,6 +14,7 @@ import { FeatureUsageFlag, TelemetryHandlerOptions } from "../../../src/middlewa
 import { TelemetryHandler } from "../../../src/middleware/TelemetryHandler";
 import { PACKAGE_VERSION } from "../../../src/Version";
 import { DummyHTTPMessageHandler } from "../../DummyHTTPMessageHandler";
+import { DUMMY_BASE_URL } from "../../test-helper";
 
 describe("TelemetryHandler.ts", () => {
 	describe("execute", function() {
@@ -121,7 +122,7 @@ describe("TelemetryHandler.ts", () => {
 		});
 
 		it("Should delete Telemetry in the header when Request object is passed with non Graph URL", async () => {
-			const request = new Request(GRAPH_BASE_URL + "test_url");
+			const request = new Request(DUMMY_BASE_URL + "/test_url");
 			const context: Context = {
 				request,
 				options: {

test/test-helper.ts

@@ -19,3 +19,5 @@ export function randomString() {
 		.toString(36)
 		.substring(7);
 }
+
+export const DUMMY_BASE_URL = "https://localhost";