audible 0.2.0 → 0.4.0

data/lib/audible.rb

@@ -1,3 +1,3 @@
-dirname = File.expand_path File.dirname(__FILE__)
-
-Dir.glob(File.join(dirname, "audible", "**", "*.rb")).each{|f| require f}
+dirname = File.expand_path File.dirname(__FILE__)
+
+Dir.glob(File.join(dirname, "audible", "**", "*.rb")).each{|f| require f}

data/lib/audible/audible.rb

@@ -1,42 +1,50 @@
-module Audible
-  def on(*events, &block)
-    events.each{|e| attach(e, &block)}
-  end
-
-  def relay(source, event, opts ={})
-    name = opts[:as] || event
-    source.on(event){|e,args| notify name, args.first}
-  end
-
-  protected
-
-  def attach(event,&block)
-    fail no_way_to_notify unless block_given?
-
-    listeners_for(event) << block
-  end
-
-  def notify(event, *args)
-    listeners_for(event).each do |listener|
-      listener.call event, args
-    end
-  end
-
-  def accepts?(e); accept_all_by_default; end
-
-  private
-
-  def no_way_to_notify
-    "No block supplied. How will I notify you?"
-  end
-
-  def accept_all_by_default; true; end
-
-  def listeners_for(event)
-    fail "Event <#{event}> not supported by #{self.class}." unless accepts?(event) === true
-    listeners[event] = [] unless listeners.has_key? event
-    listeners[event]
-  end
-
-  def listeners; @listeners ||= {}; end
-end
+module Audible
+  def on(*events, &block)
+    events.each{|e| attach(e, &block)}
+  end
+
+  def relay(source, event, opts ={})
+    name = opts[:as] || event
+    source.on(event){|e,args| notify name, args.first}
+  end
+
+  protected
+
+  def attach(event,&block)
+    fail no_way_to_notify unless block_given?
+
+    listeners_for(event) << block
+  end
+
+  def notify(event, *args)
+    the_first_error = nil
+    
+    listeners_for(event).each do |listener|
+      begin
+        listener.call event, args
+      rescue Exception => e
+        the_first_error ||= e
+      end
+    end
+    
+    raise the_first_error if the_first_error
+  end
+
+  def accepts?(e); accept_all_by_default; end
+
+  private
+
+  def no_way_to_notify
+    "No block supplied. How will I notify you?"
+  end
+
+  def accept_all_by_default; true; end
+
+  def listeners_for(event)
+    fail "Event <#{event}> not supported by #{self.class}." unless accepts?(event) === true
+    listeners[event] = [] unless listeners.has_key? event
+    listeners[event]
+  end
+
+  def listeners; @listeners ||= {}; end
+end

data/lib/audible/pidfile.rb

@@ -1,16 +1,16 @@
-class Pidfile
-  require "fileutils"; extend FileUtils
-
-  class << self
-    def new(pid=$$)
-      File.open path, "w" do |f|
-        f.puts pid
-      end
-    end
-
-    def missing?; false == exists? end
-    def exists?; File.exists? path; end
-    def delete; rm path; end
-    def path; File.join ".", ".pid"; end
-  end
-end
+class Pidfile
+  require "fileutils"; extend FileUtils
+
+  class << self
+    def new(pid=$$)
+      File.open path, "w" do |f|
+        f.puts pid
+      end
+    end
+
+    def missing?; false == exists? end
+    def exists?; File.exists? path; end
+    def delete; rm path; end
+    def path; File.join ".", ".pid"; end
+  end
+end

data/spec/spec_helper.rb

@@ -1,7 +1,7 @@
-require "rspec"
-require File.join ".", "lib", "audible"
-
-Object.class_eval do
-  alias :must :should
-  alias :must_not :should_not
-end
+require "rspec"
+require File.join ".", "lib", "audible"
+
+Object.class_eval do
+  alias :must :should
+  alias :must_not :should_not
+end

data/spec/system.tests/drb/bin/server

@@ -1,45 +1,45 @@
-require File.join ".", "lib", "audible"
-
-class TimeServer
-  require 'drb/observer'
-  include DRb::DRbObservable
-  include Audible
-
-  def get_current_time
-    return Time.now
-  end
-
-  def progress
-    3.times do 
-      changed
-      notify_observers Time.now.to_s
-      notify :progress, Time.now.to_s
-    end		    
-  end   
-end
-
-FRONT_OBJECT=TimeServer.new
-
-$SAFE = 0
-
-require 'drb/drb'
-
-port = 9999
-
-uri="druby://127.0.0.1:#{port}"
-
-DRb.start_service(uri, FRONT_OBJECT)
-
-puts "Server has started on port <#{port}>, process <#{$$}>"
-
-Signal.trap("INT") do
-  puts "Stopping..."
-
-  Pidfile.delete
-
-  exit
-end
-
-Pidfile.new $$
-
+require File.join ".", "lib", "audible"
+
+class TimeServer
+  require 'drb/observer'
+  include DRb::DRbObservable
+  include Audible
+
+  def get_current_time
+    return Time.now
+  end
+
+  def progress
+    3.times do 
+      changed
+      notify_observers Time.now.to_s
+      notify :progress, Time.now.to_s
+    end		    
+  end   
+end
+
+FRONT_OBJECT=TimeServer.new
+
+$SAFE = 0
+
+require 'drb/drb'
+
+port = 9999
+
+uri="druby://127.0.0.1:#{port}"
+
+DRb.start_service(uri, FRONT_OBJECT)
+
+puts "Server has started on port <#{port}>, process <#{$$}>"
+
+Signal.trap("INT") do
+  puts "Stopping..."
+
+  Pidfile.delete
+
+  exit
+end
+
+Pidfile.new $$
+
 DRb.thread.join

