From 87e291da6480225dc675553a8572f9f379c6df54 Mon Sep 17 00:00:00 2001
From: "David Humphrey (:humph) david.humphrey@senecacollege.ca"
 <david.humphrey@senecacollege.ca>
Date: Mon, 14 Jul 2014 15:51:40 -0400
Subject: [PATCH] Fix issue #235 - Allow recursive watches on root directory

---
 src/fs-watcher.js                   | 10 +++++++---
 tests/index.js                      |  1 +
 tests/spec/fs.watch.spec.js         | 27 ++++++++++++++++++++++++++-
 tests/spec/trailing-slashes.spec.js | 15 +++++++++++++++
 4 files changed, 49 insertions(+), 4 deletions(-)
 create mode 100644 tests/spec/trailing-slashes.spec.js

diff --git a/src/fs-watcher.js b/src/fs-watcher.js
index fcc47e2..1ecd61a 100644
--- a/src/fs-watcher.js
+++ b/src/fs-watcher.js
@@ -1,5 +1,5 @@
 var EventEmitter = require('../lib/eventemitter.js');
-var isNullPath = require('./path.js').isNull;
+var Path = require('./path.js');
 var Intercom = require('../lib/intercom.js');
 
 /**
@@ -26,11 +26,15 @@ function FSWatcher() {
       return;
     }
 
-    if(isNullPath(filename_)) {
+    if(Path.isNull(filename_)) {
       throw new Error('Path must be a string without null bytes.');
     }
+
     // TODO: get realpath for symlinks on filename...
-    filename = filename_;
+
+    // Filer's Path.normalize strips trailing slashes, which we use here.
+    // See https://github.com/js-platform/filer/issues/105
+    filename = Path.normalize(filename_);
 
     // Whether to watch beneath this path or not
     recursive = recursive_ === true;
diff --git a/tests/index.js b/tests/index.js
index 1a59c8c..0fb467a 100644
--- a/tests/index.js
+++ b/tests/index.js
@@ -32,6 +32,7 @@ require("./spec/fs.utimes.spec");
 require("./spec/fs.xattr.spec");
 require("./spec/fs.stats.spec");
 require("./spec/path-resolution.spec");
+require("./spec/trailing-slashes.spec");
 require("./spec/times.spec");
 require("./spec/time-flags.spec");
 require("./spec/fs.watch.spec");
diff --git a/tests/spec/fs.watch.spec.js b/tests/spec/fs.watch.spec.js
index 3126cfd..67af4d2 100644
--- a/tests/spec/fs.watch.spec.js
+++ b/tests/spec/fs.watch.spec.js
@@ -26,7 +26,7 @@ describe('fs.watch', function() {
     });
   });
 
-  it('should get a change event when writing a file in a dir with recursive=true', function(done) {
+  it('should get a change event when writing a file beneath root dir with recursive=true', function(done) {
     var fs = util.fs();
 
     var watcher = fs.watch('/', { recursive: true }, function(event, filename) {
@@ -40,4 +40,29 @@ describe('fs.watch', function() {
       if(error) throw error;
     });
   });
+
+  it('should get a change event when writing a file in a dir with recursive=true', function(done) {
+    var fs = util.fs();
+
+    fs.mkdir('/foo', function(err) {
+      if(err) throw err;
+
+      var watcher = fs.watch('/foo', { recursive: true }, function(event, filename) {
+        expect(event).to.equal('change');
+        expect(filename).to.equal('/foo');
+        watcher.close();
+        done();
+      });
+
+      // This shouldn't produce a change event
+      fs.writeFile('/myfile-not-in-foo', 'data', function(error) {
+        if(error) throw error;
+      });
+
+      // This should
+      fs.writeFile('/foo/myfile', 'data', function(error) {
+        if(error) throw error;
+      });
+    });
+  });
 });
diff --git a/tests/spec/trailing-slashes.spec.js b/tests/spec/trailing-slashes.spec.js
new file mode 100644
index 0000000..c6d1708
--- /dev/null
+++ b/tests/spec/trailing-slashes.spec.js
@@ -0,0 +1,15 @@
+var Path = require('../..').Path;
+var expect = require('chai').expect;
+
+describe('Path.normalize and trailing slashes', function() {
+
+  it('should remove trailing slashes as expected', function() {
+    var strip = Path.normalize;
+
+    expect(strip('/')).to.equal('/');
+    expect(strip('/foo/')).to.equal('/foo');
+    expect(strip('/foo//')).to.equal('/foo');
+    expect(strip('/foo/bar/baz/')).to.equal('/foo/bar/baz');
+  });
+
+});