chore: lint fixes

Author: d-goog

Diff for: src/auth/computeclient.ts

@@ -55,8 +55,8 @@ export class Compute extends OAuth2Client {
     this.scopes = Array.isArray(options.scopes)
       ? options.scopes
       : options.scopes
-      ? [options.scopes]
-      : [];
+        ? [options.scopes]
+        : [];
   }
 
   /**

Diff for: src/transporters.ts

@@ -64,9 +64,8 @@ export class DefaultTransporter implements Transporter {
       if (!uaValue) {
         opts.headers['User-Agent'] = DefaultTransporter.USER_AGENT;
       } else if (!uaValue.includes(`${PRODUCT_NAME}/`)) {
-        opts.headers[
-          'User-Agent'
-        ] = `${uaValue} ${DefaultTransporter.USER_AGENT}`;
+        opts.headers['User-Agent'] =
+          `${uaValue} ${DefaultTransporter.USER_AGENT}`;
       }
       // track google-auth-library-nodejs version:
       if (!opts.headers['x-goog-api-client']) {

Diff for: test/test.downscopedclient.ts

@@ -54,11 +54,11 @@ class TestAuthClient extends AuthClient {
     this.credentials.expiry_date = expirationTime;
   }
 
-  async getRequestHeaders(url?: string): Promise<Headers> {
+  async getRequestHeaders(): Promise<Headers> {
     throw new Error('Not implemented.');
   }
 
-  request<T>(opts: GaxiosOptions): GaxiosPromise<T> {
+  request<T>(): GaxiosPromise<T> {
     throw new Error('Not implemented.');
   }
 }

Diff for: test/test.jwt.ts

@@ -28,7 +28,6 @@ describe('jwt', () => {
   const keypair = require('keypair');
   const PEM_PATH = './test/fixtures/private.pem';
   const PEM_CONTENTS = fs.readFileSync(PEM_PATH, 'utf8');
-  const P12_PATH = './test/fixtures/key.p12';
 
   nock.disableNetConnect();
 

Diff for: test/test.oauth2common.ts

@@ -104,9 +104,8 @@ describe('OAuthClientAuthHandler', () => {
     };
     const actualOptions = Object.assign({}, originalOptions);
     const expectedOptions = Object.assign({}, originalOptions);
-    (
-      expectedOptions.headers as Headers
-    ).Authorization = `Basic ${expectedBase64EncodedCred}`;
+    (expectedOptions.headers as Headers).Authorization =
+      `Basic ${expectedBase64EncodedCred}`;
 
     handler.testApplyClientAuthenticationOptions(actualOptions);
     assert.deepStrictEqual(expectedOptions, actualOptions);
@@ -127,9 +126,8 @@ describe('OAuthClientAuthHandler', () => {
     };
     const actualOptions = Object.assign({}, originalOptions);
     const expectedOptions = Object.assign({}, originalOptions);
-    (
-      expectedOptions.headers as Headers
-    ).Authorization = `Basic ${expectedBase64EncodedCredNoSecret}`;
+    (expectedOptions.headers as Headers).Authorization =
+      `Basic ${expectedBase64EncodedCredNoSecret}`;
 
     handler.testApplyClientAuthenticationOptions(actualOptions);
     assert.deepStrictEqual(expectedOptions, actualOptions);
@@ -146,9 +144,8 @@ describe('OAuthClientAuthHandler', () => {
     };
     const actualOptions = Object.assign({}, originalOptions);
     const expectedOptions = Object.assign({}, originalOptions);
-    (
-      expectedOptions.headers as Headers
-    ).Authorization = `Basic ${expectedBase64EncodedCred}`;
+    (expectedOptions.headers as Headers).Authorization =
+      `Basic ${expectedBase64EncodedCred}`;
 
     handler.testApplyClientAuthenticationOptions(actualOptions);
     assert.deepStrictEqual(expectedOptions, actualOptions);
@@ -349,9 +346,8 @@ describe('OAuthClientAuthHandler', () => {
     };
     const actualOptions = Object.assign({}, originalOptions);
     const expectedOptions = Object.assign({}, originalOptions);
-    (
-      expectedOptions.headers as Headers
-    ).Authorization = `Bearer ${bearerToken}`;
+    (expectedOptions.headers as Headers).Authorization =
+      `Bearer ${bearerToken}`;
 
     handler.testApplyClientAuthenticationOptions(actualOptions, bearerToken);
     assert.deepStrictEqual(expectedOptions, actualOptions);
@@ -374,9 +370,8 @@ describe('OAuthClientAuthHandler', () => {
     const actualOptions = Object.assign({}, originalOptions);
     // Expected options should have bearer token in header.
     const expectedOptions = Object.assign({}, originalOptions);
-    (
-      expectedOptions.headers as Headers
-    ).Authorization = `Bearer ${bearerToken}`;
+    (expectedOptions.headers as Headers).Authorization =
+      `Bearer ${bearerToken}`;
 
     handler.testApplyClientAuthenticationOptions(actualOptions, bearerToken);
     assert.deepStrictEqual(expectedOptions, actualOptions);
@@ -399,9 +394,8 @@ describe('OAuthClientAuthHandler', () => {
     const actualOptions = Object.assign({}, originalOptions);
     // Expected options should have bearer token in header.
     const expectedOptions = Object.assign({}, originalOptions);
-    (
-      expectedOptions.headers as Headers
-    ).Authorization = `Bearer ${bearerToken}`;
+    (expectedOptions.headers as Headers).Authorization =
+      `Bearer ${bearerToken}`;
 
     handler.testApplyClientAuthenticationOptions(actualOptions, bearerToken);
     assert.deepStrictEqual(expectedOptions, actualOptions);

Diff for: test/test.pluggableauthclient.ts

@@ -17,7 +17,7 @@ import {
   ExecutableError,
   PluggableAuthClient,
 } from '../src/auth/pluggable-auth-client';
-import {BaseExternalAccountClient, IdentityPoolClient} from '../src';
+import {BaseExternalAccountClient} from '../src';
 import {
   assertGaxiosResponsePresent,
   getAudience,

Diff for: test/test.transporters.ts

@@ -155,9 +155,9 @@ describe('transporters', () => {
         assert.strictEqual(res!.status, 200);
         done();
       },
-      _error => {
+      error => {
         scope.done();
-        done('Unexpected promise failure');
+        done(error);
       }
     );
   });