Merge pull request #113 from WojtekKruszewski/notify_option

Optionally push system notifications when reloading

README.md

@@ -67,14 +67,16 @@ end
 Available options:
 
 ``` ruby
+notify: true          # default false
 host: '127.3.3.1'     # default '0.0.0.0'
 port: '12345'         # default '35729'
 apply_css_live: false # default true
 override_url: false   # default false
 grace_period: 0.5     # default 0 (seconds)
 ```
 
-See [LiveReload configuration doc](https://github.com/mockko/livereload/blob/master/README-old.md) from version 1.x for more info about those options.
+`notify` uses Guard's [system notifications](https://github.com/guard/guard/wiki/System-notifications).
+See [LiveReload configuration doc](https://github.com/mockko/livereload/blob/master/README-old.md) from version 1.x for more info about other options.
 
 ## Development
 

lib/guard/livereload/reactor.rb

@@ -17,7 +17,12 @@ def stop
       end
 
       def reload_browser(paths = [])
-        UI.info "Reloading browser: #{paths.join(' ')}"
+        msg = "Reloading browser: #{paths.join(' ')}"
+        UI.info msg
+        if options[:notify]
+          Notifier.notify(msg, title: 'Reloading browser', image: :success)
+        end
+
         paths.each do |path|
           data = _data(path)
           UI.debug(data)

spec/lib/guard/livereload/reactor_spec.rb

@@ -6,10 +6,22 @@
 
   describe "#reload_browser(paths = [])" do
     it "displays a message" do
-      expect(Guard::UI).to receive(:info).with("Reloading browser: stylesheets/layout.css stylesheets/style.css")
+      expect(Guard::UI).to receive(:info).
+        with('Reloading browser: stylesheets/layout.css stylesheets/style.css')
       new_live_reactor.reload_browser(paths)
     end
 
+    it 'by default does not send notification' do
+      expect(::Guard::Notifier).to_not receive(:notify)
+      new_live_reactor.reload_browser(paths)
+    end
+
+    it 'optionally pushes notification' do
+      expect(::Guard::Notifier).to receive(:notify).
+        with(kind_of(String), have_key(:title))
+      new_live_reactor(notify: true).reload_browser(paths)
+    end
+
     it "each web socket receives send with data containing default options for each path modified" do
       reactor = new_live_reactor
       paths.each do |path|