fix(tests): bring node-request-retry to a 100% code-coverage

FGRibreau · Sep 3, 2016 · 6275780 · 6275780
1 parent e6c2160
commit 6275780
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 0 deletions.
diff --git a/test/api-surface.test.js b/test/api-surface.test.js
@@ -5,6 +5,14 @@ var t = require('chai').assert;
 
 describe('API surface', function () {
 
+  describe('request methods', function(){
+    it('should work with .on', function(f){
+      request('http://httpbin.org/delay/0.1').on('end', function(){
+        f();
+      });
+    });
+  });
+
   describe('callback api', function(){
     [['request', request], ['request.get', request.get]].forEach(function(pair){
       it('should work with '+pair[0]+'(url, f)', function (done) {

diff --git a/test/promises.test.js b/test/promises.test.js
@@ -58,6 +58,20 @@ describe('Promises support', function () {
     });
   });
 
+  it('should reject the promise on request aborted', function (done) {
+    var req = request({
+      url: 'http://www.filltext.com/?rows=1', // return 1 row of data
+    });
+
+    req._req = null;
+    req.abort();
+
+    req.catch(function (err) {
+      t.strictEqual(err.message, 'Aborted');
+      done();
+    });
+  });
+
   it('should reject the response on any error', function (done) {
     request({
       url: 'http://localhost:1', // return 1 row of data