fix bad INPUT_ALLOWED_CONTEXTS input

Author: GrantBirki

__tests__/functions/context-check.test.js

@@ -49,6 +49,15 @@ test('checks the event context for an issue comment and finds that it is valid -
   })
 })
 
+test('checks the event context and exits because bad input was used', async () => {
+  process.env.INPUT_ALLOWED_CONTEXTS = 'bad'
+  context.payload.issue = {}
+  expect(await contextCheck(context)).toStrictEqual({
+    valid: false,
+    context: 'issue_comment'
+  })
+})
+
 test('checks the event context for a pr comment and finds that it is valid - when only pr comments are allowed', async () => {
   process.env.INPUT_ALLOWED_CONTEXTS = 'pull_request'
   expect(await contextCheck(context)).toStrictEqual({

dist/index.js

@@ -1,6 +1,8 @@
 import * as core from '@actions/core'
 import {stringToArray} from './string-to-array'
 
+const contextDefaults = ['pull_request', 'issue']
+
 // A simple function that checks the event context to make sure it is valid
 // :param context: The GitHub Actions event context
 // :returns: Map - {valid: true/false, context: 'issue'/'pull_request'}
@@ -22,6 +24,17 @@ export async function contextCheck(context) {
     core.getInput('allowed_contexts', {required: true})
   )
 
+  // check to see if the allowedContexts variable contains at least one item from the contextDefaults array
+  // if it does not, log a warning and exit
+  if (!allowedContexts.some(r => contextDefaults.includes(r))) {
+    core.warning(
+      `the 'allowed_contexts' input must contain at least one of the following: ${contextDefaults.join(
+        ', '
+      )}`
+    )
+    return {valid: false, context: context.eventName}
+  }
+
   // check if the event is a PR
   const isPullRequest = context?.payload?.issue?.pull_request !== undefined