Analyze all files if modified files cannot be determined

Author: adangel

dist/index.js

@@ -21,7 +21,7 @@ async function main() {
     if (core.getInput('analyzeModifiedFilesOnly', { required: true }) === 'true') {
       core.info(`Determining modified files in ${sourcePath}...`);
       modifiedFiles = await util.determineModifiedFiles(token, sourcePath);
-      if (modifiedFiles.length === 0) {
+      if (modifiedFiles !== undefined && modifiedFiles.length === 0) {
         core.info(`No modified files have been found in ${sourcePath} - exiting`);
         core.setOutput('violations', 0);
         return;

lib/index.js

@@ -146,7 +146,8 @@ const determineModifiedFiles = async function(token, sourcePath) {
 
         return modifiedFilenames;
     } else {
-        throw new Error(`Unsupported github action event '${context.eventName}'`);
+        core.warning(`Unsupported github action event '${context.eventName}' - cannot determine modified files. All files will be analyzed.`);
+        return undefined;
     }
 }
 

lib/util.js

@@ -218,10 +218,11 @@ describe('pmd-github-action-util', function () {
       .map(f => path.normalize(f)));
   })
 
-  test('throws for unsupported event', async () => {
+  test('return undefined for unsupported event', async () => {
     process.env['GITHUB_REPOSITORY'] = 'pmd/pmd-github-action-tests'
     process.env['GITHUB_EVENT_NAME'] = 'workflow_dispatch';
-    await expect(util.determineModifiedFiles('my_test_token', 'src/main/java')).rejects.toThrow();
+    let result = await util.determineModifiedFiles('my_test_token', path.normalize('src/main/java'));
+    expect(result).toBe(undefined);
   })
 
   it('can execute PMD with list of files', async () => {