fix: correctly populate inputs

Author: radiantspace

dist/index.js

@@ -3528,8 +3528,17 @@ Object.defineProperty(exports, "__esModule", { value: true });
 const core = __importStar(__webpack_require__(470));
 const github = __importStar(__webpack_require__(469));
 const create_1 = __webpack_require__(646);
+// nullify getInput empty results
+// to allow coalescence ?? operator
+function getInput(name, options) {
+    const result = core.getInput(name, options);
+    if (result === '') {
+        return null;
+    }
+    return result;
+}
 function run() {
-    var _a, _b, _c, _d, _e;
+    var _a, _b, _c, _d, _e, _f, _g;
     return __awaiter(this, void 0, void 0, function* () {
         let token;
         let type;
@@ -3545,22 +3554,22 @@ function run() {
         console.log(`ref: ${ref}`);
         try {
             console.log('Inputs..');
-            token = core.getInput('token', { required: true });
-            type = core.getInput('type', { required: true });
+            token = (_a = getInput('token', { required: true })) !== null && _a !== void 0 ? _a : '';
+            type = getInput('type', { required: true });
             console.log(`type: ${type}`);
-            logsUrl = (_a = core.getInput('logs')) !== null && _a !== void 0 ? _a : '';
+            logsUrl = (_b = getInput('logs')) !== null && _b !== void 0 ? _b : '';
             console.log(`logs: ${logsUrl}`);
-            description = (_b = core.getInput('description')) !== null && _b !== void 0 ? _b : `deployed by ${actor}`;
+            description = (_c = getInput('description')) !== null && _c !== void 0 ? _c : `deployed by ${actor}`;
             console.log(`description: ${description}`);
-            initialStatus = ((_c = core.getInput('initial_status')) !== null && _c !== void 0 ? _c : 'in_progress');
+            initialStatus = ((_d = getInput('initial_status')) !== null && _d !== void 0 ? _d : 'in_progress');
             console.log(`initialStatus: ${initialStatus}`);
             // default to branch name w/o `deploy-` prefix
-            environment = (_d = core.getInput('environment')) !== null && _d !== void 0 ? _d : (ref).replace(/^refs\/heads/, '').replace(/^deploy-/, '');
+            environment = (_e = getInput('environment')) !== null && _e !== void 0 ? _e : ref.replace('refs/heads/', '').replace(/^deploy-/, '');
             console.log(`environment: ${environment}`);
-            environmentUrl = (_e = core.getInput('environment_url')) !== null && _e !== void 0 ? _e : '';
+            environmentUrl = (_f = getInput('environment_url')) !== null && _f !== void 0 ? _f : '';
             console.log(`environmentUrl: ${environmentUrl}`);
             const shouldRequireDeploymentId = type === 'status' || type === 'delete';
-            deploymentId = core.getInput('deployment_id', { required: shouldRequireDeploymentId });
+            deploymentId = (_g = getInput('deployment_id', { required: shouldRequireDeploymentId })) !== null && _g !== void 0 ? _g : '0';
         }
         catch (error) {
             core.error(error);

src/main.ts

@@ -5,6 +5,16 @@ import { DeploymentStatus } from './deployment-status'
 
 type ActionType = 'create' | 'delete' | 'delete-all' | 'status'
 
+// nullify getInput empty results
+// to allow coalescence ?? operator
+function getInput (name: string, options?: core.InputOptions): string | null {
+  const result = core.getInput(name, options)
+  if (result === '') {
+    return null
+  }
+  return result
+}
+
 async function run (): Promise<void> {
   let token: string
   let type: ActionType
@@ -23,29 +33,29 @@ async function run (): Promise<void> {
 
   try {
     console.log('Inputs..')
-    token = core.getInput('token', { required: true })
+    token = getInput('token', { required: true }) ?? ''
 
-    type = core.getInput('type', { required: true }) as ActionType
+    type = getInput('type', { required: true }) as ActionType
     console.log(`type: ${type}`)
 
-    logsUrl = core.getInput('logs') ?? ''
+    logsUrl = getInput('logs') ?? ''
     console.log(`logs: ${logsUrl}`)
 
-    description = core.getInput('description') ?? `deployed by ${actor}`
+    description = getInput('description') ?? `deployed by ${actor}`
     console.log(`description: ${description}`)
 
-    initialStatus = (core.getInput('initial_status') ?? 'in_progress') as DeploymentStatus
+    initialStatus = (getInput('initial_status') ?? 'in_progress') as DeploymentStatus
     console.log(`initialStatus: ${initialStatus}`)
 
     // default to branch name w/o `deploy-` prefix
-    environment = core.getInput('environment') ?? (ref).replace(/^refs\/heads/, '').replace(/^deploy-/, '')
+    environment = getInput('environment') ?? ref.replace('refs/heads/', '').replace(/^deploy-/, '')
     console.log(`environment: ${environment}`)
 
-    environmentUrl = core.getInput('environment_url') ?? ''
+    environmentUrl = getInput('environment_url') ?? ''
     console.log(`environmentUrl: ${environmentUrl}`)
 
     const shouldRequireDeploymentId = type === 'status' || type === 'delete'
-    deploymentId = core.getInput('deployment_id', { required: shouldRequireDeploymentId })
+    deploymentId = getInput('deployment_id', { required: shouldRequireDeploymentId }) ?? '0'
   } catch (error) {
     core.error(error)
     core.setFailed(`Wrong parameters given: ${JSON.stringify(error, null, 2)}`)

tests/main.test.ts

@@ -6,6 +6,8 @@ test('test create', () => {
   process.env['INPUT_TOKEN'] = 'fake-token'
   process.env['INPUT_TYPE'] = 'create'
   process.env['GITHUB_REPOSITORY'] = 'owner/repo'
+  process.env['GITHUB_REF'] = 'refs/heads/master'
+  process.env['GITHUB_ACTOR'] = 'fake-actor'
   const ip = path.join(__dirname, '..', 'lib', 'main.js')
   const options: cp.ExecSyncOptions = {
     env: process.env