add warning for deprecated clientId param

Author: shivam2680

lib/DBSQLClient.ts

@@ -160,6 +160,14 @@ export default class DBSQLClient extends EventEmitter implements IDBSQLClient, I
    * const session = client.connect({host, path, token});
    */
   public async connect(options: ConnectionOptions, authProvider?: IAuthentication): Promise<IDBSQLClient> {
+    const deprecatedClientId = (options as any).clientId;
+    if (deprecatedClientId !== undefined) {
+      this.logger.log(
+        LogLevel.warn,
+        'Warning: The "clientId" option is deprecated. Please use "userAgentEntry" instead.',
+      );
+    }
+
     this.authProvider = this.createAuthProvider(options, authProvider);
 
     this.connectionProvider = this.createConnectionProvider(options);

tests/unit/DBSQLClient.test.ts

@@ -79,6 +79,25 @@ describe('DBSQLClient.connect', () => {
 
     expect(thriftConnectionStub.on.called).to.be.true;
   });
+
+  it('should log a warning when deprecated clientId is passed', async () => {
+    const client = new DBSQLClient();
+    const logSpy = sinon.spy((client as any).logger, 'log');
+
+    const optionsWithDeprecated = {
+      ...connectOptions,
+      clientId: 'clientId',
+    };
+
+    await client.connect(optionsWithDeprecated as any);
+
+    const warningRegex = /Warning: The "clientId" option is deprecated\. Please use "userAgentEntry" instead\./;
+    const callFound = logSpy.getCalls().some((call) => warningRegex.test(call.args[1]));
+
+    expect(callFound).to.be.true;
+
+    logSpy.restore();
+  });
 });
 
 describe('DBSQLClient.openSession', () => {