From 26342ba349b2f2933ef50f9ac96874be87d9a101 Mon Sep 17 00:00:00 2001 From: MaksimZhukov Date: Wed, 24 May 2023 12:21:33 +0200 Subject: [PATCH] fix: correct reading of sync-labels input --- __tests__/main.test.ts | 14 ++++++++++++-- dist/index.js | 2 +- src/labeler.ts | 2 +- 3 files changed, 14 insertions(+), 4 deletions(-) diff --git a/__tests__/main.test.ts b/__tests__/main.test.ts index d5e962387..4dfe51120 100644 --- a/__tests__/main.test.ts +++ b/__tests__/main.test.ts @@ -62,12 +62,17 @@ describe('run', () => { const mockInput = { 'repo-token': 'foo', 'configuration-path': 'bar', - 'sync-labels': true + 'sync-labels': 'true' }; jest .spyOn(core, 'getInput') .mockImplementation((name: string, ...opts) => mockInput[name]); + jest + .spyOn(core, 'getBooleanInput') + .mockImplementation( + (name: string, ...opts) => mockInput[name] === 'true' + ); usingLabelerConfigYaml('only_pdfs.yml'); mockGitHubResponseChangedFiles('foo.txt'); @@ -93,12 +98,17 @@ describe('run', () => { const mockInput = { 'repo-token': 'foo', 'configuration-path': 'bar', - 'sync-labels': false + 'sync-labels': 'false' }; jest .spyOn(core, 'getInput') .mockImplementation((name: string, ...opts) => mockInput[name]); + jest + .spyOn(core, 'getBooleanInput') + .mockImplementation( + (name: string, ...opts) => mockInput[name] === 'true' + ); usingLabelerConfigYaml('only_pdfs.yml'); mockGitHubResponseChangedFiles('foo.txt'); diff --git a/dist/index.js b/dist/index.js index 661dd761a..71af26a98 100644 --- a/dist/index.js +++ b/dist/index.js @@ -288,7 +288,7 @@ function run() { try { const token = core.getInput('repo-token'); const configPath = core.getInput('configuration-path', { required: true }); - const syncLabels = !!core.getInput('sync-labels', { required: false }); + const syncLabels = core.getBooleanInput('sync-labels'); const prNumber = getPrNumber(); if (!prNumber) { core.info('Could not get pull request number from context, exiting'); diff --git a/src/labeler.ts b/src/labeler.ts index 7ddfe6d60..ca6a113b8 100644 --- a/src/labeler.ts +++ b/src/labeler.ts @@ -31,7 +31,7 @@ export async function run() { try { const token = core.getInput('repo-token'); const configPath = core.getInput('configuration-path', {required: true}); - const syncLabels = !!core.getInput('sync-labels', {required: false}); + const syncLabels = core.getBooleanInput('sync-labels'); const prNumber = getPrNumber(); if (!prNumber) {