Update package for build verification

microsoft · Apr 20, 2022 · 61defb5 · 61defb5
1 parent c6c5ce4
commit 61defb5
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/packages/abstractions/src/requestInformation.ts b/packages/abstractions/src/requestInformation.ts
@@ -70,14 +70,14 @@ export class RequestInformation {
     return this._requestOptions;
   }
   /** Adds the headers for the request. */
-  public addHeaders(source: Record<string, string>) {
+  public addRequestHeaders(source: Record<string, string> | undefined) {
     if (!source) return;
     for (const key in source) {
       this.headers[key] = source[key];
     }
   }
   /** Adds the request options for the request. */
-  public addRequestOptions(...options: RequestOption[]) {
+  public addRequestOptions(options: RequestOption[] | undefined) {
     if (!options || options.length === 0) return;
     options.forEach((option) => {
       this._requestOptions[option.getKey()] = option;
@@ -137,7 +137,10 @@ export class RequestInformation {
    * Sets the query string parameters from a raw object.
    * @param parameters the parameters.
    */
-  public setQueryStringParametersFromRawObject = (q: object): void => {
+  public setQueryStringParametersFromRawObject = (
+    q: object | undefined
+  ): void => {
+    if (!q) return;
     Object.entries(q).forEach(([k, v]) => {
       let key = k;
       if ((q as any).getQueryParameter) {

diff --git a/packages/abstractions/test/common/requestInformation.ts b/packages/abstractions/test/common/requestInformation.ts
@@ -59,7 +59,7 @@ describe("RequestInformation", () => {
     requestInformation.pathParameters["baseurl"] = baseUrl;
     requestInformation.urlTemplate = "http://localhost/me{?%24select}";
     const headers: Record<string, string> = { ConsistencyLevel: "eventual" };
-    requestInformation.addHeaders(headers);
+    requestInformation.addRequestHeaders(headers);
     assert.isNotEmpty(requestInformation.headers);
     assert.equal("eventual", requestInformation.headers["ConsistencyLevel"]);
   });