linting

Author: MarkDuckworth

dev/src/pool.ts

@@ -93,7 +93,7 @@ export class ClientPool<T extends object> {
     private readonly maxIdleClients: number,
     private readonly clientFactory: (requiresGrpc: boolean) => T,
     private readonly clientDestructor: (client: T) => Promise<void> = () =>
-      Promise.resolve()
+      Promise.resolve(),
   ) {
     this.lazyLogStringForAllClientIds = new LazyLogStringForAllClientIds({
       activeClients: this.activeClients,
@@ -141,7 +141,7 @@ export class ClientPool<T extends object> {
         requestTag,
         'Re-using existing client [%s] with %s remaining operations',
         selectedClientId,
-        this.concurrentOperationLimit - selectedClientRequestCount
+        this.concurrentOperationLimit - selectedClientRequestCount,
       );
     } else {
       const newClientId = 'cli' + generateClientId();
@@ -150,7 +150,7 @@ export class ClientPool<T extends object> {
         requestTag,
         'Creating a new client [%s] (requiresGrpc: %s)',
         newClientId,
-        requiresGrpc
+        requiresGrpc,
       );
       selectedClient = this.clientFactory(requiresGrpc);
       this.clientIdByClient.set(selectedClient, newClientId);
@@ -193,7 +193,7 @@ export class ClientPool<T extends object> {
         requestTag,
         'Garbage collecting client [%s] (%s)',
         clientId,
-        this.lazyLogStringForAllClientIds
+        this.lazyLogStringForAllClientIds,
       );
       this.activeClients.delete(client);
       this.failedClients.delete(client);
@@ -203,7 +203,7 @@ export class ClientPool<T extends object> {
         requestTag,
         'Garbage collected client [%s] (%s)',
         clientId,
-        this.lazyLogStringForAllClientIds
+        this.lazyLogStringForAllClientIds,
       );
     }
   }
@@ -337,7 +337,7 @@ export class ClientPool<T extends object> {
         /* requestTag= */ null,
         'Waiting for %s pending operations to complete before terminating (%s)',
         this.opCount,
-        this.lazyLogStringForAllClientIds
+        this.lazyLogStringForAllClientIds,
       );
       await this.terminateDeferred.promise;
     }

dev/test/pool.ts

@@ -381,23 +381,23 @@ describe('Client pool', () => {
     // Create 5 operations, which should schedule 2 operations on the first
     // client, 2 on the second and 1 on the third.
     const operationPromises = deferredPromises(7);
-    clientPool.run(REQUEST_TAG, USE_REST, client => {
+    void clientPool.run(REQUEST_TAG, USE_REST, client => {
       expect(client.count).to.be.equal(1);
       return operationPromises[0].promise;
     });
-    clientPool.run(REQUEST_TAG, USE_REST, client => {
+    void clientPool.run(REQUEST_TAG, USE_REST, client => {
       expect(client.count).to.be.equal(1);
       return operationPromises[1].promise;
     });
     const thirdOperation = clientPool.run(REQUEST_TAG, USE_REST, client => {
       expect(client.count).to.be.equal(2);
       return operationPromises[2].promise;
     });
-    clientPool.run(REQUEST_TAG, USE_REST, client => {
+    void clientPool.run(REQUEST_TAG, USE_REST, client => {
       expect(client.count).to.be.equal(2);
       return operationPromises[3].promise;
     });
-    clientPool.run(REQUEST_TAG, USE_REST, client => {
+    void clientPool.run(REQUEST_TAG, USE_REST, client => {
       expect(client.count).to.be.equal(3);
       return operationPromises[4].promise;
     });
@@ -408,7 +408,7 @@ describe('Client pool', () => {
 
     // A newly scheduled operation should use the first client that has a free
     // slot.
-    clientPool.run(REQUEST_TAG, USE_REST, async client => {
+    void clientPool.run(REQUEST_TAG, USE_REST, async client => {
       expect(client.count).to.be.equal(2);
     });
   });