fix: prettier formatting and add coverage/ to .prettierignore

Co-authored-by: samikshya-chand_data

Author: samikshya-db

.prettierignore

@@ -5,6 +5,7 @@ node_modules
 .nyc_output
 coverage_e2e
 coverage_unit
+coverage
 .clinic
 
 dist

lib/telemetry/TelemetryClient.ts

@@ -25,10 +25,7 @@ import { LogLevel } from '../contracts/IDBSQLLogger';
 class TelemetryClient {
   private closed: boolean = false;
 
-  constructor(
-    private context: IClientContext,
-    private host: string
-  ) {
+  constructor(private context: IClientContext, private host: string) {
     const logger = context.getLogger();
     logger.log(LogLevel.debug, `Created TelemetryClient for host: ${host}`);
   }

lib/telemetry/TelemetryClientProvider.ts

@@ -68,10 +68,7 @@ class TelemetryClientProvider {
 
     // Increment reference count
     holder.refCount += 1;
-    logger.log(
-      LogLevel.debug,
-      `TelemetryClient reference count for ${host}: ${holder.refCount}`
-    );
+    logger.log(LogLevel.debug, `TelemetryClient reference count for ${host}: ${holder.refCount}`);
 
     return holder.client;
   }
@@ -93,10 +90,7 @@ class TelemetryClientProvider {
 
     // Decrement reference count
     holder.refCount -= 1;
-    logger.log(
-      LogLevel.debug,
-      `TelemetryClient reference count for ${host}: ${holder.refCount}`
-    );
+    logger.log(LogLevel.debug, `TelemetryClient reference count for ${host}: ${holder.refCount}`);
 
     // Close and remove client when reference count reaches zero
     if (holder.refCount <= 0) {

tests/unit/.stubs/ClientContextStub.ts

@@ -48,5 +48,4 @@ export default class ClientContextStub implements IClientContext {
   public async getDriver(): Promise<IDriver> {
     return this.driver;
   }
-
 }

tests/unit/telemetry/TelemetryClient.test.ts

@@ -38,8 +38,7 @@ describe('TelemetryClient', () => {
 
       new TelemetryClient(context, HOST);
 
-      expect(logSpy.calledWith(LogLevel.debug, `Created TelemetryClient for host: ${HOST}`)).to.be
-        .true;
+      expect(logSpy.calledWith(LogLevel.debug, `Created TelemetryClient for host: ${HOST}`)).to.be.true;
     });
   });
 
@@ -87,8 +86,7 @@ describe('TelemetryClient', () => {
 
       await client.close();
 
-      expect(logSpy.calledWith(LogLevel.debug, `Closing TelemetryClient for host: ${HOST}`)).to.be
-        .true;
+      expect(logSpy.calledWith(LogLevel.debug, `Closing TelemetryClient for host: ${HOST}`)).to.be.true;
     });
 
     it('should be idempotent', async () => {

tests/unit/telemetry/TelemetryClientProvider.test.ts

@@ -91,9 +91,7 @@ describe('TelemetryClientProvider', () => {
 
       provider.getOrCreateClient(HOST1);
 
-      expect(
-        logSpy.calledWith(LogLevel.debug, `Created new TelemetryClient for host: ${HOST1}`)
-      ).to.be.true;
+      expect(logSpy.calledWith(LogLevel.debug, `Created new TelemetryClient for host: ${HOST1}`)).to.be.true;
     });
 
     it('should log reference count at debug level', () => {
@@ -103,9 +101,7 @@ describe('TelemetryClientProvider', () => {
 
       provider.getOrCreateClient(HOST1);
 
-      expect(
-        logSpy.calledWith(LogLevel.debug, `TelemetryClient reference count for ${HOST1}: 1`)
-      ).to.be.true;
+      expect(logSpy.calledWith(LogLevel.debug, `TelemetryClient reference count for ${HOST1}: 1`)).to.be.true;
     });
 
     it('should pass context to TelemetryClient', () => {
@@ -184,8 +180,7 @@ describe('TelemetryClientProvider', () => {
 
       await provider.releaseClient(HOST1);
 
-      expect(logSpy.calledWith(LogLevel.debug, `No TelemetryClient found for host: ${HOST1}`)).to
-        .be.true;
+      expect(logSpy.calledWith(LogLevel.debug, `No TelemetryClient found for host: ${HOST1}`)).to.be.true;
     });
 
     it('should log reference count decrease at debug level', async () => {
@@ -198,9 +193,7 @@ describe('TelemetryClientProvider', () => {
 
       await provider.releaseClient(HOST1);
 
-      expect(
-        logSpy.calledWith(LogLevel.debug, `TelemetryClient reference count for ${HOST1}: 1`)
-      ).to.be.true;
+      expect(logSpy.calledWith(LogLevel.debug, `TelemetryClient reference count for ${HOST1}: 1`)).to.be.true;
     });
 
     it('should log client closure at debug level', async () => {
@@ -211,9 +204,7 @@ describe('TelemetryClientProvider', () => {
       provider.getOrCreateClient(HOST1);
       await provider.releaseClient(HOST1);
 
-      expect(
-        logSpy.calledWith(LogLevel.debug, `Closed and removed TelemetryClient for host: ${HOST1}`)
-      ).to.be.true;
+      expect(logSpy.calledWith(LogLevel.debug, `Closed and removed TelemetryClient for host: ${HOST1}`)).to.be.true;
     });
 
     it('should swallow errors during client closure', async () => {
@@ -227,9 +218,7 @@ describe('TelemetryClientProvider', () => {
 
       await provider.releaseClient(HOST1);
 
-      expect(
-        logSpy.calledWith(LogLevel.debug, `Error releasing TelemetryClient: ${error.message}`)
-      ).to.be.true;
+      expect(logSpy.calledWith(LogLevel.debug, `Error releasing TelemetryClient: ${error.message}`)).to.be.true;
     });
   });
 
@@ -388,9 +377,7 @@ describe('TelemetryClientProvider', () => {
 
       await provider.releaseClient(HOST1);
 
-      const errorLogs = logSpy
-        .getCalls()
-        .filter((call) => call.args[1].includes('Error releasing'));
+      const errorLogs = logSpy.getCalls().filter((call) => call.args[1].includes('Error releasing'));
       expect(errorLogs.length).to.be.greaterThan(0);
       errorLogs.forEach((call) => {
         expect(call.args[0]).to.equal(LogLevel.debug);