data/spec/system.tests/drb/drb_observers_spec.rb

@@ -1,65 +1,65 @@
-require "spec_helper"
-require_relative File.join 'support', 'drb_server'
- 
-describe "Basic drb connections" do
-  before :all do
-    @server = DrbServer.new("druby://127.0.0.1:9999").tap {|server| server.start}
-    @timeserver = DRbObject.new_with_uri @server.url 
-  end
-
-  after :all do
-    @server.kill
-  end
-
-  before do
-    @an_observer = Class.new do
-      def update(args)
-        @notified = true
-      end
-      
-      def notified?; @notified === true; end
-
-      def reset; @notified = false; end
-    end.new
-  end
-
-  it "tells me the time" do
-    expect(@timeserver.get_current_time.to_s).to match /^#{Time.now.year}/
-  end
-
-  it "can notify" do
-    @timeserver.add_observer @an_observer
-
-    @timeserver.progress
-    
-    expect(@an_observer).to be_notified
-  end
-
-  it "can detach observer which stops notifications" do
-    @timeserver.add_observer @an_observer
-
-    @timeserver.progress
-    
-    expect(@an_observer).to be_notified
-
-    @an_observer.reset
-
-    @timeserver.delete_observer @an_observer
-
-    @timeserver.progress
-    
-    expect(@an_observer).to_not be_notified
-  end
-
-  it "can also get audible notifications" do
-    notified = false
-
-    @timeserver.on(:progress) do |e,args|
-      notified = true
-    end
-
-    @timeserver.progress
-
-    expect(notified).to be_true
-  end
-end
+require "spec_helper"
+require_relative File.join 'support', 'drb_server'
+ 
+describe "Basic drb connections" do
+  before :all do
+    @server = DrbServer.new("druby://127.0.0.1:9999").tap {|server| server.start}
+    @timeserver = DRbObject.new_with_uri @server.url 
+  end
+
+  after :all do
+    @server.kill
+  end
+
+  before do
+    @an_observer = Class.new do
+      def update(args)
+        @notified = true
+      end
+      
+      def notified?; @notified === true; end
+
+      def reset; @notified = false; end
+    end.new
+  end
+
+  it "tells me the time" do
+    expect(@timeserver.get_current_time.to_s).to match /^#{Time.now.year}/
+  end
+
+  it "can notify" do
+    @timeserver.add_observer @an_observer
+
+    @timeserver.progress
+    
+    expect(@an_observer).to be_notified
+  end
+
+  it "can detach observer which stops notifications" do
+    @timeserver.add_observer @an_observer
+
+    @timeserver.progress
+    
+    expect(@an_observer).to be_notified
+
+    @an_observer.reset
+
+    @timeserver.delete_observer @an_observer
+
+    @timeserver.progress
+    
+    expect(@an_observer).to_not be_notified
+  end
+
+  it "can also get audible notifications" do
+    notified = false
+
+    @timeserver.on(:progress) do |e,args|
+      notified = true
+    end
+
+    @timeserver.progress
+
+    expect(notified).to be_true
+  end
+end

data/spec/system.tests/drb/support/drb_server.rb

@@ -1,26 +1,26 @@
-class DrbServer
-  attr_reader :url
-
-  def initialize(url)
-    @url = url
-  end
-
-  def start
-    exe = File.join(File.dirname(__FILE__), "..", "bin", "server")
-    
-    @pid = Process.spawn "bundle exec ruby #{exe}", :err=>:out, :out => ".log"
-
-    while Pidfile.missing?
-      sleep 0.25
-    end
-
-    require 'drb/drb'
-
-    DRb.start_service
-  end
-
-  def kill
-    Process.kill("INT", @pid)
-    Process.wait    
-  end
-end
+class DrbServer
+  attr_reader :url
+
+  def initialize(url)
+    @url = url
+  end
+
+  def start
+    exe = File.join(File.dirname(__FILE__), "..", "bin", "server")
+    
+    @pid = Process.spawn "bundle exec ruby #{exe}", :err=>:out, :out => ".log"
+
+    while Pidfile.missing?
+      sleep 0.25
+    end
+
+    require 'drb/drb'
+
+    DRb.start_service
+  end
+
+  def kill
+    Process.kill("INT", @pid)
+    Process.wait    
+  end
+end

