snoop 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5110d8e59d54d0852a8345496faf7c98a1b8499a
-  data.tar.gz: 8fa5cda09082e4da2e339e5018aa12b7d216ac39
+  metadata.gz: 4cd732ffdcde8dc20c8859e3206b5d3b860e7865
+  data.tar.gz: 5ef83ae68a60de3e5ad236b9c6f4bbb757218769
 SHA512:
-  metadata.gz: 5981dddcfee4514309c22cc7fb51c0256c3e2452adc5d5b7e3167a56f28cd922176bf4c5d3ba1b1249eabefe93683a72476087ea4d3ff571e5c68ee6bee675d3
-  data.tar.gz: 9189b8ef3880a7d7a0d14aa20c5f5b82147ff5a2a4d8df23514cf0139f526f5990d9c28fdebfe9f83f1626caccafd08c7494c128f5a5048dca26e85ba4b1e8a0
+  metadata.gz: 50552b68abf7e15d0c19b7b5a38e6fd46c13404ba2a79c2833ecec87dd58f761349e0be0c4cf8873c6bcddae31fd5bd0e2c1af42716a6cfc27751432011bd9e4
+  data.tar.gz: 1a04911d8ed62afb356a253c87b0b36d6299c073798130042ad3c35f7dc31fcf86efbe98cdd4e460e050a16149692ea04789fdae168496dfec30ad485dd25b40

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    snoop (0.0.1)
+    snoop (0.0.2)
       httparty
 
 GEM

data/README.md

@@ -1,5 +1,9 @@
 # Snoop `wip`
-Monitor content for changes and be notified
+Be notified when content changes.
+
+Supported snoops:
+
+  - `HTTP`
 
 ## Installation
 

data/lib/snoop/{http_monitor.rb → http.rb}

@@ -1,33 +1,28 @@
 require 'httparty'
 
 module Snoop
-  class HttpMonitor
+  class Http
     UrlRequiredException = Class.new(StandardError)
 
-    attr_reader :url, :notifier, :content, :http_client, :interval
+    attr_reader :url, :http_client, :interval, :content
     attr_accessor :content
 
-    def initialize(
-      url: nil, http_client: HTTParty, notifier: MacOSNotifier.new, interval: 1
-    )
+    def initialize(url: nil, http_client: HTTParty)
       raise UrlRequiredException if url.nil?
+
       @url = url
       @http_client = http_client
-      @notifier = notifier
-      @interval = interval
     end
 
-    def monitor
-      while true
-        notify
-        sleep interval
+    def notify(
+      delay: 0, times: 1, while_true: -> { false }, while_false: -> { true }
+    )
+      while (times -= 1) >= 0 || while_true.call || !while_false.call
+        yield content if content_changed?
+        sleep delay
       end
     end
 
-    def notify
-      notifier.notify if content_changed?
-    end
-
     def content_changed?
       old_content = @content
       @content = fetch_content
@@ -38,7 +33,4 @@ module Snoop
       http_client.get(url).body
     end
   end
-
-  class MacOSNotifier
-  end
 end

data/lib/snoop/version.rb

@@ -1,3 +1,3 @@
 module Snoop
-  VERSION = '0.0.1'
+  VERSION = '0.0.2'
 end

data/lib/snoop.rb

@@ -1,2 +1,2 @@
 require 'snoop/version'
-require 'snoop/http_monitor'
+require 'snoop/http'

data/spec/acceptance/http_spec.rb

@@ -0,0 +1,30 @@
+require 'snoop/http'
+require 'support/http_server'
+
+describe Snoop::Http do
+  with_http_server do |url|
+    it 'notifies when content has changed' do
+      snoop = described_class.new(url: "#{url}/dynamic-content")
+
+      notification_count = 0
+
+      snoop.notify times: 2 do
+        notification_count += 1
+      end
+
+      expect(notification_count).to eq 2
+    end
+
+    it 'does not notify when content has not changed' do
+      snoop = described_class.new(url: "#{url}/static-content")
+
+      notification_count = 0
+
+      snoop.notify times: 2 do
+        notification_count += 1
+      end
+
+      expect(notification_count).to eq 1
+    end
+  end
+end

data/spec/unit/{http_monitor_spec.rb → http_spec.rb}

