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>
v5.x
Rich Trott 9 years ago
committed by Jeremiah Senkpiel
parent
commit
b19d19efaa
  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