@wiajs/request 3.0.1 → 3.0.2

package/tests/test-rfc3986.js

@@ -1,106 +0,0 @@
-'use strict'
-
-var http = require('http')
-var request = require('../index')
-var tape = require('tape')
-
-function runTest (t, options) {
-  var server = http.createServer(function (req, res) {
-    var data = ''
-    req.setEncoding('utf8')
-
-    req.on('data', function (d) {
-      data += d
-    })
-
-    req.on('end', function () {
-      if (options.qs) {
-        t.equal(req.url, '/?rfc3986=%21%2A%28%29%27')
-      }
-      t.equal(data, options._expectBody)
-
-      res.writeHead(200)
-      res.end('done')
-    })
-  })
-
-  server.listen(0, function () {
-    var port = this.address().port
-    request.post('http://localhost:' + port, options, function (err, res, body) {
-      t.equal(err, null)
-      server.close(function () {
-        t.end()
-      })
-    })
-  })
-}
-
-var bodyEscaped = 'rfc3986=%21%2A%28%29%27'
-var bodyJson = '{"rfc3986":"!*()\'"}'
-
-var cases = [
-  {
-    _name: 'qs',
-    qs: {rfc3986: "!*()'"},
-    _expectBody: ''
-  },
-  {
-    _name: 'qs + json',
-    qs: {rfc3986: "!*()'"},
-    json: true,
-    _expectBody: ''
-  },
-  {
-    _name: 'form',
-    form: {rfc3986: "!*()'"},
-    _expectBody: bodyEscaped
-  },
-  {
-    _name: 'form + json',
-    form: {rfc3986: "!*()'"},
-    json: true,
-    _expectBody: bodyEscaped
-  },
-  {
-    _name: 'qs + form',
-    qs: {rfc3986: "!*()'"},
-    form: {rfc3986: "!*()'"},
-    _expectBody: bodyEscaped
-  },
-  {
-    _name: 'qs + form + json',
-    qs: {rfc3986: "!*()'"},
-    form: {rfc3986: "!*()'"},
-    json: true,
-    _expectBody: bodyEscaped
-  },
-  {
-    _name: 'body + header + json',
-    headers: {'content-type': 'application/x-www-form-urlencoded; charset=UTF-8'},
-    body: "rfc3986=!*()'",
-    json: true,
-    _expectBody: bodyEscaped
-  },
-  {
-    _name: 'body + json',
-    body: {rfc3986: "!*()'"},
-    json: true,
-    _expectBody: bodyJson
-  },
-  {
-    _name: 'json object',
-    json: {rfc3986: "!*()'"},
-    _expectBody: bodyJson
-  }
-]
-
-var libs = ['qs', 'querystring']
-
-libs.forEach(function (lib) {
-  cases.forEach(function (options) {
-    options.useQuerystring = (lib === 'querystring')
-    tape(lib + ' rfc3986 ' + options._name, function (t) {
-      runTest(t, options)
-    })
-  })
-})

package/tests/test-stream.js

@@ -1,36 +0,0 @@
-var fs = require('fs')
-var path = require('path')
-var http = require('http')
-var tape = require('tape')
-var request = require('../')
-var server
-
-tape('before', function (t) {
-  server = http.createServer()
-  server.on('request', function (req, res) {
-    req.pipe(res)
-  })
-  server.listen(0, function () {
-    server.url = 'http://localhost:' + this.address().port
-    t.end()
-  })
-})
-
-tape('request body stream', function (t) {
-  var fpath = path.join(__dirname, 'unicycle.jpg')
-  var input = fs.createReadStream(fpath, {highWaterMark: 1000})
-  request({
-    uri: server.url,
-    method: 'POST',
-    body: input,
-    encoding: null
-  }, function (err, res, body) {
-    t.error(err)
-    t.equal(body.length, fs.statSync(fpath).size)
-    t.end()
-  })
-})
-
-tape('after', function (t) {
-  server.close(t.end)
-})

package/tests/test-timeout.js

