add option to inject codeowners

Author: joelharkes

__fixtures__/core.ts

@@ -4,7 +4,11 @@ import { jest } from '@jest/globals';
 export const debug = jest.fn<typeof core.debug>();
 export const error = jest.fn<typeof core.error>();
 export const info = jest.fn<typeof core.info>();
+export const notice = jest.fn<typeof core.notice>();
 export const getInput = jest.fn<typeof core.getInput>();
 export const setOutput = jest.fn<typeof core.setOutput>();
 export const setFailed = jest.fn<typeof core.setFailed>();
 export const warning = jest.fn<typeof core.warning>();
+export const startGroup = jest.fn<typeof core.startGroup>();
+export const endGroup = jest.fn<typeof core.endGroup>();
+export const isDebug = jest.fn<typeof core.isDebug>();

__tests__/main.test.ts

@@ -5,58 +5,22 @@
  * functions and objects. For example, the core module is mocked in this test,
  * so that the actual '@actions/core' module is not imported.
  */
-import { jest } from '@jest/globals'
-import * as core from '../__fixtures__/core.js'
-import { wait } from '../__fixtures__/wait.js'
+import { jest } from '@jest/globals';
+import * as core from '../__fixtures__/core.js';
 
 // Mocks should be declared before the module being tested is imported.
-jest.unstable_mockModule('@actions/core', () => core)
-jest.unstable_mockModule('../src/wait.js', () => ({ wait }))
+jest.unstable_mockModule('@actions/core', () => core);
 
 // The module being tested should be imported dynamically. This ensures that the
 // mocks are used in place of any actual dependencies.
-const { run } = await import('../src/main.js')
-
-describe('main.ts', () => {
-  beforeEach(() => {
-    // Set the action's inputs as return values from core.getInput().
-    core.getInput.mockImplementation(() => '500')
-
-    // Mock the wait function so that it does not actually wait.
-    wait.mockImplementation(() => Promise.resolve('done!'))
-  })
-
-  afterEach(() => {
-    jest.resetAllMocks()
-  })
-
-  it('Sets the time output', async () => {
-    await run()
-
-    // Verify the time output was set.
-    expect(core.setOutput).toHaveBeenNthCalledWith(
-      1,
-      'time',
-      // Simple regex to match a time string in the format HH:MM:SS.
-      expect.stringMatching(/^\d{2}:\d{2}:\d{2}/)
-    )
-  })
-
-  it('Sets a failed status', async () => {
-    // Clear the getInput mock and return an invalid value.
-    core.getInput.mockClear().mockReturnValueOnce('this is not a number')
-
-    // Clear the wait mock and return a rejected promise.
-    wait
-      .mockClear()
-      .mockRejectedValueOnce(new Error('milliseconds is not a number'))
-
-    await run()
-
-    // Verify that the action was marked as failed.
-    expect(core.setFailed).toHaveBeenNthCalledWith(
-      1,
-      'milliseconds is not a number'
-    )
-  })
-})
+const { runAction } = await import('../src/main.js');
+
+test('run action', async () => {
+  core.isDebug.mockReturnValue(false);
+  return runAction({
+    'include-gitignore': false,
+    'ignore-default': false,
+    files: '',
+    allRulesMustHit: false,
+  });
+});

action.yml

@@ -25,6 +25,9 @@ inputs:
     description: Wether all rules must hit for the action to pass. Do not use when using files input to check only modified files.
     required: false
     default: 'false'
+  codeownersContent:
+    description: The content of the CODEOWNERS file. If not provided, the action will look for a CODEOWNERS file in the repository.
+    required: false
 
 # Define your outputs here.
 outputs:

src/main.ts

@@ -8,6 +8,7 @@ interface Input {
   'ignore-default': boolean;
   files: string;
   allRulesMustHit: boolean;
+  codeownersContent?: string;
 }
 
 function getInputs(): Input {
@@ -16,6 +17,7 @@ function getInputs(): Input {
   result['ignore-default'] = getBoolInput('ignore-default');
   result.allRulesMustHit = getBoolInput('allRulesMustHit');
   result.files = core.getInput('files');
+  result.codeownersContent = core.getInput('codeownersContent');
   return result;
 }
 
@@ -27,7 +29,7 @@ export const runAction = async (input: Input): Promise<void> => {
   let filesToCheck: string[] = [];
   core.startGroup(`Loading files to check.`);
   if (input.files) {
-    filesToCheck = input.files.split(' ');
+    filesToCheck = input.files.split(' ').map((file) => file.startsWith('/') ? file : `/${file}`);
   } else {
     filesToCheck = await (await glob.create('*')).glob();
     if (input['include-gitignore'] === true) {
@@ -56,7 +58,7 @@ export const runAction = async (input: Input): Promise<void> => {
   core.endGroup();
 
   core.startGroup('Parsing CODEOWNERS File');
-  const codeownerContent = getCodeownerContent();
+  const codeownerContent = input.codeownersContent || getCodeownerContent();
   let parsedCodeowners = parseCodeowners(codeownerContent);
   if (input['ignore-default'] === true) {
     parsedCodeowners = parsedCodeowners.filter((rule) => rule.pattern !== '*');