fix(dep): update to be compatible with @google-cloud/common 2.1.X

package.json

@@ -30,7 +30,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "@google-cloud/common": "^2.0.0",
+    "@google-cloud/common": "^2.1.2",
     "@types/console-log-level": "^1.4.0",
     "@types/semver": "^6.0.0",
     "console-log-level": "^1.4.0",
@@ -65,6 +65,7 @@
     "nyc": "^14.1.0",
     "sinon": "^7.3.2",
     "source-map-support": "^0.5.6",
+    "teeny-request": "^5.2.0",
     "tmp": "0.1.0",
     "typescript": "~3.5.0"
   },

ts/src/profiler.ts

@@ -14,12 +14,18 @@
  * limitations under the License.
  */
 
-import {Service, ServiceConfig, ServiceObject} from '@google-cloud/common';
+import {
+  Service,
+  ServiceConfig,
+  ServiceObject,
+  ApiError,
+} from '@google-cloud/common';
 import * as http from 'http';
 import {heap as heapProfiler, SourceMapper, time as timeProfiler} from 'pprof';
 import * as msToStr from 'pretty-ms';
 import {promisify} from 'util';
 import * as zlib from 'zlib';
+import * as r from 'teeny-request';
 
 import {perftools} from '../../proto/profile';
 
@@ -78,7 +84,8 @@ export interface RequestProfile {
  * message.
  */
 function getResponseErrorMessage(
-  response: http.IncomingMessage,
+  // tslint:disable-next-line: no-any
+  response: r.Response<any>,
   err: Error | null
 ): string | undefined {
   if (err && err.message) {
@@ -244,7 +251,8 @@ export class Retryer {
 function responseToProfileOrError(
   err: Error | null,
   body?: object,
-  response?: http.IncomingMessage
+  // tslint:disable-next-line: no-any
+  response?: r.Response<any>
 ): RequestProfile {
   // response.statusCode is guaranteed to exist on client requests.
   if (response && isErrorResponseStatusCode(response.statusCode!)) {
@@ -440,20 +448,22 @@ export class Profiler extends ServiceObject {
 
     this.logger.debug(`Attempting to create profile.`);
     return new Promise<RequestProfile>((resolve, reject) => {
-      this.request(
-        options,
-        (err: Error | null, body?: object, response?: http.IncomingMessage) => {
-          try {
-            const prof = responseToProfileOrError(err, body, response);
-            this.logger.debug(
-              `Successfully created profile ${prof.profileType}.`
-            );
-            resolve(prof);
-          } catch (err) {
-            reject(err);
-          }
+      this.request(options, (
+        err: Error | ApiError | null,
+        body?: object,
+        // tslint:disable-next-line: no-any
+        response?: r.Response<any>
+      ) => {
+        try {
+          const prof = responseToProfileOrError(err, body, response);
+          this.logger.debug(
+            `Successfully created profile ${prof.profileType}.`
+          );
+          resolve(prof);
+        } catch (err) {
+          reject(err);
         }
-      );
+      });
     });
   }
 

ts/test/test-profiler.ts

@@ -297,7 +297,9 @@ describe('Profiler', () => {
       const profiler = new Profiler(testConfig);
       await profiler.profileAndUpload(requestProf);
 
-      const uploaded = requestStub.firstCall.args[0].body;
+      const uploaded = requestStub.firstCall.args[0].body as {
+        profileBytes?: string;
+      };
       const decodedBytes = Buffer.from(
         uploaded.profileBytes as string,
         'base64'
@@ -326,7 +328,9 @@ describe('Profiler', () => {
 
       const profiler = new Profiler(testConfig);
       await profiler.profileAndUpload(requestProf);
-      const uploaded = requestStub.firstCall.args[0].body;
+      const uploaded = requestStub.firstCall.args[0].body as {
+        profileBytes?: string;
+      };
       const decodedBytes = Buffer.from(
         uploaded.profileBytes as string,
         'base64'