remove raise on empty params to fix the integration test

github · Sep 28, 2021 · a74f168 · a74f168
1 parent 6e57db3
commit a74f168
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 7 deletions.
diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts
@@ -68,7 +68,7 @@ describe('run', () => {
     test('it fails the workflow', async () => {
       await run(context)
 
-      expect(core.setFailed).toHaveBeenCalled()
+      expect(core.setFailed).not.toHaveBeenCalled()
       expect(core.info).toHaveBeenCalledWith(
         "Dependabot Updater Action does not support 'issue_created' events."
       )

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/inputs.ts b/src/inputs.ts
@@ -20,8 +20,6 @@ export function getJobParameters(ctx: Context): JobParameters | null {
 function fromWorkflowInputs(ctx: Context): JobParameters {
   const evt = ctx.payload as WorkflowDispatchEvent
 
-  core.debug(JSON.stringify(evt.inputs))
-
   const dependabotApiDockerUrl =
     evt.inputs.dependabotApiDockerUrl || evt.inputs.dependabotApiUrl
 

diff --git a/src/main.ts b/src/main.ts
@@ -24,7 +24,6 @@ export async function run(context: Context): Promise<void> {
     // Decode JobParameters:
     const params = getJobParameters(context)
     if (params === null) {
-      core.setFailed('no parameters!')
       return // No parameters, nothing to do
     }