Fix tests

Author: maschwenk

__tests__/labeler.test.ts

@@ -235,21 +235,39 @@ describe('labeler error handling', () => {
 
   it('returns true when PR author is in the list of authors', () => {
     const matchConfigWithAuthor: MatchConfig[] = [
-      {any: ['*.txt'], authors: ['monalisa', 'hubot']}
+      {
+        any: [
+          {changedFiles: [{anyGlobToAnyFile: ['*.txt']}]},
+          {authors: ['monalisa', 'hubot']}
+        ]
+      }
     ];
-    const changedFiles = ['foo.txt'];
+    const changedFiles = ['not_match.pdf'];
 
-    const result = checkGlobs(changedFiles, matchConfigWithAuthor);
+    const result = checkMatchConfigs(
+      changedFiles,
+      matchConfigWithAuthor,
+      false
+    );
     expect(result).toBeTruthy();
   });
 
   it('returns false when PR author is not in the list of authors', () => {
     const matchConfigWithAuthor: MatchConfig[] = [
-      {any: ['*.txt'], authors: ['foo', 'bar']}
+      {
+        any: [
+          {changedFiles: [{anyGlobToAnyFile: ['*.txt']}]},
+          {authors: ['foo', 'bar']}
+        ]
+      }
     ];
-    const changedFiles = ['foo.txt'];
+    const changedFiles = ['not_match.pdf'];
 
-    const result = checkGlobs(changedFiles, matchConfigWithAuthor);
+    const result = checkMatchConfigs(
+      changedFiles,
+      matchConfigWithAuthor,
+      false
+    );
     expect(result).toBeFalsy();
   });
 });

dist/index.js

@@ -277,7 +277,12 @@ const fs_1 = __importDefault(__nccwpck_require__(9896));
 const get_content_1 = __nccwpck_require__(6519);
 const changedFiles_1 = __nccwpck_require__(5145);
 const branch_1 = __nccwpck_require__(2234);
-const ALLOWED_CONFIG_KEYS = ['changed-files', 'head-branch', 'base-branch'];
+const ALLOWED_CONFIG_KEYS = [
+    'changed-files',
+    'head-branch',
+    'base-branch',
+    'authors'
+];
 const getLabelConfigs = (client, configurationPath) => Promise.resolve()
     .then(() => {
     if (!fs_1.default.existsSync(configurationPath)) {
@@ -1117,6 +1122,13 @@ function labeler() {
         }
     });
 }
+function getPrAuthor() {
+    const pullRequest = github.context.payload.pull_request;
+    if (!pullRequest) {
+        return undefined;
+    }
+    return pullRequest.user.login;
+}
 function checkMatchConfigs(changedFiles, matchConfigs, dot) {
     for (const config of matchConfigs) {
         core.debug(` checking config ${JSON.stringify(config)}`);
@@ -1170,6 +1182,12 @@ function checkAny(matchConfigs, changedFiles, dot) {
                 return true;
             }
         }
+        if (matchConfig.authors) {
+            if (checkAuthors(matchConfig.authors)) {
+                core.debug(`  "any" patterns matched`);
+                return true;
+            }
+        }
     }
     core.debug(`  "any" patterns did not match any configs`);
     return false;
@@ -1205,10 +1223,29 @@ function checkAll(matchConfigs, changedFiles, dot) {
                 return false;
             }
         }
+        if (matchConfig.authors) {
+            if (!checkAuthors(matchConfig.authors)) {
+                core.debug(`  "all" patterns did not match`);
+                return false;
+            }
+        }
     }
     core.debug(`  "all" patterns matched all configs`);
     return true;
 }
+function checkAuthors(authors) {
+    const prAuthor = getPrAuthor();
+    if (!prAuthor) {
+        core.info('Could not get pull request author from context, exiting');
+        return false;
+    }
+    if (authors.includes(prAuthor)) {
+        core.debug(`  author ${prAuthor} is on the list`);
+        return true;
+    }
+    core.debug(`  author ${prAuthor} is not on the list`);
+    return false;
+}
 
 
 /***/ }),

src/api/get-label-configs.ts

@@ -11,14 +11,25 @@ import {
 
 import {toBranchMatchConfig, BranchMatchConfig} from '../branch';
 
+export interface AuthorsMatchConfig {
+  authors?: string[];
+}
+
 export interface MatchConfig {
   all?: BaseMatchConfig[];
   any?: BaseMatchConfig[];
 }
 
-export type BaseMatchConfig = BranchMatchConfig & ChangedFilesMatchConfig;
+export type BaseMatchConfig = BranchMatchConfig &
+  ChangedFilesMatchConfig &
+  AuthorsMatchConfig;
 
-const ALLOWED_CONFIG_KEYS = ['changed-files', 'head-branch', 'base-branch'];
+const ALLOWED_CONFIG_KEYS = [
+  'changed-files',
+  'head-branch',
+  'base-branch',
+  'authors'
+];
 
 export const getLabelConfigs = (
   client: ClientType,

src/labeler.ts

@@ -191,6 +191,13 @@ export function checkAny(
         return true;
       }
     }
+
+    if (matchConfig.authors) {
+      if (checkAuthors(matchConfig.authors)) {
+        core.debug(`  "any" patterns matched`);
+        return true;
+      }
+    }
   }
 
   core.debug(`  "any" patterns did not match any configs`);
@@ -238,13 +245,19 @@ export function checkAll(
         return false;
       }
     }
+
+    if (matchConfig.authors) {
+      if (!checkAuthors(matchConfig.authors)) {
+        core.debug(`  "all" patterns did not match`);
+        return false;
+      }
+    }
   }
 
   core.debug(`  "all" patterns matched all configs`);
   return true;
 }
 
-
 function checkAuthors(authors: string[]): boolean {
   const prAuthor = getPrAuthor();
   if (!prAuthor) {