Browse Source

test: extend timeout in Debug mode

Debug mode slows execution speed. There is work afoot to enable Debug
mode runs on the continuous integration infrastructure for the project.
Some tests are timing out, such as test-net-GH-5504.js.

This change doubles the timeout returned from `common.platformTimeout()`
when running in Debug mode. It also removes an unused variable from the
aforementioned test-net-GH-5504.js.

PR-URL: https://github.com/nodejs/node/pull/4431
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
process-exit-stdio-flushing
Rich Trott 9 years ago
parent
commit
11136bf334
  1. 3
      test/common.js
  2. 1
      test/sequential/test-net-GH-5504.js

3
test/common.js

@ -255,6 +255,9 @@ exports.spawnPwd = function(options) {
};
exports.platformTimeout = function(ms) {
if (process.config.target_defaults.default_configuration === 'Debug')
ms = 2 * ms;
if (process.arch !== 'arm')
return ms;

1
test/sequential/test-net-GH-5504.js

@ -1,6 +1,5 @@
'use strict';
var common = require('../common');
var assert = require('assert');
// this test only fails with CentOS 6.3 using kernel version 2.6.32
// On other linuxes and darwin, the `read` call gets an ECONNRESET in

Loading…
Cancel
Save