Merge pull request #262 from techpeace/features/add-429-detection

Handle 429 Too Many Requests status

Author: jshemas

lib/request.ts

@@ -90,6 +90,8 @@ export default async function requestAndResultsFormatter(options: OpenGraphScrap
           throw new Error('408 Request Timeout');
         case 410:
           throw new Error('410 Gone');
+        case 429:
+          throw new Error('429 Too Many Requests');
         case 500:
           throw new Error('500 Internal Server Error');
         case 502:

tests/unit/openGraphScraper.spec.ts

@@ -505,6 +505,28 @@ describe('return ogs', function () {
         });
     });
 
+    it('when the request sends a 429 error', function () {
+      mockAgent.get('http://www.testerror.com')
+        .intercept({ path: '/' })
+        .reply(429, {
+          message: '429 Too Many Requests',
+          code: '429',
+        });
+
+      return ogs({ url: 'http://www.testerror.com' })
+        .then(function () {
+          expect('').to.be.eql('this should not happen');
+        })
+        .catch(function (data) {
+          expect(data.error).to.be.eql(true);
+          expect(data.result.error).to.eql('429 Too Many Requests');
+          expect(data.result.errorDetails.toString()).to.eql('Error: 429 Too Many Requests');
+          expect(data.result.success).to.eql(false);
+          expect(data.response).to.be.eql(undefined);
+          expect(data.html).to.be.eql(undefined);
+        });
+    });
+
     it('when the request sends a general error', function () {
       mockAgent.get('http://www.testerror.com')
         .intercept({ path: '/' })