fix(get-changed-files): allow space delimiter (#526)

ExpediaGroup · Jan 10, 2024 · a29a874 · a29a874
1 parent a7fd137
commit a29a874
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 9 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -47554,7 +47554,10 @@ limitations under the License.
 
 const getActionInputs = (requiredInputs = []) => {
     const yamlContents = (0,external_fs_.readFileSync)(`${__dirname}/action.yml`).toString();
-    const inputsFromFile = getInputsFromFile(yamlContents).reduce((acc, current) => (Object.assign(Object.assign({}, acc), { [current]: (0,core.getInput)(current, { required: requiredInputs.includes(current) }) })), {});
+    const inputsFromFile = getInputsFromFile(yamlContents).reduce((acc, current) => {
+        const trimWhitespaceOptions = current === 'delimiter' ? { trimWhitespace: false } : {};
+        return Object.assign(Object.assign({}, acc), { [current]: (0,core.getInput)(current, Object.assign({ required: requiredInputs.includes(current) }, trimWhitespaceOptions)) });
+    }, {});
     return (0,lodash.pickBy)(inputsFromFile);
 };
 

diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/utils/get-action-inputs.ts b/src/utils/get-action-inputs.ts
@@ -18,13 +18,13 @@ import { readFileSync } from 'fs';
 
 export const getActionInputs = (requiredInputs: string[] = []) => {
   const yamlContents = readFileSync(`${__dirname}/action.yml`).toString();
-  const inputsFromFile = getInputsFromFile(yamlContents).reduce(
-    (acc, current) => ({
+  const inputsFromFile = getInputsFromFile(yamlContents).reduce((acc, current) => {
+    const trimWhitespaceOptions = current === 'delimiter' ? { trimWhitespace: false } : {};
+    return {
       ...acc,
-      [current]: getInput(current, { required: requiredInputs.includes(current) })
-    }),
-    {}
-  );
+      [current]: getInput(current, { required: requiredInputs.includes(current), ...trimWhitespaceOptions })
+    };
+  }, {});
 
   return pickBy(inputsFromFile);
 };
diff --git a/test/utils/get-action-inputs.test.ts b/test/utils/get-action-inputs.test.ts
@@ -26,13 +26,13 @@ jest.mock('fs', () => ({
   }))
 }));
 
-(getInputsFromFile as jest.Mock).mockReturnValue(['input1', 'input2', 'input3']);
 (getInput as jest.Mock).mockImplementation(input => (input === 'input2' ? '' : input));
 
 describe('getActionInputs', () => {
   const requiredInputs = ['input1'];
 
   it('should call getInput with correct params and return expected inputs', () => {
+    (getInputsFromFile as jest.Mock).mockReturnValue(['input1', 'input2', 'input3']);
     const result = getActionInputs(requiredInputs);
 
     expect(getInput).toHaveBeenCalledWith('input1', { required: true });
@@ -43,4 +43,17 @@ describe('getActionInputs', () => {
       input3: 'input3'
     });
   });
+
+  it('should call getInput with trimWhiteSpace false for delimiter input', () => {
+    (getInputsFromFile as jest.Mock).mockReturnValue(['input1', 'input2', 'delimiter']);
+    const result = getActionInputs(requiredInputs);
+
+    expect(getInput).toHaveBeenCalledWith('input1', { required: true });
+    expect(getInput).toHaveBeenCalledWith('input2', { required: false });
+    expect(getInput).toHaveBeenCalledWith('delimiter', { required: false, trimWhitespace: false });
+    expect(result).toEqual({
+      input1: 'input1',
+      delimiter: 'delimiter'
+    });
+  });
 });