fix(send-stale-inital-pr-reminder): don't try to comment if we alread…

…y commented
greenkeeperio · Jul 4, 2017 · 5e9a10a · 5e9a10a
1 parent 8df97e0
commit 5e9a10a
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 0 deletions.
diff --git a/jobs/send-stale-initial-pr-reminder.js b/jobs/send-stale-initial-pr-reminder.js
@@ -16,6 +16,7 @@ module.exports = async function (
   const ghqueue = githubQueue(installationId)
 
   if (repository.enabled) return
+  if (repository.staleInitialPRReminder) return
 
   const [owner, repo] = repository.fullName.split('/')
 

diff --git a/test/jobs/send-stale-initial-pr-reminder.js b/test/jobs/send-stale-initial-pr-reminder.js
@@ -97,6 +97,33 @@ test('send-stale-initial-pr-reminder', async t => {
     await waitFor(50)
   })
 
+  t.test('does nothing if the repo has already received the reminder', async t => {
+    t.plan(1)
+
+    await upsert(repositories, '42', {staleInitialPRReminder: true})
+
+    githubNock
+      .get('/repos/finnp/test/issues/1234')
+      .reply(200, () => {
+        t.fail('Should not query issue status')
+        return {}
+      })
+      .post('/repos/finnp/test/issues/1234/comments')
+      .reply(201, () => {
+        t.fail('Should not post comment')
+        return {}
+      })
+
+    const newJob = await worker({
+      prNumber: 1234,
+      repositoryId: 42,
+      accountId: 123
+    })
+
+    t.notOk(newJob, 'no new job')
+    await waitFor(50)
+  })
+
   t.test('does nothing if the issue was closed in the meanwhile', async t => {
     t.plan(1)