@@ -1,260 +0,0 @@
-'use strict'
-
-function checkErrCode (t, err) {
-  t.notEqual(err, null)
-  t.ok(err.code === 'ETIMEDOUT' || err.code === 'ESOCKETTIMEDOUT',
-    'Error ETIMEDOUT or ESOCKETTIMEDOUT')
-}
-
-function checkEventHandlers (t, socket) {
-  var connectListeners = socket.listeners('connect')
-  var found = false
-  for (var i = 0; i < connectListeners.length; ++i) {
-    var fn = connectListeners[i]
-    if (typeof fn === 'function' && fn.name === 'onReqSockConnect') {
-      found = true
-      break
-    }
-  }
-  t.ok(!found, 'Connect listener should not exist')
-}
-
-var server = require('./server')
-var request = require('../index')
-var tape = require('tape')
-
-var s = server.createServer()
-
-// Request that waits for 200ms
-s.on('/timeout', function (req, res) {
-  setTimeout(function () {
-    res.writeHead(200, {'content-type': 'text/plain'})
-    res.write('waited')
-    res.end()
-  }, 200)
-})
-
-tape('setup', function (t) {
-  s.listen(0, function () {
-    t.end()
-  })
-})
-
-tape('should timeout', function (t) {
-  var shouldTimeout = {
-    url: s.url + '/timeout',
-    timeout: 100
-  }
-
-  request(shouldTimeout, function (err, res, body) {
-    checkErrCode(t, err)
-    t.end()
-  })
-})
-
-tape('should set connect to false', function (t) {
-  var shouldTimeout = {
-    url: s.url + '/timeout',
-    timeout: 100
-  }
-
-  request(shouldTimeout, function (err, res, body) {
-    checkErrCode(t, err)
-    t.ok(err.connect === false, 'Read Timeout Error should set \'connect\' property to false')
-    t.end()
-  })
-})
-
-tape('should timeout with events', function (t) {
-  t.plan(3)
-
-  var shouldTimeoutWithEvents = {
-    url: s.url + '/timeout',
-    timeout: 100
-  }
-
-  var eventsEmitted = 0
-  request(shouldTimeoutWithEvents)
-    .on('error', function (err) {
-      eventsEmitted++
-      t.equal(1, eventsEmitted)
-      checkErrCode(t, err)
-    })
-})
-
-tape('should not timeout', function (t) {
-  var shouldntTimeout = {
-    url: s.url + '/timeout',
-    timeout: 1200
-  }
-
-  var socket
-  request(shouldntTimeout, function (err, res, body) {
-    t.equal(err, null)
-    t.equal(body, 'waited')
-    checkEventHandlers(t, socket)
-    t.end()
-  }).on('socket', function (socket_) {
-    socket = socket_
-  })
-})
-
-tape('no timeout', function (t) {
-  var noTimeout = {
-    url: s.url + '/timeout'
-  }
-
-  request(noTimeout, function (err, res, body) {
-    t.equal(err, null)
-    t.equal(body, 'waited')
-    t.end()
-  })
-})
-
-tape('negative timeout', function (t) { // should be treated a zero or the minimum delay
-  var negativeTimeout = {
-    url: s.url + '/timeout',
-    timeout: -1000
-  }
-
-  request(negativeTimeout, function (err, res, body) {
-    // Only verify error if it is set, since using a timeout value of 0 can lead
-    // to inconsistent results, depending on a variety of factors
-    if (err) {
-      checkErrCode(t, err)
-    }
-    t.end()
-  })
-})
-
-tape('float timeout', function (t) { // should be rounded by setTimeout anyway
-  var floatTimeout = {
-    url: s.url + '/timeout',
-    timeout: 100.76
-  }
-
-  request(floatTimeout, function (err, res, body) {
-    checkErrCode(t, err)
-    t.end()
-  })
-})
-
-// We need a destination that will not immediately return a TCP Reset
-// packet. StackOverflow suggests these hosts:
-// (https://stackoverflow.com/a/904609/329700)
-var nonRoutable = [
-  '10.255.255.1',
-  '10.0.0.0',
-  '192.168.0.0',
-  '192.168.255.255',
-  '172.16.0.0',
-  '172.31.255.255'
-]
-var nrIndex = 0
-function getNonRoutable () {
-  var ip = nonRoutable[nrIndex]
-  if (!ip) {
-    throw new Error('No more non-routable addresses')
-  }
-  ++nrIndex
-  return ip
-}
-tape('connect timeout', function tryConnect (t) {
-  var tarpitHost = 'http://' + getNonRoutable()
-  var shouldConnectTimeout = {
-    url: tarpitHost + '/timeout',
-    timeout: 100
-  }
-  var socket
-  request(shouldConnectTimeout, function (err) {
-    t.notEqual(err, null)
-    if (err.code === 'ENETUNREACH' && nrIndex < nonRoutable.length) {
-      // With some network configurations, some addresses will be reported as
-      // unreachable immediately (before the timeout occurs). In those cases,
-      // try other non-routable addresses before giving up.
-      return tryConnect(t)
-    }
-    checkErrCode(t, err)
-    t.ok(err.connect === true, 'Connect Timeout Error should set \'connect\' property to true')
-    checkEventHandlers(t, socket)
-    nrIndex = 0
-    t.end()
-  }).on('socket', function (socket_) {
-    socket = socket_
-  })
-})
-
-tape('connect timeout with non-timeout error', function tryConnect (t) {
-  var tarpitHost = 'http://' + getNonRoutable()
-  var shouldConnectTimeout = {
-    url: tarpitHost + '/timeout',
-    timeout: 1000
-  }
-  var socket
-  request(shouldConnectTimeout, function (err) {
-    t.notEqual(err, null)
-    if (err.code === 'ENETUNREACH' && nrIndex < nonRoutable.length) {
-      // With some network configurations, some addresses will be reported as
-      // unreachable immediately (before the timeout occurs). In those cases,
-      // try other non-routable addresses before giving up.
-      return tryConnect(t)
-    }
-    // Delay the check since the 'connect' handler is removed in a separate
-    // 'error' handler which gets triggered after this callback
-    setImmediate(function () {
-      checkEventHandlers(t, socket)
-      nrIndex = 0
-      t.end()
-    })
-  }).on('socket', function (socket_) {
-    socket = socket_
-    setImmediate(function () {
-      socket.emit('error', new Error('Fake Error'))
-    })
-  })
-})
-
-tape('request timeout with keep-alive connection', function (t) {
-  var Agent = require('http').Agent
-  var agent = new Agent({ keepAlive: true })
-  var firstReq = {
-    url: s.url + '/timeout',
-    agent: agent
-  }
-  request(firstReq, function (err) {
-    // We should now still have a socket open. For the second request we should
-    // see a request timeout on the active socket ...
-    t.equal(err, null)
-    var shouldReqTimeout = {
-      url: s.url + '/timeout',
-      timeout: 100,
-      agent: agent
-    }
-    request(shouldReqTimeout, function (err) {
-      checkErrCode(t, err)
-      t.ok(err.connect === false, 'Error should have been a request timeout error')
-      t.end()
-    }).on('socket', function (socket) {
-      var isConnecting = socket._connecting || socket.connecting
-      t.ok(isConnecting !== true, 'Socket should already be connected')
-    })
-  }).on('socket', function (socket) {
-    var isConnecting = socket._connecting || socket.connecting
-    t.ok(isConnecting === true, 'Socket should be new')
-  })
-})
-
-tape('calling abort clears the timeout', function (t) {
-  const req = request({ url: s.url + '/timeout', timeout: 2500 })
-  setTimeout(function () {
-    req.abort()
-    t.equal(req.timeoutTimer, null)
-    t.end()
-  }, 5)
-})
-
-tape('cleanup', function (t) {
-  s.close(function () {
-    t.end()
-  })
-})

