From 5c057e11b19fdbedfa407f1e3bc53474945abc46 Mon Sep 17 00:00:00 2001
From: Nathan Sobo <nathan@github.com>
Date: Wed, 25 Apr 2012 20:14:35 -0600
Subject: [PATCH] Make unwatchPath spec make a valid negative assertion and
 return the watchId from $native.watchPath

---
 Atom/src/native_handler.mm     |  3 ++-
 spec/app/directory-spec.coffee | 16 ++++++++--------
 2 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/Atom/src/native_handler.mm b/Atom/src/native_handler.mm
index 18141a10d..7f5a8562a 100644
--- a/Atom/src/native_handler.mm
+++ b/Atom/src/native_handler.mm
@@ -296,7 +296,8 @@ bool NativeHandler::Execute(const CefString& name,
       context->Exit();
     };
 
-    [PathWatcher watchPath:path callback:[[callback copy] autorelease]];
+    NSString *watchId = [PathWatcher watchPath:path callback:[[callback copy] autorelease]];
+    retval = CefV8Value::CreateString([watchId UTF8String]);
     
     return true;
   }
diff --git a/spec/app/directory-spec.coffee b/spec/app/directory-spec.coffee
index 531c7ac25..907dc05aa 100644
--- a/spec/app/directory-spec.coffee
+++ b/spec/app/directory-spec.coffee
@@ -1,13 +1,13 @@
 Directory = require 'directory'
 fs = require 'fs'
 
-describe "Directory", ->
+fdescribe "Directory", ->
   directory = null
 
   beforeEach ->
     directory = new Directory(require.resolve('fixtures'))
 
-  fdescribe "when the contents of the directory change on disk", ->
+  describe "when the contents of the directory change on disk", ->
     temporaryFilePath = null
 
     beforeEach ->
@@ -33,7 +33,7 @@ describe "Directory", ->
 
       waitsFor "second change", -> changeHandler.callCount > 0
 
-  fdescribe "when the directory unsubscribes from events", ->
+  describe "when the directory unsubscribes from events", ->
     temporaryFilePath = null
 
     beforeEach ->
@@ -51,14 +51,14 @@ describe "Directory", ->
         directory.on 'contents-change', changeHandler
         fs.write(temporaryFilePath, '')
 
-      waitsFor "first change", -> changeHandler.callCount > 0
+      waitsFor "change event", -> changeHandler.callCount > 0
 
       runs ->
         changeHandler.reset()
         directory.unsubscribeFromNativeChangeEvents()
+      waits 20
 
-      ticks = 0
-      waitsFor "100 ticks", -> ticks++ < 100
+      runs -> fs.remove(temporaryFilePath)
+      waits 20
+      runs -> expect(changeHandler.callCount).toBe 0
 
-      runs ->
-        expect(changeHandler.callCount).toBe 0