From 7d2887e3b9aa851c245ddd068158e3e898ac22bd Mon Sep 17 00:00:00 2001
From: Will Taylor <willgeorgetaylor@gmail.com>
Date: Fri, 29 Dec 2023 20:46:52 +0000
Subject: [PATCH] fix: surface arguments correctly

---
 badges/coverage.svg | 2 +-
 dist/index.js       | 7 ++++---
 src/main.ts         | 7 ++++---
 3 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/badges/coverage.svg b/badges/coverage.svg
index cb1ae74..bf0e188 100644
--- a/badges/coverage.svg
+++ b/badges/coverage.svg
@@ -1 +1 @@
-<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="116" height="20" role="img" aria-label="Coverage: 15.21%"><title>Coverage: 15.21%</title><linearGradient id="s" x2="0" y2="100%"><stop offset="0" stop-color="#bbb" stop-opacity=".1"/><stop offset="1" stop-opacity=".1"/></linearGradient><clipPath id="r"><rect width="116" height="20" rx="3" fill="#fff"/></clipPath><g clip-path="url(#r)"><rect width="63" height="20" fill="#555"/><rect x="63" width="53" height="20" fill="#e05d44"/><rect width="116" height="20" fill="url(#s)"/></g><g fill="#fff" text-anchor="middle" font-family="Verdana,Geneva,DejaVu Sans,sans-serif" text-rendering="geometricPrecision" font-size="110"><text aria-hidden="true" x="325" y="150" fill="#010101" fill-opacity=".3" transform="scale(.1)" textLength="530">Coverage</text><text x="325" y="140" transform="scale(.1)" fill="#fff" textLength="530">Coverage</text><text aria-hidden="true" x="885" y="150" fill="#010101" fill-opacity=".3" transform="scale(.1)" textLength="430">15.21%</text><text x="885" y="140" transform="scale(.1)" fill="#fff" textLength="430">15.21%</text></g></svg>
\ No newline at end of file
+<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" width="116" height="20" role="img" aria-label="Coverage: 14.89%"><title>Coverage: 14.89%</title><linearGradient id="s" x2="0" y2="100%"><stop offset="0" stop-color="#bbb" stop-opacity=".1"/><stop offset="1" stop-opacity=".1"/></linearGradient><clipPath id="r"><rect width="116" height="20" rx="3" fill="#fff"/></clipPath><g clip-path="url(#r)"><rect width="63" height="20" fill="#555"/><rect x="63" width="53" height="20" fill="#e05d44"/><rect width="116" height="20" fill="url(#s)"/></g><g fill="#fff" text-anchor="middle" font-family="Verdana,Geneva,DejaVu Sans,sans-serif" text-rendering="geometricPrecision" font-size="110"><text aria-hidden="true" x="325" y="150" fill="#010101" fill-opacity=".3" transform="scale(.1)" textLength="530">Coverage</text><text x="325" y="140" transform="scale(.1)" fill="#fff" textLength="530">Coverage</text><text aria-hidden="true" x="885" y="150" fill="#010101" fill-opacity=".3" transform="scale(.1)" textLength="430">14.89%</text><text x="885" y="140" transform="scale(.1)" fill="#fff" textLength="430">14.89%</text></g></svg>
\ No newline at end of file
diff --git a/dist/index.js b/dist/index.js
index f791e6e..4b24f25 100644
--- a/dist/index.js
+++ b/dist/index.js
@@ -6626,7 +6626,7 @@ function enumerateInputs() {
         if (key.startsWith('INPUT_')) {
             const inputName = key.slice('INPUT_'.length).toLowerCase();
             const inputValue = process.env[key];
-            if (inputName !== 'version') {
+            if (inputValue && inputName !== 'version') {
                 inputs[inputName] = inputValue || '';
             }
         }
@@ -6673,11 +6673,12 @@ async function run() {
             core.addPath(pathToCLI);
         }
         const inputs = enumerateInputs();
-        const args = Object.entries(inputs).map(([key, value]) => `${key}=${value}`);
-        core.info(`Running junit-reducer with arguments: `);
+        const args = Object.entries(inputs).map(([key, value]) => `--${key}='${value}'`);
+        core.startGroup(`Running junit-reducer with arguments: `);
         core.info(Object.entries(inputs)
             .map(([key, value]) => `${key}: ${value}`)
             .join('\n'));
+        core.endGroup();
         await exec.exec('junit-reducer', args);
     }
     catch (error) {
diff --git a/src/main.ts b/src/main.ts
index 69cb967..4d6dfb8 100644
--- a/src/main.ts
+++ b/src/main.ts
@@ -13,7 +13,7 @@ function enumerateInputs(): Record<string, string> {
     if (key.startsWith('INPUT_')) {
       const inputName = key.slice('INPUT_'.length).toLowerCase()
       const inputValue = process.env[key]
-      if (inputName !== 'version') {
+      if (inputValue && inputName !== 'version') {
         inputs[inputName] = inputValue || ''
       }
     }
@@ -78,15 +78,16 @@ export async function run(): Promise<void> {
 
     const inputs = enumerateInputs()
     const args: string[] = Object.entries(inputs).map(
-      ([key, value]) => `${key}=${value}`
+      ([key, value]) => `--${key}='${value}'`
     )
 
-    core.info(`Running junit-reducer with arguments: `)
+    core.startGroup(`Running junit-reducer with arguments: `)
     core.info(
       Object.entries(inputs)
         .map(([key, value]) => `${key}: ${value}`)
         .join('\n')
     )
+    core.endGroup()
 
     await exec.exec('junit-reducer', args)
   } catch (error) {