diff --git a/packages/core/__tests__/core.test.ts b/packages/core/__tests__/core.test.ts index 2928788d7f..ffdcbf0a87 100644 --- a/packages/core/__tests__/core.test.ts +++ b/packages/core/__tests__/core.test.ts @@ -193,10 +193,12 @@ describe('@actions/core', () => { it('getInput gets non-required input', () => { expect(core.getInput('my input')).toBe('val') + expect(core.getInput('my-input')).toBe('val') }) it('getInput gets required input', () => { expect(core.getInput('my input', {required: true})).toBe('val') + expect(core.getInput('my-input', {required: true})).toBe('val') }) it('getInput throws on missing required input', () => { diff --git a/packages/core/src/core.ts b/packages/core/src/core.ts index e9091ba206..603d661cd4 100644 --- a/packages/core/src/core.ts +++ b/packages/core/src/core.ts @@ -150,7 +150,7 @@ export function addPath(inputPath: string): void { */ export function getInput(name: string, options?: InputOptions): string { const val: string = - process.env[`INPUT_${name.replace(/ /g, '_').toUpperCase()}`] || '' + process.env[`INPUT_${name.replace(/[- ]/g, '_').toUpperCase()}`] || '' if (options && options.required && !val) { throw new Error(`Input required and not supplied: ${name}`) }