From: Fedor Indutny Date: Thu, 21 Nov 2013 17:06:29 +0000 (+0400) Subject: test: make fs-watch-recursive less racy X-Git-Tag: v0.11.10~47 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=78cd4533d9f14cd83e9cd7817dffdd4e6af600c5;p=platform%2Fupstream%2Fnodejs.git test: make fs-watch-recursive less racy FSEventStream may emit events that happened right before it has started. Ignore changes emitted for the directory itself, since they may come from the stale events. --- diff --git a/test/simple/test-fs-watch-recursive.js b/test/simple/test-fs-watch-recursive.js index a5ebad6..e421ab0 100644 --- a/test/simple/test-fs-watch-recursive.js +++ b/test/simple/test-fs-watch-recursive.js @@ -30,7 +30,8 @@ if (process.platform === 'darwin') { var testDir = common.tmpDir; var filenameOne = 'watch.txt'; - var testsubdir = path.join(testDir, 'testsubdir'); + var testsubdirName = 'testsubdir'; + var testsubdir = path.join(testDir, testsubdirName); var relativePathOne = path.join('testsubdir', filenameOne); var filepathOne = path.join(testsubdir, filenameOne); @@ -44,12 +45,16 @@ if (process.platform === 'darwin') { }; try { fs.mkdirSync(testsubdir, 0700); } catch (e) {} - fs.writeFileSync(filepathOne, 'hello'); assert.doesNotThrow(function() { var watcher = fs.watch(testDir, {recursive: true}); watcher.on('change', function(event, filename) { assert.ok('change' === event || 'rename' === event); + + // Ignore stale events generated by mkdir + if (filename === testsubdirName) + return; + assert.equal(relativePathOne, filename); watcher.close();