diff --git a/lib/workers/pr/index.js b/lib/workers/pr/index.js index 505b292a8551c4d57e0806315f3006386ab99148..d7555b83cd5a68efed114967387c66daac9f22ea 100644 --- a/lib/workers/pr/index.js +++ b/lib/workers/pr/index.js @@ -407,7 +407,9 @@ async function checkAutoMerge(pr, config) { return false; } const res = await platform.mergePr(pr.number, branchName); - logger.info({ pr: pr.number }, 'PR automerged'); + if (res) { + logger.info({ pr: pr.number }, 'PR automerged'); + } return res; } logger.debug('No automerge'); diff --git a/test/workers/pr/index.spec.js b/test/workers/pr/index.spec.js index 4e6d831416154f7d0083e8fac2ff8f65e799ab17..d2f2518d3ba4cff1cccb7ef86e5a3bcbf2cb0710 100644 --- a/test/workers/pr/index.spec.js +++ b/test/workers/pr/index.spec.js @@ -58,6 +58,7 @@ describe('workers/pr', () => { config.automerge = true; pr.canRebase = true; platform.getBranchStatus.mockReturnValueOnce('success'); + platform.mergePr.mockReturnValueOnce(true); await prWorker.checkAutoMerge(pr, config); expect(platform.mergePr.mock.calls.length).toBe(1); });