Merge pull request #227 from github/handle-github-emu-usernames

use new github emu check

__tests__/functions/admin.test.js

@@ -64,6 +64,28 @@ test('runs isAdmin checks and finds a valid admin via handle reference', async (
   )
 })
 
+test('runs isAdmin checks and finds a valid handle that is a GitHub EMU', async () => {
+  process.env.INPUT_ADMINS = 'username_company'
+  const contextNoAdmin = {
+    actor: 'username_company'
+  }
+  expect(await isAdmin(contextNoAdmin)).toStrictEqual(true)
+  expect(debugMock).toHaveBeenCalledWith(
+    'username_company is an admin via handle reference'
+  )
+})
+
+test('runs isAdmin checks and does not find a valid admin due to a bad GitHub handle', async () => {
+  process.env.INPUT_ADMINS = 'mona%lisa-'
+  const contextNoAdmin = {
+    actor: 'mona%lisa-'
+  }
+  expect(await isAdmin(contextNoAdmin)).toStrictEqual(false)
+  expect(debugMock).toHaveBeenCalledWith(
+    'mona%lisa- is not a valid GitHub username... skipping admin check'
+  )
+})
+
 test('runs isAdmin checks and does not find a valid admin', async () => {
   process.env.INPUT_ADMINS = 'monalisa'
   const contextNoAdmin = {

package.json

@@ -32,7 +32,7 @@
     "@actions/github": "^6.0.0",
     "@octokit/plugin-retry": "^6.0.1",
     "dedent-js": "^1.0.1",
-    "github-username-regex": "^1.0.0",
+    "github-username-regex-js": "^1.0.0",
     "nunjucks": "^3.2.4",
     "@octokit/rest": "^20.0.2"
   },

src/functions/admin.js

@@ -1,6 +1,6 @@
 import * as core from '@actions/core'
 import * as github from '@actions/github'
-import githubUsernameRegex from 'github-username-regex'
+import githubUsernameRegex from 'github-username-regex-js'
 import {octokitRetry} from '@octokit/plugin-retry'
 import {COLORS} from './colors'