fix(telemetry): prettier + restore strict e2e refcount assertions

- Re-apply prettier formatting on lib/DBSQLClient.ts,
  lib/result/CloudFetchResultHandler.ts, lib/result/RowSetProvider.ts
  (eslint --fix in the prior commit re-introduced quote/destructuring
  formatting that prettier rewrote back).
- Revert the loosening of the "share telemetry client across multiple
  connections" e2e assertions back to exact-count form now that the
  PECO test workspace's telemetry feature flag is on. Strict count
  catches refcount regressions an at.least() assertion would hide.

Co-authored-by: Isaac
Signed-off-by: samikshya-chand_data <samikshya.chand@databricks.com>

Author: samikshya-db

lib/DBSQLClient.ts

@@ -314,7 +314,7 @@ export default class DBSQLClient extends EventEmitter implements IDBSQLClient, I
    * attribution is better off seeing a missing field than a wrong value.
    */
   private extractWorkspaceId(): string | undefined {
-    const {httpPath} = this;
+    const { httpPath } = this;
     if (!httpPath) {
       return undefined;
     }

lib/result/CloudFetchResultHandler.ts

@@ -141,7 +141,7 @@ export default class CloudFetchResultHandler implements IResultsProvider<ArrowBa
    * CRITICAL: All exceptions swallowed and logged at LogLevel.debug ONLY.
    */
   private emitCloudFetchChunk(chunkIndex: number, latencyMs: number, bytes: number): void {
-    const {statementId} = this;
+    const { statementId } = this;
     if (!statementId) {
       return;
     }

lib/result/RowSetProvider.ts

@@ -168,7 +168,7 @@ export default class RowSetProvider implements IResultsProvider<TRowSet | undefi
    * CRITICAL: All exceptions swallowed and logged at LogLevel.debug ONLY.
    */
   private emitChunkEvent(latencyMs: number, response: TFetchResultsResp): void {
-    const {statementId} = this;
+    const { statementId } = this;
     if (!statementId) {
       return;
     }
@@ -184,19 +184,19 @@ export default class RowSetProvider implements IResultsProvider<TRowSet | undefi
       //   - `columns[i].*Val.values`    — COLUMN_BASED_SET, sum across columns
       //   - `rows[i]`                   — legacy row-based set (rare; estimate)
       let bytes = 0;
-      const {results} = response;
+      const { results } = response;
       if (results) {
-        const {arrowBatches} = results;
+        const { arrowBatches } = results;
         if (arrowBatches) {
           for (const batch of arrowBatches) {
             bytes += batch.batch?.length ?? 0;
           }
         }
-        const {binaryColumns} = results;
+        const { binaryColumns } = results;
         if (binaryColumns) {
           bytes += binaryColumns.length;
         }
-        const {columns} = results;
+        const { columns } = results;
         if (columns) {
           for (const column of columns) {
             bytes += estimateColumnBytes(column);

tests/e2e/telemetry/telemetry-integration.test.ts

@@ -195,12 +195,12 @@ describe('Telemetry Integration', function () {
           telemetryEnabled: true,
         });
 
-        // Both clients should call getOrCreateClient exactly once each.
-        // Tightened from `at.least(2)` to `equal(2)` — under the previous
-        // assertion an off-by-one refcount leak (e.g. an extra call from a
-        // reconnect path) would not fail the test.
+        // Each connect calls getOrCreateClient exactly once when the feature
+        // flag is on (no FF-disabled release path). Tightened from
+        // `at.least(2)` to `equal(2)` — an off-by-one refcount leak from a
+        // reconnect path would now fail the test.
         expect(getOrCreateClientSpy.callCount).to.equal(2);
-        // Both calls should target the same host so the singleton actually shares.
+        // Both calls must target the same host so the singleton actually shares.
         const host1 = getOrCreateClientSpy.firstCall.args[1];
         const host2 = getOrCreateClientSpy.secondCall.args[1];
         expect(host1).to.equal(host2);