From 6275780fbe0c3d82c6f726d5dc0f1d309403dd44 Mon Sep 17 00:00:00 2001 From: FG Ribreau Date: Sat, 3 Sep 2016 18:49:52 +0200 Subject: [PATCH] fix(tests): bring node-request-retry to a 100% code-coverage --- test/api-surface.test.js | 8 ++++++++ test/promises.test.js | 14 ++++++++++++++ 2 files changed, 22 insertions(+) diff --git a/test/api-surface.test.js b/test/api-surface.test.js index 0df137f..b99da0b 100644 --- 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 index d77991d..1f578a6 100644 --- 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