Fix lint and depcheck issues

southworks · Oct 20, 2023 · 455fa40 · 455fa40
1 parent 9fb5c92
commit 455fa40
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 13 deletions.
diff --git a/libraries/botbuilder/package.json b/libraries/botbuilder/package.json
@@ -28,6 +28,7 @@
   },
   "dependencies": {
     "@azure/ms-rest-js": "^2.7.0",
+    "@azure/msal-node": "^1.2.0",
     "axios": "^0.25.0",
     "botbuilder-core": "4.1.6",
     "botbuilder-stdlib": "4.1.6",

diff --git a/libraries/botframework-connector/src/auth/authenticatorResult.ts b/libraries/botframework-connector/src/auth/authenticatorResult.ts
@@ -15,4 +15,4 @@ export interface AuthenticatorResult {
      *  The date and time of expiration relative to Coordinated Universal Time (UTC).
      */
     expiresOn: Date;
-};
+}
diff --git a/libraries/botframework-connector/src/auth/certificateAppCredentials.ts b/libraries/botframework-connector/src/auth/certificateAppCredentials.ts
@@ -38,19 +38,24 @@ export class CertificateAppCredentials extends AppCredentials {
         certificatePrivateKey: string,
         channelAuthTenant?: string,
         oAuthScope?: string,
-        x5c?: string,
+        x5c?: string
     ) {
         super(appId, channelAuthTenant, oAuthScope);
         this.certificateThumbprint = certificateThumbprint;
         this.certificatePrivateKey = certificatePrivateKey;
         this.x5c = x5c;
     }
-    
+
     /**
      * @inheritdoc
-    */
-   async getToken(forceRefresh = false): Promise<string> {
-        this.credentials ??= new MsalAppCredentials(this.createClientApplication(), this.appId, this.oAuthEndpoint, this.oAuthScope);
+     */
+    async getToken(forceRefresh = false): Promise<string> {
+        this.credentials ??= new MsalAppCredentials(
+            this.createClientApplication(),
+            this.appId,
+            this.oAuthEndpoint,
+            this.oAuthScope
+        );
         return this.credentials.getToken(forceRefresh);
     }
 

diff --git a/libraries/botframework-connector/src/auth/microsoftAppCredentials.ts b/libraries/botframework-connector/src/auth/microsoftAppCredentials.ts
@@ -32,11 +32,11 @@ export class MicrosoftAppCredentials extends AppCredentials {
     constructor(appId: string, public appPassword: string, channelAuthTenant?: string, oAuthScope?: string) {
         super(appId, channelAuthTenant, oAuthScope);
     }
-    
+
     /**
      * @inheritdoc
-    */
-   async getToken(forceRefresh = false): Promise<string> {
+     */
+    async getToken(forceRefresh = false): Promise<string> {
         this.credentials ??= new MsalAppCredentials(this.appId, this.appPassword, this.oAuthEndpoint, this.oAuthScope);
         return this.credentials.getToken(forceRefresh);
     }

diff --git a/libraries/botframework-connector/tests/appCredentials.test.js b/libraries/botframework-connector/tests/appCredentials.test.js
@@ -84,10 +84,7 @@ describe('AppCredentials', function () {
 
         it('should fail to get a token with an appId and no appPassword', async function () {
             const tokenGenerator = new MicrosoftAppCredentials(APP_ID);
-            await assert.rejects(
-                tokenGenerator.getToken(true),
-                { errorCode: "invalid_client_credential" }
-            );
+            await assert.rejects(tokenGenerator.getToken(true), { errorCode: 'invalid_client_credential' });
         });
     });
 });
-Original file line number
+Diff line change
@@ Expand Up / @@ -15,4 +15,4 @@ export interface AuthenticatorResult { @@
          *  The date and time of expiration relative to Coordinated Universal Time (UTC).
          */
         expiresOn: Date;
-    };
+    }