Skip to content

Commit

Permalink
test,fs: delay unlink in test-regress-nodejsGH-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.

Fixes: nodejs#13800
  • Loading branch information
jaimecbernardo committed Jun 30, 2017
1 parent a84c3be commit e0dfdd2
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, 300, filename);

0 comments on commit e0dfdd2

Please sign in to comment.