@@ -1,6 +1,6 @@
-require 'snoop/http_monitor'
+require 'snoop/http'
 
-describe Snoop::HttpMonitor do
+describe Snoop::Http do
   subject { described_class.new(url: url) }
 
   let(:url) { 'http://example.com' }
@@ -16,36 +16,69 @@ describe Snoop::HttpMonitor do
       it 'raises an exception' do
         expect {
           subject
-        }.to raise_error Snoop::HttpMonitor::UrlRequiredException
+        }.to raise_error Snoop::Http::UrlRequiredException
       end
     end
   end
 
   describe '#notify' do
-    let(:notifier) { stub }
+    let(:content) { 'abc123' }
+    let(:content_changed?) { true }
 
     before do
       subject.stub(
-        content_changed?: content_changed?,
-        notifier: notifier
+        content: content,
+        content_changed?: content_changed?
       )
     end
 
-    context 'when the content changes' do
-      let(:content_changed?) { true }
+    it 'notifies the requested number of times' do
+      notification_count = 0
 
-      it 'triggers a notification' do
-        notifier.should_receive(:notify)
-        subject.notify
+      subject.notify times: 5 do
+        notification_count += 1
       end
+
+      expect(notification_count).to eq 5
+    end
+
+    it 'notifies while an expression is true' do
+      notification_count = 0
+
+      subject.notify while_true: -> { notification_count < 3 }  do
+        notification_count += 1
+      end
+
+      expect(notification_count).to eq 3
+    end
+
+    it 'notifies while an expression is false' do
+      notification_count = 0
+
+      subject.notify while_false: -> { notification_count > 2 }  do
+        notification_count += 1
+      end
+
+      expect(notification_count).to eq 3
+    end
+
+    it 'yields the content to the notification block' do
+      yielded_content = nil
+
+      subject.notify { |content| yielded_content = content }
+
+      expect(yielded_content).to eq content
     end
 
     context 'when the content does not change' do
       let(:content_changed?) { false }
 
-      it 'does not trigger a notification' do
-        notifier.should_not_receive(:notify)
-        subject.notify
+      it 'does not yield the notification block' do
+        yielded = false
+
+        subject.notify { yielded = true }
+
+        expect(yielded).to be_false
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: snoop
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Chris Hunt
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-10 00:00:00.000000000 Z
+date: 2013-04-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -108,12 +108,12 @@ files:
 - README.md
 - Rakefile
 - lib/snoop.rb
-- lib/snoop/http_monitor.rb
+- lib/snoop/http.rb
 - lib/snoop/version.rb
 - snoop.gemspec
-- spec/acceptance/http_monitor_spec.rb
+- spec/acceptance/http_spec.rb
 - spec/support/http_server.rb
-- spec/unit/http_monitor_spec.rb
+- spec/unit/http_spec.rb
 homepage: https://github.com/chrishunt/snoop
 licenses:
 - MIT
@@ -139,6 +139,6 @@ signing_key:
 specification_version: 4
 summary: Monitor content for changes and be notified
 test_files:
-- spec/acceptance/http_monitor_spec.rb
+- spec/acceptance/http_spec.rb
 - spec/support/http_server.rb
-- spec/unit/http_monitor_spec.rb
+- spec/unit/http_spec.rb

data/spec/acceptance/http_monitor_spec.rb

@@ -1,40 +0,0 @@
-require 'snoop/http_monitor'
-require 'support/http_server'
-
-describe Snoop::HttpMonitor do
-  class TestNotifier
-    attr_reader :notification_count
-
-    def initialize(notification_count = 0)
-      @notification_count = notification_count
-    end
-
-    def notify
-      @notification_count += 1
-    end
-  end
-
-  let(:notifier) { TestNotifier.new }
-
-  with_http_server do |url|
-    it 'notifies when http content has changed' do
-      monitor = described_class.new(
-        url: "#{url}/dynamic-content",
-        notifier: notifier
-      )
-
-      2.times { monitor.notify }
-      expect(notifier.notification_count).to eq 2
-    end
-
-    it 'does not notify when http content has not changed' do
-      monitor = described_class.new(
-        url: "#{url}/static-content",
-        notifier: notifier
-      )
-
-      2.times { monitor.notify }
-      expect(notifier.notification_count).to eq 1
-    end
-  end
-end