Revert "fix(repo): stop trimming whitespace of inputs (#524)" (#525)

This reverts commit 95eec42.
ExpediaGroup · Jan 10, 2024 · a7fd137 · a7fd137
1 parent 95eec42
commit a7fd137
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -47554,7 +47554,7 @@ 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), trimWhitespace: false }) })), {});
+    const inputsFromFile = getInputsFromFile(yamlContents).reduce((acc, current) => (Object.assign(Object.assign({}, acc), { [current]: (0,core.getInput)(current, { required: requiredInputs.includes(current) }) })), {});
     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
@@ -21,7 +21,7 @@ export const getActionInputs = (requiredInputs: string[] = []) => {
   const inputsFromFile = getInputsFromFile(yamlContents).reduce(
     (acc, current) => ({
       ...acc,
-      [current]: getInput(current, { required: requiredInputs.includes(current), trimWhitespace: false })
+      [current]: getInput(current, { required: requiredInputs.includes(current) })
     }),
     {}
   );

diff --git a/test/utils/get-action-inputs.test.ts b/test/utils/get-action-inputs.test.ts
@@ -35,9 +35,9 @@ describe('getActionInputs', () => {
   it('should call getInput with correct params and return expected inputs', () => {
     const result = getActionInputs(requiredInputs);
 
-    expect(getInput).toHaveBeenCalledWith('input1', { required: true, trimWhitespace: false });
-    expect(getInput).toHaveBeenCalledWith('input2', { required: false, trimWhitespace: false });
-    expect(getInput).toHaveBeenCalledWith('input3', { required: false, trimWhitespace: false });
+    expect(getInput).toHaveBeenCalledWith('input1', { required: true });
+    expect(getInput).toHaveBeenCalledWith('input2', { required: false });
+    expect(getInput).toHaveBeenCalledWith('input3', { required: false });
     expect(result).toEqual({
       input1: 'input1',
       input3: 'input3'