Call pmd.bat under win32

Fixes #21

Author: adangel

dist/index.js

@@ -2,6 +2,7 @@ const core = require('@actions/core');
 const { Octokit } = require("@octokit/rest");
 const tc = require('@actions/tool-cache');
 const exec = require('@actions/exec');
+const os = require('os');
 
 const downloadPmd = async function(version) {
     try {
@@ -25,9 +26,12 @@ const downloadPmd = async function(version) {
 
 const executePmd = async function(pmdPath, sourcePath, ruleset, reportFormat, reportFile) {
     try {
-        const { exitCode, stdout, stderr } = await exec.getExecOutput(`${pmdPath}/bin/run.sh`,
+        let pmdExecutable = '/bin/run.sh pmd';
+        if (os.platform() === 'win32') {
+            pmdExecutable = '\\bin\\pmd.bat';
+        }
+        const { exitCode, stdout, stderr } = await exec.getExecOutput(`${pmdPath}${pmdExecutable}`,
             [
-                'pmd',
                 '-no-cache',
                 '-d', sourcePath,
                 '-f', reportFormat,

lib/util.js

@@ -4,6 +4,7 @@ const nock = require('nock');
 const io = require('@actions/io');
 const os = require('os');
 const fs = require('fs');
+const exec = require('@actions/exec');
 const util = require('../lib/util');
 
 const cachePath = path.join(__dirname, 'CACHE')
@@ -13,19 +14,26 @@ process.env['RUNNER_TEMP'] = tempPath
 process.env['RUNNER_TOOL_CACHE'] = cachePath
 
 describe('pmd-github-action-util', function() {
+  let platformMock;
+  let execMock;
+
   beforeAll(function() {
     setGlobal('TEST_DOWNLOAD_TOOL_RETRY_MIN_SECONDS', 0)
     setGlobal('TEST_DOWNLOAD_TOOL_RETRY_MAX_SECONDS', 0)
   })
 
   beforeEach(async function() {
+    platformMock = jest.spyOn(os, 'platform');
+    execMock = jest.spyOn(exec, 'getExecOutput');
     await io.rmRF(cachePath)
     await io.rmRF(tempPath)
     await io.mkdirP(cachePath)
     await io.mkdirP(tempPath)
   })
 
   afterEach(function() {
+    platformMock.mockRestore();
+    execMock.mockRestore();
   })
 
   afterAll(async function() {
@@ -86,6 +94,33 @@ describe('pmd-github-action-util', function() {
     expect(report.runs[0].tool.driver.version).toBe('6.40.0');
     await io.rmRF(reportFile)
   })
+
+  it('can execute PMD win32', async () => {
+    platformMock.mockReturnValueOnce('win32');
+    execMock.mockReturnValueOnce({ exitCode: 0, stdout: '', stderr: '' });
+    nock('https://api.github.com')
+      .get('/repos/pmd/pmd/releases/latest')
+        .replyWithFile(200, __dirname + '/data/releases-latest.json', {
+          'Content-Type': 'application/json',
+        })
+    nock('https://github.com')
+      .get('/pmd/pmd/releases/download/pmd_releases/6.40.0/pmd-bin-6.40.0.zip')
+      .replyWithFile(200, __dirname + '/data/pmd-bin-6.40.0.zip')
+
+    const cachedPmdPath = await util.downloadPmd('latest');
+    await util.executePmd(cachedPmdPath, '.', 'ruleset.xml', 'sarif', 'pmd-report.sarif');
+
+    expect(execMock).toBeCalledWith(`${cachedPmdPath}\\bin\\pmd.bat`, [
+          '-no-cache',
+          '-d', '.',
+          '-f', 'sarif',
+          '-R', 'ruleset.xml',
+          '-r', 'pmd-report.sarif',
+      ],
+      {
+          ignoreReturnCode: true
+      });
+  })
 });
 
 function setGlobal(key, value) {