From 506eab2547fe258f9010d9dab22c770703b6f4e1 Mon Sep 17 00:00:00 2001
From: David Buzinski <davidb@mathworks.com>
Date: Thu, 14 Dec 2023 13:03:45 -0500
Subject: [PATCH] try cacheFile to fix file location

---
 src/mpm.ts           | 5 +++--
 src/mpm.unit.test.ts | 7 ++++---
 2 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/mpm.ts b/src/mpm.ts
index b431a9f..9b38413 100644
--- a/src/mpm.ts
+++ b/src/mpm.ts
@@ -27,12 +27,13 @@ export async function setup(platform: string, architecture: string): Promise<str
     }
 
     let mpm: string = await tc.downloadTool(mpmUrl);
+    let mpmPath= await tc.cacheFile(mpm, `mpm${ext}`, "mpm", "1.0.0");
 
-    const exitCode = await exec.exec(`chmod +x ${mpm + ext}`);
+    const exitCode = await exec.exec(`chmod +x mpmPath`);
     if (exitCode !== 0) {
         return Promise.reject(Error("Unable to set up mpm."));
     }
-    return mpm
+    return mpmPath
 }
 
 export async function install(mpmPath: string, release: matlab.Release, products: string[], destination: string) {
diff --git a/src/mpm.unit.test.ts b/src/mpm.unit.test.ts
index 857c268..5e86fc3 100644
--- a/src/mpm.unit.test.ts
+++ b/src/mpm.unit.test.ts
@@ -17,6 +17,7 @@ afterEach(() => {
 
 describe("setup mpm", () => {
     let tcDownloadToolMock: jest.Mock;
+    let tcCacheFileMock: jest.Mock;
     let execMock: jest.Mock;
     let defaultInstallRootMock: jest.Mock;
     const arch = "x64";
@@ -24,15 +25,17 @@ describe("setup mpm", () => {
 
     beforeEach(() => {
         tcDownloadToolMock = tc.downloadTool as jest.Mock;
+        tcCacheFileMock = tc.cacheFile as jest.Mock;
         execMock = exec.exec as jest.Mock;
         defaultInstallRootMock = script.defaultInstallRoot as jest.Mock;
+        tcDownloadToolMock.mockResolvedValue(mpmMockPath);
+        tcCacheFileMock.mockResolvedValue(mpmMockPath);
         process.env.RUNNER_TEMP = path.join("runner", "workdir", "tmp");
     });
 
     describe("test on all supported platforms", () => {
         it(`works on linux`, async () => {
             const platform = "linux";
-            tcDownloadToolMock.mockResolvedValue(mpmMockPath);
             execMock.mockResolvedValue(0);
             await expect(mpm.setup(platform, arch)).resolves.toBe(mpmMockPath);
             expect(tcDownloadToolMock.mock.calls[0][0]).toContain("glnxa64");
@@ -40,7 +43,6 @@ describe("setup mpm", () => {
     
         it(`works on windows`, async () => {
             const platform = "win32";
-            tcDownloadToolMock.mockResolvedValue(mpmMockPath);
             execMock.mockResolvedValue(0);
             await expect(mpm.setup(platform, arch)).resolves.toBe(mpmMockPath);
             expect(tcDownloadToolMock.mock.calls[0][0]).toContain("win64");
@@ -48,7 +50,6 @@ describe("setup mpm", () => {
 
         it(`works on mac`, async () => {
             const platform = "darwin";
-            tcDownloadToolMock.mockResolvedValue(mpmMockPath);
             execMock.mockResolvedValue(0);
             await expect(mpm.setup(platform, arch)).resolves.toBe(mpmMockPath);
             expect(tcDownloadToolMock.mock.calls[0][0]).toContain("maci64");