feat: ignore HTTP 422 errors

octokit · Feb 10, 2019 · 5e1d577 · 5e1d577
1 parent 8c5cd82
commit 5e1d577
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/index.js b/lib/index.js
@@ -7,7 +7,7 @@ function retryPlugin (octokit, octokitOptions = {}) {
   const state = Object.assign({
     enabled: true,
     retryAfterBaseValue: 1000,
-    doNotRetry: [ 400, 401, 403, 404 ],
+    doNotRetry: [ 400, 401, 403, 404, 422 ],
     retries: 3
   }, octokitOptions.retry)
 

diff --git a/test/integration/backend-errors.js b/test/integration/backend-errors.js
@@ -30,10 +30,10 @@ describe('Trigger Retries', function () {
     expect(octokit.__requestTimings[3] - octokit.__requestTimings[2]).to.be.closeTo(450, 20)
   })
 
-  it('Should not retry 3xx/400/401/403 errors', async function () {
+  it('Should not retry 3xx/400/401/403/422 errors', async function () {
     const octokit = new Octokit({ retry: { retryAfterBaseValue: 50 } })
     let caught = 0
-    const testStatuses = [ 304, 400, 401, 403, 404 ]
+    const testStatuses = [ 304, 400, 401, 403, 404, 422 ]
 
     for (const status of testStatuses) {
       try {