Skip to content

Commit

Permalink
test: fix test-readline-interface
Browse files Browse the repository at this point in the history
Previous unit test delay is too short for parallel test on raspberry pi,
it will fail sometimes.  This PR use common.platformTimeout and widen
the time gap.

PR-URL: #14677
Ref: #14674
Reviewed-By: Refael Ackermann <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
  • Loading branch information
Azard authored and addaleax committed Aug 10, 2017
1 parent ef8ac7b commit 935d34b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-readline-interface.js
Original file line number Diff line number Diff line change
Expand Up @@ -286,8 +286,8 @@ function isWarned(emitter) {
// over the default crlfDelay but within the setting value
{
const fi = new FakeInput();
const delay = 200;
const crlfDelay = 500;
const delay = 125;
const crlfDelay = common.platformTimeout(1000);
const rli = new readline.Interface({
input: fi,
output: fi,
Expand Down

0 comments on commit 935d34b

Please sign in to comment.