data/spec/{and_how_it_differs_from_observable_spec.rb → unit.tests/and_how_it_differs_from_observable_spec.rb}

@@ -1,101 +1,101 @@
-require "spec_helper"
-
-describe "Observable" do
-  let(:an_observable_object) do
-    Class.new do
-      require "observer"; include Observable
-
-      def poke
-        changed
-        notify_observers :poked
-      end
-
-      def tap
-        changed
-        notify_observers :tapped
-      end
-    end.new
-  end
-
-  it "you can register for notifications" do
-    an_observer = Class.new do
-      def update(args)
-        @notified = true
-      end
-      
-      def notified?; @notified === true; end
-    end.new
-    
-    an_observable_object.add_observer(an_observer)
-
-    an_observable_object.poke
-
-    an_observer.must be_notified
-  end
-
-  it "and you can send arguments with your notification" do
-    an_observer = Class.new do
-      def update(args)
-        @args = args
-      end
-      
-      def args; @args; end
-    end.new
-    
-    an_observable_object.add_observer(an_observer)
-
-    an_observable_object.poke
-
-    an_observer.args.must === :poked
-  end
-
-  it "for multiple events the observer must decide for themselves" do
-    an_observer = Class.new do
-      def initialize
-        @poked,@tapped = 0,0
-      end
-
-      def update(args)
-        @poked  = true if args === :poked
-        @tapped = true if args === :tapped
-      end
-      
-      def poked?; @poked === true; end
-      def tapped?; @tapped === true; end
-    end.new
-    
-    an_observable_object.add_observer(an_observer)
-
-    an_observable_object.poke
-    an_observable_object.tap
-
-    an_observer.must be_tapped
-    an_observer.must be_poked
-  end
-
-  it "notifications are not sent if `changed` not called" do
-    a_bung_observable_object_that_does_not_call_changed = Class.new do
-      require "observer"; include Observable
-
-      def poke
-        notify_observers :poked
-      end
-    end.new
-
-    an_observer = Class.new do
-      def update(args)
-        @notified = true
-      end
-      
-      def notified?; @notified === true; end
-    end.new
-
-    a_bung_observable_object_that_does_not_call_changed.add_observer an_observer
-
-    a_bung_observable_object_that_does_not_call_changed.poke
-
-    an_observer.must_not(be_notified, 
-      "Even though `notify_observers` was called, no notification was sent because you have to call `changed` beforehand."
-    )
-  end
-end
+require "spec_helper"
+
+describe "Observable" do
+  let(:an_observable_object) do
+    Class.new do
+      require "observer"; include Observable
+
+      def poke
+        changed
+        notify_observers :poked
+      end
+
+      def tap
+        changed
+        notify_observers :tapped
+      end
+    end.new
+  end
+
+  it "you can register for notifications" do
+    an_observer = Class.new do
+      def update(args)
+        @notified = true
+      end
+      
+      def notified?; @notified === true; end
+    end.new
+    
+    an_observable_object.add_observer(an_observer)
+
+    an_observable_object.poke
+
+    an_observer.must be_notified
+  end
+
+  it "and you can send arguments with your notification" do
+    an_observer = Class.new do
+      def update(args)
+        @args = args
+      end
+      
+      def args; @args; end
+    end.new
+    
+    an_observable_object.add_observer(an_observer)
+
+    an_observable_object.poke
+
+    an_observer.args.must === :poked
+  end
+
+  it "for multiple events the observer must decide for themselves" do
+    an_observer = Class.new do
+      def initialize
+        @poked,@tapped = 0,0
+      end
+
+      def update(args)
+        @poked  = true if args === :poked
+        @tapped = true if args === :tapped
+      end
+      
+      def poked?; @poked === true; end
+      def tapped?; @tapped === true; end
+    end.new
+    
+    an_observable_object.add_observer(an_observer)
+
+    an_observable_object.poke
+    an_observable_object.tap
+
+    an_observer.must be_tapped
+    an_observer.must be_poked
+  end
+
+  it "notifications are not sent if `changed` not called" do
+    a_bung_observable_object_that_does_not_call_changed = Class.new do
+      require "observer"; include Observable
+
+      def poke
+        notify_observers :poked
+      end
+    end.new
+
+    an_observer = Class.new do
+      def update(args)
+        @notified = true
+      end
+      
+      def notified?; @notified === true; end
+    end.new
+
+    a_bung_observable_object_that_does_not_call_changed.add_observer an_observer
+
+    a_bung_observable_object_that_does_not_call_changed.poke
+
+    an_observer.must_not(be_notified, 
+      "Even though `notify_observers` was called, no notification was sent because you have to call `changed` beforehand."
+    )
+  end
+end