optimize the annotation

Signed-off-by: yubonluo <[email protected]>
yubonluo · Jun 21, 2024 · c78e1af · c78e1af
1 parent 6de5625
commit c78e1af
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/server/auth/types/authentication_type.test.ts b/server/auth/types/authentication_type.test.ts
@@ -134,7 +134,7 @@ describe('test capabilities request authinfo', () => {
     asScoped: jest.fn().mockImplementation(() => {
       return {
         callAsCurrentUser: jest.fn().mockImplementation(() => {
-          return { username: 'capabolities-username' };
+          return { username: 'capabilities-username' };
         }),
       };
     }),
@@ -162,6 +162,6 @@ describe('test capabilities request authinfo', () => {
       authenticated: jest.fn((value) => value),
     };
     const result = await dummyAuthType.authHandler(request, response, toolkit);
-    expect(result.state.authInfo.username).toEqual('capabolities-username');
+    expect(result.state.authInfo.username).toEqual('capabilities-username');
   });
 });
diff --git a/server/auth/types/authentication_type.ts b/server/auth/types/authentication_type.ts
@@ -151,7 +151,7 @@ export abstract class AuthenticationType implements IAuthenticationType {
           return toolkit.notHandled();
         }
 
-        // Before users login, skip auth capabilities request.
+        // allow optional authentication
         if (this.authOptional(request)) {
           return toolkit.authenticated();
         }
@@ -244,7 +244,6 @@ export abstract class AuthenticationType implements IAuthenticationType {
     if (!pathname) {
       return false;
     }
-    // allow requests to ignored routes
     if (AuthenticationType.ROUTES_AUTH_OPTIONAL.includes(pathname!)) {
       return true;
     }