doc,test: enable recursive file watching in Windows

Recursive file watching is supported by libuv since 1.7.0. Refer https://github.com/nodejs/node/blob/master/deps/uv/ChangeLog#L126. This patch notes that in the docs and enables testing this feature. It also adds proper TAP plugin parsable message for other platforms. PR-URL: #2649 Fixes: #375 Reviewed-By: rvagg - Rod Vagg <[email protected]> Reviewed-By: silverwind - Roman Reiss <[email protected]>
nodejs · Sep 6, 2015 · 8372ea2 · 8372ea2
1 parent aa1140e
commit 8372ea2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/doc/api/fs.markdown b/doc/api/fs.markdown
@@ -625,9 +625,9 @@ the event.
 The `fs.watch` API is not 100% consistent across platforms, and is
 unavailable in some situations.
 
-The recursive option is currently supported on OS X. Only FSEvents supports this
-type of file watching so it is unlikely any additional platforms will be added
-soon.
+The recursive option is currently supported on OS X and Windows. Only FSEvents
+ supports this type of file watching so it is unlikely any additional platforms
+ will be added soon.
 
 #### Availability
 

diff --git a/test/sequential/test-fs-watch-recursive.js b/test/sequential/test-fs-watch-recursive.js
@@ -4,7 +4,7 @@ var assert = require('assert');
 var path = require('path');
 var fs = require('fs');
 
-if (process.platform === 'darwin') {
+if (process.platform === 'darwin' || common.isWindows) {
   var watchSeenOne = 0;
 
   var testDir = common.tmpDir;
@@ -46,4 +46,6 @@ if (process.platform === 'darwin') {
   setTimeout(function() {
     fs.writeFileSync(filepathOne, 'world');
   }, 10);
+} else {
+  console.log('1..0 # Skipped: recursive option is darwin/windows specific');
 }