package/tests/test-timing.js

@@ -1,147 +0,0 @@
-'use strict'
-
-var server = require('./server')
-var request = require('../index')
-var tape = require('tape')
-var http = require('http')
-
-var plainServer = server.createServer()
-var redirectMockTime = 10
-
-tape('setup', function (t) {
-  plainServer.listen(0, function () {
-    plainServer.on('/', function (req, res) {
-      res.writeHead(200)
-      res.end('plain')
-    })
-    plainServer.on('/redir', function (req, res) {
-      // fake redirect delay to ensure strong signal for rollup check
-      setTimeout(function () {
-        res.writeHead(301, { 'location': 'http://localhost:' + plainServer.port + '/' })
-        res.end()
-      }, redirectMockTime)
-    })
-
-    t.end()
-  })
-})
-
-tape('non-redirected request is timed', function (t) {
-  var options = {time: true}
-
-  var start = new Date().getTime()
-  var r = request('http://localhost:' + plainServer.port + '/', options, function (err, res, body) {
-    var end = new Date().getTime()
-
-    t.equal(err, null)
-    t.equal(typeof res.elapsedTime, 'number')
-    t.equal(typeof res.responseStartTime, 'number')
-    t.equal(typeof res.timingStart, 'number')
-    t.equal((res.timingStart >= start), true)
-    t.equal(typeof res.timings, 'object')
-    t.equal((res.elapsedTime > 0), true)
-    t.equal((res.elapsedTime <= (end - start)), true)
-    t.equal((res.responseStartTime > r.startTime), true)
-    t.equal((res.timings.socket >= 0), true)
-    t.equal((res.timings.lookup >= res.timings.socket), true)
-    t.equal((res.timings.connect >= res.timings.lookup), true)
-    t.equal((res.timings.response >= res.timings.connect), true)
-    t.equal((res.timings.end >= res.timings.response), true)
-    t.equal(typeof res.timingPhases, 'object')
-    t.equal((res.timingPhases.wait >= 0), true)
-    t.equal((res.timingPhases.dns >= 0), true)
-    t.equal((res.timingPhases.tcp >= 0), true)
-    t.equal((res.timingPhases.firstByte > 0), true)
-    t.equal((res.timingPhases.download > 0), true)
-    t.equal((res.timingPhases.total > 0), true)
-    t.equal((res.timingPhases.total <= (end - start)), true)
-
-    // validate there are no unexpected properties
-    var propNames = []
-    for (var propName in res.timings) {
-      if (res.timings.hasOwnProperty(propName)) {
-        propNames.push(propName)
-      }
-    }
-    t.deepEqual(propNames, ['socket', 'lookup', 'connect', 'response', 'end'])
-
-    propNames = []
-    for (propName in res.timingPhases) {
-      if (res.timingPhases.hasOwnProperty(propName)) {
-        propNames.push(propName)
-      }
-    }
-    t.deepEqual(propNames, ['wait', 'dns', 'tcp', 'firstByte', 'download', 'total'])
-
-    t.end()
-  })
-})
-
-tape('redirected request is timed with rollup', function (t) {
-  var options = {time: true}
-  var r = request('http://localhost:' + plainServer.port + '/redir', options, function (err, res, body) {
-    t.equal(err, null)
-    t.equal(typeof res.elapsedTime, 'number')
-    t.equal(typeof res.responseStartTime, 'number')
-    t.equal((res.elapsedTime > 0), true)
-    t.equal((res.responseStartTime > 0), true)
-    t.equal((res.elapsedTime > redirectMockTime), true)
-    t.equal((res.responseStartTime > r.startTime), true)
-    t.end()
-  })
-})
-
-tape('keepAlive is timed', function (t) {
-  var agent = new http.Agent({ keepAlive: true })
-  var options = { time: true, agent: agent }
-  var start1 = new Date().getTime()
-
-  request('http://localhost:' + plainServer.port + '/', options, function (err1, res1, body1) {
-    var end1 = new Date().getTime()
-
-    // ensure the first request's timestamps look ok
-    t.equal((res1.timingStart >= start1), true)
-    t.equal((start1 <= end1), true)
-
-    t.equal((res1.timings.socket >= 0), true)
-    t.equal((res1.timings.lookup >= res1.timings.socket), true)
-    t.equal((res1.timings.connect >= res1.timings.lookup), true)
-    t.equal((res1.timings.response >= res1.timings.connect), true)
-
-    // open a second request with the same agent so we re-use the same connection
-    var start2 = new Date().getTime()
-    request('http://localhost:' + plainServer.port + '/', options, function (err2, res2, body2) {
-      var end2 = new Date().getTime()
-
-      // ensure the second request's timestamps look ok
-      t.equal((res2.timingStart >= start2), true)
-      t.equal((start2 <= end2), true)
-
-      // ensure socket==lookup==connect for the second request
-      t.equal((res2.timings.socket >= 0), true)
-      t.equal((res2.timings.lookup === res2.timings.socket), true)
-      t.equal((res2.timings.connect === res2.timings.lookup), true)
-      t.equal((res2.timings.response >= res2.timings.connect), true)
-
-      // explicitly shut down the agent
-      if (typeof agent.destroy === 'function') {
-        agent.destroy()
-      } else {
-        // node < 0.12
-        Object.keys(agent.sockets).forEach(function (name) {
-          agent.sockets[name].forEach(function (socket) {
-            socket.end()
-          })
-        })
-      }
-
-      t.end()
-    })
-  })
-})
-
-tape('cleanup', function (t) {
-  plainServer.close(function () {
-    t.end()
-  })
-})

