fix(reset): ignore 422 errors as that means the branch is already del…

…eted anyway
greenkeeperio · Aug 1, 2017 · 96c0366 · 96c0366
1 parent 02b661f
commit 96c0366
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/jobs/reset.js b/jobs/reset.js
@@ -52,7 +52,9 @@ module.exports = async function ({ repositoryFullName }) {
         ref: `heads/${branch.head}`
       }))
     } catch (e) {
-      if (e.code === 422 && e.message === 'Reference does not exist') {
+      // branch was deleted already and since we wanted to delete it anyway, we're cool
+      // with this error
+      if (e.code === 422) {
         continue
       }
       if (branch.head === 'greenkeeper/initial' || branch.head === 'greenkeeper-initial') {

diff --git a/test/jobs/reset.js b/test/jobs/reset.js
@@ -194,7 +194,7 @@ test('reset repo', async t => {
       .delete('/repos/finnp/abc/git/refs/heads/greenkeeper-standard-10.0.0')
       .reply(200)
       .delete('/repos/finnp/abc/git/refs/heads/greenkeeper/initial')
-      .reply(422, 'Reference does not exist')
+      .reply(422)
       .get('/repos/finnp/abc')
       .reply(200, githubRepository)