Enhance MATLAB installation handling to ignore 'already installed' messages and update test cases accordingly

Author: mcafaro

.github/workflows/bat.yml

@@ -92,6 +92,15 @@ jobs:
         with:
           release: ${{ matrix.release }}
           products: ${{ matrix.products }}
+      - name: Install additional products
+        uses: ./
+        with:
+          release: ${{ matrix.release }}
+          products: Image_Processing_Toolbox
+      - name: Check additional product was installed
+        uses: matlab-actions/run-command@v2
+        with:
+          command: assert(any(strcmp({ver().Name},'Image Processing Toolbox')))
       - name: Call setup MATLAB again with different release # should not error as in issue 130
         uses: ./
         with:

src/mpm.ts

@@ -78,14 +78,32 @@ export async function install(mpmPath: string, release: matlab.Release, products
     }
     mpmArguments = mpmArguments.concat("--products", ...parsedProducts);
 
-    const exitCode = await exec.exec(mpmPath, mpmArguments).catch(async e => {
-        // Fully remove failed MATLAB installation for self-hosted runners
+    let output = "";
+    const options = {
+        listeners: {
+            stdout: (data: Buffer) => {
+                const text = data.toString();
+                output += text;
+                process.stdout.write(text);
+            },
+            stderr: (data: Buffer) => {
+                const text = data.toString();
+                output += text;
+                process.stderr.write(text);
+            },
+        },
+        ignoreReturnCode: true,
+    };
+
+    const exitCode = await exec.exec(mpmPath, mpmArguments, options).catch(async e => {
         await rmRF(destination);
         throw e;
     });
-    if (exitCode !== 0) {
+
+    if (exitCode !== 0 && !output.toLowerCase().includes("already installed")) {
         await rmRF(destination);
         return Promise.reject(Error(`Script exited with non-zero code ${exitCode}`));
     }
-    return
+
+    return;
 }

src/mpm.unit.test.ts

@@ -180,4 +180,20 @@ describe("mpm install", () => {
         await expect(mpm.install(mpmPath, releaseInfo, products, destination)).rejects.toBeDefined();
         expect(rmRFMock).toHaveBeenCalledWith(destination);
     });
+
+    it("does not reject when mpm exits non-zero but reports already installed", async () => {
+        const destination = "/opt/matlab";
+        const products = ["MATLAB", "Compiler"];
+
+        // Simulate mpm writing the "already installed" message to stdout and returning non-zero
+        execMock.mockImplementation((cmd: string, args: string[], options?: exec.ExecOptions) => {
+            if (options && options.listeners && typeof options.listeners.stdout === 'function') {
+                options.listeners.stdout(Buffer.from("All specified products are already installed."));
+            }
+            return Promise.resolve(1);
+        });
+
+        await expect(mpm.install(mpmPath, releaseInfo, products, destination)).resolves.toBeUndefined();
+        expect(rmRFMock).not.toHaveBeenCalled();
+    });
 });