package/tests/test-toJSON.js

@@ -1,45 +0,0 @@
-'use strict'
-
-var request = require('../index')
-var http = require('http')
-var tape = require('tape')
-
-var s = http.createServer(function (req, resp) {
-  resp.statusCode = 200
-  resp.end('asdf')
-})
-
-tape('setup', function (t) {
-  s.listen(0, function () {
-    s.url = 'http://localhost:' + this.address().port
-    t.end()
-  })
-})
-
-tape('request().toJSON()', function (t) {
-  var r = request({
-    url: s.url,
-    headers: { foo: 'bar' }
-  }, function (err, res) {
-    var jsonR = JSON.parse(JSON.stringify(r))
-    var jsonRes = JSON.parse(JSON.stringify(res))
-
-    t.equal(err, null)
-
-    t.equal(jsonR.uri.href, r.uri.href)
-    t.equal(jsonR.method, r.method)
-    t.equal(jsonR.headers.foo, r.headers.foo)
-
-    t.equal(jsonRes.statusCode, res.statusCode)
-    t.equal(jsonRes.body, res.body)
-    t.equal(jsonRes.headers.date, res.headers.date)
-
-    t.end()
-  })
-})
-
-tape('cleanup', function (t) {
-  s.close(function () {
-    t.end()
-  })
-})