test: use platform-based timeout for reliability
authorRich Trott <rtrott@gmail.com>
Wed, 25 Nov 2015 04:59:49 +0000 (20:59 -0800)
committerMyles Borins <mborins@us.ibm.com>
Tue, 19 Jan 2016 19:52:15 +0000 (11:52 -0800)
test-http-client-timeout-with-data fails on Raspberry Pi in CI from time
to time.

Use a platform-based timeout to improve reliability.

PR-URL: https://github.com/nodejs/node/pull/4015
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Roman Reiss <me@silverwind.io>
test/parallel/test-http-client-timeout-with-data.js

index 0aefe62..673908f 100644 (file)
@@ -1,7 +1,7 @@
 'use strict';
-var common = require('../common');
-var assert = require('assert');
-var http = require('http');
+const common = require('../common');
+const assert = require('assert');
+const http = require('http');
 
 var ntimeouts = 0;
 var nchunks = 0;
@@ -11,21 +11,21 @@ process.on('exit', function() {
   assert.equal(nchunks, 2);
 });
 
-var options = {
+const options = {
   method: 'GET',
   port: common.PORT,
   host: '127.0.0.1',
   path: '/'
 };
 
-var server = http.createServer(function(req, res) {
+const server = http.createServer(function(req, res) {
   res.writeHead(200, {'Content-Length':'2'});
   res.write('*');
-  setTimeout(function() { res.end('*'); }, 100);
+  setTimeout(function() { res.end('*'); }, common.platformTimeout(100));
 });
 
 server.listen(options.port, options.host, function() {
-  var req = http.request(options, onresponse);
+  const req = http.request(options, onresponse);
   req.end();
 
   function onresponse(res) {