Stream-start will raise an exception on error.

ymendel · Jun 12, 2008 · 761dba4 · 761dba4
1 parent a12b52a
commit 761dba4
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 2 deletions.
diff --git a/lib/fsevents/stream.rb b/lib/fsevents/stream.rb
@@ -22,7 +22,7 @@ def schedule
     end
 
     def start
-      OSX.FSEventStreamStart(stream)
+      OSX.FSEventStreamStart(stream) or raise StreamError, 'Could not start stream'
     end
 
     def startup

diff --git a/spec/stream_spec.rb b/spec/stream_spec.rb
@@ -198,10 +198,23 @@
   end
 
   describe 'when starting' do
+    before :each do
+      OSX.stubs(:FSEventStreamStart).returns(true)
+    end
+
     it 'should start the stream' do
-      OSX.expects(:FSEventStreamStart).with(@stream.stream)
+      OSX.expects(:FSEventStreamStart).with(@stream.stream).returns(true)
       @stream.start
     end
+
+    it 'should raise a StreamError exception if the stream could not be started' do
+      OSX.stubs(:FSEventStreamStart).returns(nil)
+      lambda { @stream.start }.should raise_error(FSEvents::Stream::StreamError)
+    end
+
+    it 'should not raise a StreamError exception if the stream could be started' do
+      lambda { @stream.start }.should_not raise_error(FSEvents::Stream::StreamError)
+    end
   end
 
   it 'should stop itself' do