Add unit tests

__tests__/config.test.ts

@@ -171,3 +171,29 @@ describe('licenses that are not valid SPDX licenses', () => {
     )
   })
 })
+
+test('it parses the comment-summary-in-pr input', async () => {
+  setInput('comment-summary-in-pr', 'true')
+  let config = await readConfig()
+  expect(config.comment_summary_in_pr).toBe('always')
+
+  clearInputs()
+  setInput('comment-summary-in-pr', 'false')
+  config = await readConfig()
+  expect(config.comment_summary_in_pr).toBe('never')
+
+  clearInputs()
+  setInput('comment-summary-in-pr', 'always')
+  config = await readConfig()
+  expect(config.comment_summary_in_pr).toBe('always')
+
+  clearInputs()
+  setInput('comment-summary-in-pr', 'never')
+  config = await readConfig()
+  expect(config.comment_summary_in_pr).toBe('never')
+
+  clearInputs()
+  setInput('comment-summary-in-pr', 'on-failure')
+  config = await readConfig()
+  expect(config.comment_summary_in_pr).toBe('on-failure')
+})

__tests__/summary.test.ts

@@ -26,7 +26,7 @@ const defaultConfig: ConfigurationOptions = {
   deny_licenses: [],
   deny_packages: [],
   deny_groups: [],
-  comment_summary_in_pr: 'never'
+  comment_summary_in_pr: true
 }
 
 const changesWithEmptyManifests: Changes = [

src/schemas.ts

@@ -2,11 +2,6 @@ import * as z from 'zod'
 
 export const SEVERITIES = ['critical', 'high', 'moderate', 'low'] as const
 export const SCOPES = ['unknown', 'runtime', 'development'] as const
-export const COMMENT_SUMMARY_OPTIONS = [
-  'always',
-  'never',
-  'on-failure'
-] as const
 
 export const SeveritySchema = z.enum(SEVERITIES).default('low')
 
@@ -55,7 +50,13 @@ export const ConfigurationOptionsSchema = z
     base_ref: z.string().optional(),
     head_ref: z.string().optional(),
     comment_summary_in_pr: z
-      .union([z.boolean(), z.enum(COMMENT_SUMMARY_OPTIONS)])
+      .union([
+        z.preprocess(
+          val => (val === 'true' ? true : val === 'false' ? false : val),
+          z.boolean()
+        ),
+        z.enum(['always', 'never', 'on-failure'])
+      ])
       .default('never')
   })
   .transform(config => {