Skip to content

Commit

Permalink
test,fs: delay unlink in test-regress-GH-4027.js
Browse files Browse the repository at this point in the history
The sequential/test-regress-GH-4027 test is flaky with an increased
system load, failing when the watched file is unlinked before the
first state of the watched file is retrieved.

After increasing the delay before unlinking and calling setTimeout
after watchFile, the flakiness stopped reproducing.

PR-URL: #14010
Fixes: #13800
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
  • Loading branch information
jaimecbernardo authored and addaleax committed Jul 11, 2017
1 parent 1dcb086 commit 2daff32
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/sequential/test-regress-GH-4027.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,11 @@ common.refreshTmpDir();

const filename = path.join(common.tmpDir, 'watched');
fs.writeFileSync(filename, 'quis custodiet ipsos custodes');
setTimeout(fs.unlinkSync, 100, filename);

fs.watchFile(filename, { interval: 50 }, common.mustCall(function(curr, prev) {
assert.strictEqual(prev.nlink, 1);
assert.strictEqual(curr.nlink, 0);
fs.unwatchFile(filename);
}));

setTimeout(fs.unlinkSync, common.platformTimeout(300), filename);

0 comments on commit 2daff32

Please sign in to comment.