build!: update to latest version of gts/typescript (#183)

9c3ed12
commit 9c3ed12
Author: Benjamin E. Coe <bencoe@google.com>
Date:   Mon Mar 23 11:51:57 2020 -0700

    build!: update to latest version of gts/typescript (#183)

package.json

@@ -33,13 +33,14 @@
     "@types/node": "^10.5.2",
     "@types/sinon": "^7.0.0",
     "c8": "^7.0.0",
+    "chai": "^4.2.0",
     "codecov": "^3.0.4",
-    "gts": "^1.0.0",
+    "gts": "^2.0.0-alpha.5",
     "hard-rejection": "^2.1.0",
     "linkinator": "^2.0.0",
-    "mocha": "^7.0.0",
+    "mocha": "^7.1.1",
     "sinon": "^9.0.0",
-    "typescript": "3.6.4"
+    "typescript": "^3.8.3"
   },
   "engines": {
     "node": ">=10"

samples/package.json

@@ -5,6 +5,10 @@
   "engines": {
     "node": ">=8"
   },
+  "files": [
+    "*.js",
+    "!test/"
+  ],
   "repository": "googleapis/nodejs-promisify",
   "private": true,
   "scripts": {

samples/test/test.js

@@ -16,7 +16,7 @@ const {assert} = require('chai');
 const {describe, it} = require('mocha');
 const cp = require('child_process');
 
-const execSync = (cmd) => cp.execSync(cmd, {encoding: 'utf-8'});
+const execSync = cmd => cp.execSync(cmd, {encoding: 'utf-8'});
 
 describe('quickstart samples', () => {
   it('should run the quickstart', async () => {

src/index.ts

@@ -1,3 +1,5 @@
+/* eslint-disable prefer-rest-params */
+
 // Copyright 2014 Google LLC
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
@@ -66,8 +68,7 @@ export function promisify(
   const slice = Array.prototype.slice;
 
   // tslint:disable-next-line:no-any
-  const wrapper: any = function(this: WithPromise) {
-    const context = this;
+  const wrapper: any = function (this: WithPromise) {
     let last;
 
     for (last = arguments.length - 1; last >= 0; last--) {
@@ -81,7 +82,7 @@ export function promisify(
         break; // non-callback last argument found.
       }
 
-      return originalMethod.apply(context, arguments);
+      return originalMethod.apply(this, arguments);
     }
 
     // peel trailing undefined.
@@ -93,8 +94,8 @@ export function promisify(
     // Because dedupe will likely create a single install of
     // @google-cloud/common to be shared amongst all modules, we need to
     // localize it at the Service level.
-    if (context && context.Promise) {
-      PromiseCtor = context.Promise;
+    if (this && this.Promise) {
+      PromiseCtor = this.Promise;
     }
 
     return new PromiseCtor((resolve, reject) => {
@@ -114,7 +115,7 @@ export function promisify(
         }
       });
 
-      originalMethod.apply(context, args);
+      originalMethod.apply(this, args);
     });
   };
 
@@ -165,16 +166,14 @@ export function callbackify(originalMethod: CallbackMethod) {
   }
 
   // tslint:disable-next-line:no-any
-  const wrapper = function(this: any) {
-    const context = this;
-
+  const wrapper = function (this: any) {
     if (typeof arguments[arguments.length - 1] !== 'function') {
-      return originalMethod.apply(context, arguments);
+      return originalMethod.apply(this, arguments);
     }
 
     const cb = Array.prototype.pop.call(arguments);
 
-    originalMethod.apply(context, arguments).then(
+    originalMethod.apply(this, arguments).then(
       // tslint:disable-next-line:no-any
       (res: any) => {
         res = Array.isArray(res) ? res : [res];

synth.metadata

@@ -1,5 +1,5 @@
 {
-  "updateTime": "2020-04-01T12:20:55.968161Z",
+  "updateTime": "2020-04-01T12:20:55.097782Z",
   "sources": [
     {
       "git": {

test/index.ts

@@ -12,8 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+/* eslint-disable @typescript-eslint/no-empty-function,prefer-rest-params */
+
 import * as assert from 'assert';
-import {describe, it, afterEach} from 'mocha';
+import {describe, it, afterEach, beforeEach} from 'mocha';
 import * as sinon from 'sinon';
 import * as util from '../src';
 
@@ -24,13 +26,12 @@ describe('promisifyAll', () => {
   const fakeArgs = [null, 1, 2, 3];
   const fakeError = new Error('err.');
 
-  // tslint:disable-next-line
   let FakeClass: any;
 
   beforeEach(() => {
     FakeClass = class {
       methodName(callback: Function) {
-        callback.apply(null, fakeArgs);
+        callback(...fakeArgs);
       }
       methodSingle(callback: Function) {
         callback(null, fakeArgs[1]);
@@ -83,7 +84,6 @@ describe('promisifyAll', () => {
       });
     `);
   } catch (error) {
-    // tslint:disable-next-line ban
     it.skip('should work on ES classes');
   }
 
@@ -131,8 +131,7 @@ describe('promisify', () => {
 
   beforeEach(() => {
     fakeArgs = [null, 1, 2, 3];
-    func = util.promisify(function(this: {}, callback: () => void) {
-      // tslint:disable-next-line no-any
+    func = util.promisify(function (this: {}, callback: () => void) {
       (callback as any).apply(this, fakeArgs);
     });
   });
@@ -144,12 +143,13 @@ describe('promisify', () => {
   });
 
   it('should not return a promise in callback mode', done => {
-    let returnVal: {};
-    returnVal = func.call(fakeContext, function(this: {}) {
-      const args = [].slice.call(arguments);
+    let returnVal: any;
+    returnVal = func.call(fakeContext, function (this: {}) {
+      const args = [...arguments];
       assert.deepStrictEqual(args, fakeArgs);
       assert.strictEqual(this, fakeContext);
       assert(!returnVal);
+      returnVal = null; // this is to suppress prefer-const.
       done();
     });
   });
@@ -174,7 +174,6 @@ describe('promisify', () => {
   });
 
   it('should allow the Promise object to be overridden', () => {
-    // tslint:disable-next-line:variable-name
     const FakePromise = class {};
     const promise = func.call({Promise: FakePromise});
     assert(promise instanceof FakePromise);
@@ -185,7 +184,6 @@ describe('promisify', () => {
 
     func = util.promisify(
       (callback: () => void) => {
-        // tslint:disable-next-line no-any
         (callback as any).apply(func, [null, fakeArg]);
       },
       {
@@ -199,7 +197,6 @@ describe('promisify', () => {
   });
 
   it('should ignore singular when multiple args are present', () => {
-    // tslint:disable-next-line:no-any
     const fakeArgs: any[] = ['a', 'b'];
 
     func = util.promisify(
@@ -259,7 +256,6 @@ describe('callbackifyAll', () => {
   const fakeArgs = [1, 2, 3];
   const fakeError = new Error('err.');
 
-  // tslint:disable-next-line
   let FakeClass: any;
 
   beforeEach(() => {
@@ -313,7 +309,7 @@ describe('callbackify', () => {
   beforeEach(() => {
     fakeArgs = [1, 2, 3];
 
-    func = util.callbackify(async function(this: {}) {
+    func = util.callbackify(async (_this: {}) => {
       return fakeArgs;
     });
   });
@@ -331,15 +327,15 @@ describe('callbackify', () => {
   });
 
   it('should call the callback if it is provided', done => {
-    func(function(this: {}) {
+    func(function (this: {}) {
       const args = [].slice.call(arguments);
       assert.deepStrictEqual(args, [null, ...fakeArgs]);
       done();
     });
   });
 
   it('should call the provided callback with undefined', done => {
-    func = util.callbackify(async function(this: {}) {});
+    func = util.callbackify(async (_this: {}) => {});
     func((err: Error, resp: {}) => {
       assert.strictEqual(err, null);
       assert.strictEqual(resp, undefined);
@@ -348,10 +344,10 @@ describe('callbackify', () => {
   });
 
   it('should call the provided callback with null', done => {
-    func = util.callbackify(async function(this: {}) {
+    func = util.callbackify(async (_this: {}) => {
       return null;
     });
-    func(function(this: {}) {
+    func(function (this: {}) {
       const args = [].slice.call(arguments);
       assert.deepStrictEqual(args, [null, null]);
       done();

tsconfig.json

@@ -1,6 +1,7 @@
 {
   "extends": "./node_modules/gts/tsconfig-google.json",
   "compilerOptions": {
+    "lib": ["es2018", "dom"],
     "rootDir": ".",
     "outDir": "build",
   },