signal 1.0.0 → 1.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0da1e84da778b581b1e7b325a1437ecc720f56ca
-  data.tar.gz: d1356905f81403f12aac8bebc3205ffb1c031f74
+  metadata.gz: 03e8ba417f4a98d92feeef3793b692fb4adec8eb
+  data.tar.gz: c0d5bdfa9dc3e51ee050009b8d4164ca0622c91f
 SHA512:
-  metadata.gz: 1dd9dc2736009aefe24b4aa0dea8d4febd50a3ab9c6b0a8b5eadb95c15e513007c8fedb0bf332809e229e38a0606059b9a8f2a44507f74a47bfabfcb788fbac1
-  data.tar.gz: 292aa7515e1e260860a298da7680235e5418462301042d260d44bc23b0016619eeec6654cc5c045a2ce2527b17b13a1dec9d6808b6f1c2fa67d889c7646057ce
+  metadata.gz: d1f56b758b92d0786a7808a75f7765fe8c612c8d26183627e5401a7eebfb02cbd78bb597c7249f42edd532e73283d3ca39fd8fbf626e010489d141aa72350f5a
+  data.tar.gz: f2219aa9e7d6551deceb844a3b302313e254126495cd369561e3f2f194a79acd5aadd487e8003143e35038b93783e94835d12fc14334baf865dc98a1806d7c9f

data/.travis.yml

@@ -1,11 +1,14 @@
 language: ruby
 cache: bundler
 sudo: false
-script: bundle exec rspec
+script: bundle exec rake
 rvm:
+- '2.3.0'
 - '2.2'
 - '2.1'
 addons:
   code_climate:
     repo_token:
       secure: "ASogQTLS/xYYczRtFwuo8DIkNdOl16mez4GHOw4RtdfTQ4BMROxE/CUemdPulMW1b/p7jsp5ghOtH2EJ4jN203s83HN4QjGC2wXhigkk6LzvxqjkmHxIWxJi2JS2sLq8nzOCcD0geO1pcbBFyuAzAdlwWa2g1WDP7O0L0fNjCtM="
+notifications:
+  email: false

data/Rakefile

@@ -1 +1,10 @@
-require 'bundler/gem_tasks'
+require "bundler/gem_tasks"
+require "rake/testtask"
+
+Rake::TestTask.new(:test) do |t|
+  t.libs << 'test'
+  t.test_files = FileList['test/**/*_test.rb']
+  t.warning = false
+end
+
+task :default => :test

data/examples/activerecord_model.rb

@@ -1,10 +1,10 @@
-$:.unshift File.expand_path('../../lib', __FILE__)
-require 'signal'
-require 'active_record'
+$:.unshift File.expand_path("../../lib", __FILE__)
+require "signal"
+require "active_record"
 
 ActiveRecord::Base.establish_connection({
-  :adapter => 'sqlite3',
-  :database => ':memory:'
+  :adapter => "sqlite3",
+  :database => ":memory:"
 })
 
 ActiveRecord::Schema.define(:version => 0) do
@@ -20,7 +20,7 @@ class Thing < ActiveRecord::Base
   validates_presence_of :name
 end
 
-thing = Thing.new(:name => 'Stuff')
+thing = Thing.new(:name => "Stuff")
 thing.on(:create) {|model| puts model.updated_at, model.name }
 thing.on(:update) {|model| puts model.updated_at, model.name }
 thing.on(:remove) {|model| puts model.destroyed? }
@@ -30,12 +30,12 @@ thing.save!
 #=> 2013-01-26 10:32:39 -0200
 #=> Stuff
 
-thing.update_attributes(:name => 'Updated stuff')
+thing.update_attributes(:name => "Updated stuff")
 #=> 2013-01-26 10:33:11 -0200
 #=> Updated stuff
 
 thing.update_attributes(:name => nil)
-#=> ['Name can't be blank']
+#=> ["Name can"t be blank"]
 
 thing.destroy
 #=> true

data/examples/arguments.rb

@@ -1,5 +1,5 @@
-$:.unshift File.expand_path('../../lib', __FILE__)
-require 'signal'
+$:.unshift File.expand_path("../../lib", __FILE__)
+require "signal"
 
 class Arguments
   include Signal

data/examples/block_context.rb

@@ -1,5 +1,5 @@
-$:.unshift File.expand_path('../../lib', __FILE__)
-require 'signal'
+$:.unshift File.expand_path("../../lib", __FILE__)
+require "signal"
 
 class Contact
   include Signal
@@ -15,6 +15,6 @@ class Contact
   end
 end
 
-contact = Contact.new('John Doe', 'john@example.org')
+contact = Contact.new("John Doe", "john@example.org")
 contact.on(:output) {|contact| puts contact.name, contact.email }
 contact.output!

data/examples/blocks.rb

@@ -1,5 +1,5 @@
-$:.unshift File.expand_path('../../lib', __FILE__)
-require 'signal'
+$:.unshift File.expand_path("../../lib", __FILE__)
+require "signal"
 
 class Status
   include Signal
@@ -10,7 +10,7 @@ class Status
 end
 
 status = Status.new
-status.before(:ready) { puts 'Before the ready event!' }
+status.before(:ready) { puts "Before the ready event!" }
 status.on(:ready) { puts "I'm ready!" }
-status.after(:ready) { puts 'After the ready event!' }
+status.after(:ready) { puts "After the ready event!" }
 status.ready!

data/examples/call.rb

@@ -0,0 +1,20 @@
+$:.unshift File.expand_path("../../lib", __FILE__)
+require "signal"
+
+class Contact
+  include Signal.call
+
+  attr_reader :name, :email
+
+  def initialize(name, email)
+    @name, @email = name, email
+  end
+
+  def call
+    emit(:output, self)
+  end
+end
+
+Contact.call("John", "john@example.com") do |o|
+  o.on(:output) {|contact| puts contact.name }
+end

data/examples/chain.rb

@@ -1,10 +1,10 @@
-$:.unshift File.expand_path('../../lib', __FILE__)
-require 'signal'
-require 'active_record'
+$:.unshift File.expand_path("../../lib", __FILE__)
+require "signal"
+require "active_record"
 
 ActiveRecord::Base.establish_connection({
-  :adapter => 'sqlite3',
-  :database => ':memory:'
+  :adapter => "sqlite3",
+  :database => ":memory:"
 })
 
 ActiveRecord::Schema.define(:version => 0) do
@@ -15,7 +15,7 @@ ActiveRecord::Schema.define(:version => 0) do
 end
 
 class Thing < ActiveRecord::Base
-  include Signal::ActiveRecord
+  include Signal.active_record
 
   validates_presence_of :name
 end
@@ -31,7 +31,7 @@ class MyListener
 end
 
 puts "\n=== Creating valid record"
-thing = Thing.new(:name => 'Stuff')
+thing = Thing.new(:name => "Stuff")
 thing.listeners << MyListener.new
 thing.save
 
@@ -41,16 +41,16 @@ thing.listeners << MyListener.new
 thing.save
 
 puts "\n=== Updating valid record"
-thing = Thing.create(:name => 'Stuff')
+thing = Thing.create(:name => "Stuff")
 thing.listeners << MyListener.new
-thing.update_attributes(:name => 'Updated stuff')
+thing.update_attributes(:name => "Updated stuff")
 
 puts "\n=== Updating invalid record"
-thing = Thing.create!(:name => 'Stuff')
+thing = Thing.create!(:name => "Stuff")
 thing.listeners << MyListener.new
 thing.update_attributes(:name => nil)
 
 puts "\n=== Removing record"
-thing = Thing.create(:name => 'Stuff')
+thing = Thing.create(:name => "Stuff")
 thing.listeners << MyListener.new
 thing.destroy

data/examples/listener.rb

@@ -1,5 +1,5 @@
-$:.unshift File.expand_path('../../lib', __FILE__)
-require 'signal'
+$:.unshift File.expand_path("../../lib", __FILE__)
+require "signal"
 
 class Status
   include Signal
@@ -11,7 +11,7 @@ end
 
 class MyListener
   def before_ready
-    puts 'Before the ready event!'
+    puts "Before the ready event!"
   end
 
   def on_ready
@@ -19,7 +19,7 @@ class MyListener
   end
 
   def after_ready
-    puts 'After the ready event!'
+    puts "After the ready event!"
   end
 end
 

data/lib/signal.rb

@@ -1,7 +1,7 @@
-require 'signal/version'
-require 'signal/listener'
-require 'signal/extensions/active_record'
-require 'signal/extensions/call'
+require "signal/version"
+require "signal/listener"
+require "signal/extensions/active_record"
+require "signal/extensions/call"
 
 module Signal
   def on(event, &block)

data/lib/signal/extensions/call.rb

@@ -11,9 +11,9 @@ module Signal
       end
 
       module ClassMethods
-        def call(*args, &block)
+        def call(*args)
           new(*args).tap do |instance|
-            yield(instance)
+            yield(instance) if block_given?
             instance.call
           end
         end

data/lib/signal/version.rb

@@ -1,3 +1,3 @@
 module Signal
-  VERSION = '1.0.0'
+  VERSION = "1.1.0"
 end

data/signal.gemspec

@@ -1,21 +1,22 @@
-require './lib/signal/version'
+require "./lib/signal/version"
 
 Gem::Specification.new do |gem|
-  gem.name          = 'signal'
+  gem.name          = "signal"
   gem.version       = Signal::VERSION
-  gem.authors       = ['Nando Vieira']
-  gem.email         = ['fnando.vieira@gmail.com']
-  gem.description   = 'A simple observer implementation for POROs (Plain Old Ruby Object) and ActiveRecord objects.'
+  gem.authors       = ["Nando Vieira"]
+  gem.email         = ["fnando.vieira@gmail.com"]
+  gem.description   = "A simple observer implementation for POROs (Plain Old Ruby Object) and ActiveRecord objects."
   gem.summary       = gem.description
-  gem.homepage      = 'http://github.com/fnando/signal'
+  gem.homepage      = "http://github.com/fnando/signal"
 
   gem.files         = `git ls-files`.split($/)
-  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
+  gem.executables   = gem.files.grep(%r{^bin/}).map {|f| File.basename(f) }
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
-  gem.require_paths = ['lib']
+  gem.require_paths = ["lib"]
 
-  gem.add_development_dependency 'activerecord'
-  gem.add_development_dependency 'sqlite3'
-  gem.add_development_dependency 'rspec'
-  gem.add_development_dependency 'codeclimate-test-reporter'
+  gem.add_development_dependency "activerecord"
+  gem.add_development_dependency "sqlite3"
+  gem.add_development_dependency "minitest-utils"
+  gem.add_development_dependency "codeclimate-test-reporter"
+  gem.add_development_dependency "rake"
 end

data/test/signal/activerecord_test.rb

@@ -0,0 +1,169 @@
+require "test_helper"
+
+class ARCreateEventTest < Minitest::Test
+  let(:user) { User.new(username: "johndoe") }
+
+  test "triggers before event" do
+    callable = Callable.new(:before_create)
+    user.before(:create, &callable)
+    callable.expect(:before_create, nil, [user])
+    user.save!
+
+    assert callable.verify
+  end
+
+  test "triggers event" do
+    callable = Callable.new(:on_create)
+    user.on(:create, &callable)
+    callable.expect(:on_create, nil, [user])
+    user.save!
+
+    assert callable.verify
+  end
+
+  test "triggers after event" do
+    callable = Callable.new(:after_create)
+    user.after(:create, &callable)
+    callable.expect(:after_create, nil, [user])
+    user.save!
+
+    assert callable.verify
+  end
+
+  test "does not trigger events when record is invalid" do
+    user = User.new
+    calls = []
+
+    user
+      .before(:create) { calls << :before }
+      .on(:create) { calls << :on }
+      .after(:create) { calls << :after }
+
+    user.save
+
+    assert_equal [], calls
+  end
+end
+
+class ARValidationEventTest < Minitest::Test
+  let(:user) { User.new(username: "johndoe") }
+
+  test "triggers before event" do
+    callable = Callable.new(:before_validation)
+    user.before(:validation, &callable)
+    callable.expect(:before_validation, nil, [user])
+    user.save!
+
+    assert callable.verify
+  end
+
+  test "triggers after event" do
+    callable = Callable.new(:after_validation)
+    user.after(:validation, &callable)
+    callable.expect(:after_validation, nil, [user])
+    user.save!
+
+    assert callable.verify
+  end
+
+  test "triggers validation event when record is invalid" do
+    callable = Callable.new(:on_validation)
+    user.username = nil
+    user.on(:validation, &callable)
+
+    callable.expect(:on_validation, nil, [user])
+    user.save
+
+    assert callable.verify
+  end
+
+  test "skips validation event when record is valid" do
+    calls = []
+    user.on(:validation) { calls << :on }
+    user.save!
+
+    assert_equal [], calls
+  end
+end
+
+class ARUpdateEvent < Minitest::Test
+  let(:user) { User.create!(username: "johndoe") }
+
+  test "triggers before event" do
+    callable = Callable.new(:before_update)
+    user.before(:update, &callable)
+
+    callable.expect(:before_update, nil, [user])
+    user.update_attributes!(username: "johnd")
+
+    assert callable.verify
+  end
+
+  test "triggers event" do
+    callable = Callable.new(:on_update)
+    user.on(:update, &callable)
+
+    callable.expect(:on_update, nil, [user])
+    user.update_attributes!(username: "johnd")
+
+    assert callable.verify
+  end
+
+  test "triggers after event" do
+    callable = Callable.new(:after_update)
+    user.after(:update, &callable)
+
+    callable.expect(:after_update, nil, [user])
+    user.update_attributes!(username: "johnd")
+
+    assert callable.verify
+  end
+
+  test "does not trigger events when record is invalid" do
+    user.username = nil
+    calls = []
+
+    user
+      .before(:update) { calls << :before }
+      .on(:update) { calls << :on }
+      .after(:update) { calls << :after }
+
+    user.save
+
+    assert_equal [], calls
+  end
+end
+
+class ARRemoveEvent < Minitest::Test
+  let(:user) { User.create!(username: "johndoe") }
+
+  test "triggers before event" do
+    callable = Callable.new(:before_remove)
+    user.before(:remove, &callable)
+
+    callable.expect(:before_remove, nil, [user])
+    user.destroy
+
+    assert callable.verify
+  end
+
+  test "triggers event" do
+    callable = Callable.new(:on_remove)
+    user.on(:remove, &callable)
+
+    callable.expect(:on_remove, nil, [user])
+    user.destroy
+
+    assert callable.verify
+  end
+
+  test "triggers after event" do
+    callable = Callable.new(:after_remove)
+    user.after(:remove, &callable)
+
+    callable.expect(:after_remove, nil, [user])
+    user.destroy
+
+    assert callable.verify
+  end
+end

data/test/signal/call_test.rb

@@ -0,0 +1,19 @@
+require "test_helper"
+
+class SignalCallTest < Minitest::Test
+  test "initializes observable with arguments" do
+    observable = ObservableWithCall.call(1, 2, 3)
+    assert_equal [1, 2, 3], observable.args
+  end
+
+  test "triggers event" do
+    callable = Callable.new(:on_args)
+    callable.expect(:on_args, nil, [[1, 2, 3]])
+
+    ObservableWithCall.call(1, 2, 3) do |o|
+      o.on(:args, &callable)
+    end
+
+    assert callable.verify
+  end
+end

data/test/signal/issues_test.rb

@@ -0,0 +1,15 @@
+require "test_helper"
+
+# https://github.com/fnando/signal/issues/2
+class ConstantLookupTest < Minitest::Test
+  let(:callable) { Callable.new(:on_success) }
+
+  test "triggers event" do
+    emitter = Emitter.new
+    emitter.on(:success, &callable)
+    callable.expect(:on_success, nil, [])
+    emitter.call
+
+    assert callable.verify
+  end
+end

data/test/signal/listener_test.rb

@@ -0,0 +1,8 @@
+require "test_helper"
+
+class SignalListenerTest < Minitest::Test
+  test "sets string representation" do
+    listener = Signal::Listener.new(self, :on, :ready, &->{})
+    assert_equal "<Signal::Listener event: on_ready>", listener.to_s
+  end
+end

data/test/signal/signal_test.rb

@@ -0,0 +1,146 @@
+require "test_helper"
+
+class AddListenerTest < Minitest::Test
+  let(:observable) { Observable.new }
+  let(:callable) { Callable.new(:on_ready) }
+
+  test "runs listener" do
+    callable.expect(:on_ready, nil, [])
+    observable.add_listener(callable)
+
+    observable.emit(:ready)
+
+    assert callable.verify
+  end
+
+  test "returns itself" do
+    assert_equal observable, observable.add_listener(callable)
+  end
+end
+
+class UsingBlocksTest < Minitest::Test
+  let(:observable) { Observable.new }
+  let(:callable) { Callable.new(:on_ready) }
+
+  test "triggers event" do
+    callable.expect(:on_ready, nil, [])
+    observable.on(:ready, &callable)
+
+    observable.emit(:ready)
+
+    assert callable.verify
+  end
+
+  test "triggers event with arguments" do
+    callable.expect(:on_ready, nil, [1, 2, 3])
+    observable.on(:ready, &callable)
+
+    observable.emit(:ready, 1, 2, 3)
+
+    assert callable.verify
+  end
+
+  test "triggers before event" do
+    callable = Callable.new(:before_ready)
+    callable.expect(:before_ready, nil, [])
+    observable.before(:ready, &callable)
+
+    observable.emit(:ready)
+
+    assert callable.verify
+  end
+
+  test "triggers before event with arguments" do
+    callable = Callable.new(:before_ready)
+    callable.expect(:before_ready, nil, [1, 2, 3])
+    observable.before(:ready, &callable)
+
+    observable.emit(:ready, 1, 2, 3)
+
+    assert callable.verify
+  end
+
+  test "triggers after event" do
+    callable = Callable.new(:after_ready)
+    callable.expect(:after_ready, nil, [])
+    observable.after(:ready, &callable)
+
+    observable.emit(:ready)
+
+    assert callable.verify
+  end
+
+  test "triggers after event with arguments" do
+    callable = Callable.new(:after_ready)
+    callable.expect(:after_ready, nil, [1, 2, 3])
+    observable.after(:ready, &callable)
+
+    observable.emit(:ready, 1, 2, 3)
+
+    assert callable.verify
+  end
+
+  test "chains events" do
+    calls = []
+
+    observable
+      .before(:ready) { calls << :before }
+      .on(:ready) { calls << :on }
+      .after(:ready) { calls << :after }
+
+    observable.emit(:ready)
+
+    assert_equal %i[before on after], calls
+  end
+
+  test "keeps context" do
+    context = nil
+    callable = -> { context = self }
+    observable.on(:ready, &callable)
+    observable.emit(:ready)
+
+    assert_equal self, context
+  end
+end
+
+class UsingListenersTest < Minitest::Test
+  let(:observable) { Observable.new }
+  let(:callable) { Callable.new(:on_ready) }
+
+  test "triggers event for listener" do
+    observable.listeners << callable
+    callable.expect(:after_ready, nil, [])
+
+    observable.emit(:ready)
+
+    assert callable.verify
+  end
+
+  test "triggers event for listener with arguments" do
+    observable.listeners << callable
+    callable.expect(:after_ready, nil, [1, 2, 3])
+
+    observable.emit(:ready, 1, 2, 3)
+
+    assert callable.verify
+  end
+end
+
+class MixedListenersTest < Minitest::Test
+  test "calls all listeners" do
+    observable = Observable.new
+    callable = Callable.new(:on_ready)
+    another_callable = Callable.new(:on_ready)
+
+    callable.expect(:on_ready, nil, [])
+    another_callable.expect(:on_ready, nil, [])
+
+    observable
+      .add_listener(callable)
+      .on(:ready, &another_callable)
+      .emit(:ready)
+
+    assert callable.verify
+    assert another_callable.verify
+  end
+end

data/test/support/callable.rb

@@ -0,0 +1,11 @@
+class Callable < Minitest::Mock
+  def initialize(method_name)
+    super()
+    @method_name = method_name
+  end
+
+  def to_proc
+    callable = self
+    proc {|*args| callable.send(@method_name, *args) }
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,28 @@
+require "codeclimate-test-reporter"
+CodeClimate::TestReporter.start
+
+require "bundler/setup"
+require "active_record"
+
+I18n.enforce_available_locales = false
+
+require "minitest/utils"
+require "minitest/autorun"
+
+require "signal"
+require "support/observable"
+require "support/observable_with_call"
+require "support/user"
+require "support/callable"
+require "support/emitter"
+
+ActiveRecord::Base.establish_connection({
+  :adapter => "sqlite3",
+  :database => ":memory:"
+})
+
+ActiveRecord::Schema.define(:version => 0) do
+  create_table :users do |t|
+    t.string :username
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: signal
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Nando Vieira
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-21 00:00:00.000000000 Z
+date: 2016-10-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -39,7 +39,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: minitest-utils
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -66,6 +66,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: A simple observer implementation for POROs (Plain Old Ruby Object) and
   ActiveRecord objects.
 email:
@@ -75,7 +89,6 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
-- ".rspec"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
@@ -86,6 +99,7 @@ files:
 - examples/arguments.rb
 - examples/block_context.rb
 - examples/blocks.rb
+- examples/call.rb
 - examples/chain.rb
 - examples/listener.rb
 - lib/signal.rb
@@ -94,17 +108,17 @@ files:
 - lib/signal/listener.rb
 - lib/signal/version.rb
 - signal.gemspec
-- spec/signal/activerecord_spec.rb
-- spec/signal/call_spec.rb
-- spec/signal/issues/2_spec.rb
-- spec/signal/listener_spec.rb
-- spec/signal/signal_spec.rb
-- spec/spec_helper.rb
-- spec/support/callable.rb
-- spec/support/emitter.rb
-- spec/support/observable.rb
-- spec/support/observable_with_call.rb
-- spec/support/user.rb
+- test/signal/activerecord_test.rb
+- test/signal/call_test.rb
+- test/signal/issues_test.rb
+- test/signal/listener_test.rb
+- test/signal/signal_test.rb
+- test/support/callable.rb
+- test/support/emitter.rb
+- test/support/observable.rb
+- test/support/observable_with_call.rb
+- test/support/user.rb
+- test/test_helper.rb
 homepage: http://github.com/fnando/signal
 licenses: []
 metadata: {}
@@ -124,20 +138,20 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5.1
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: A simple observer implementation for POROs (Plain Old Ruby Object) and ActiveRecord
   objects.
 test_files:
-- spec/signal/activerecord_spec.rb
-- spec/signal/call_spec.rb
-- spec/signal/issues/2_spec.rb
-- spec/signal/listener_spec.rb
-- spec/signal/signal_spec.rb
-- spec/spec_helper.rb
-- spec/support/callable.rb
-- spec/support/emitter.rb
-- spec/support/observable.rb
-- spec/support/observable_with_call.rb
-- spec/support/user.rb
+- test/signal/activerecord_test.rb
+- test/signal/call_test.rb
+- test/signal/issues_test.rb
+- test/signal/listener_test.rb
+- test/signal/signal_test.rb
+- test/support/callable.rb
+- test/support/emitter.rb
+- test/support/observable.rb
+- test/support/observable_with_call.rb
+- test/support/user.rb
+- test/test_helper.rb

data/.rspec

@@ -1 +0,0 @@
---color

data/spec/signal/activerecord_spec.rb

@@ -1,132 +0,0 @@
-require 'spec_helper'
-
-describe Signal::Extensions::ActiveRecord do
-  let(:callable) { Callable.new }
-  let(:user) { User.new(:username => 'johndoe') }
-
-  context 'create event' do
-    it 'triggers before event' do
-      user.before(:create, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.save!
-    end
-
-    it 'triggers event' do
-      user.on(:create, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.save!
-    end
-
-    it 'triggers after event' do
-      user.after(:create, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.save!
-    end
-
-    it 'does not trigger on/after events when record is invalid' do
-      user = User.new
-
-      on_callable = Callable.new
-      after_callable = Callable.new
-
-      user
-        .on(:create, &on_callable)
-        .after(:create, &after_callable)
-
-      expect(on_callable).not_to receive(:called)
-      expect(after_callable).not_to receive(:called)
-
-      user.save
-    end
-  end
-
-  context 'validation event' do
-    it 'triggers before event' do
-      user.before(:validation, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.save!
-    end
-
-    it 'triggers after event' do
-      user.after(:validation, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.save!
-    end
-
-    it 'triggers validation event when record is invalid' do
-      user.username = nil
-      user.on(:validation, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.save
-    end
-
-    it 'skips validation event when record is valid' do
-      user.on(:validation, &callable)
-      expect(callable).not_to receive(:called)
-      user.save!
-    end
-  end
-
-  context 'update event' do
-    let(:user) { User.create!(:username => 'johndoe') }
-
-    it 'triggers before event' do
-      user.before(:update, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.update_attributes!(:username => 'johnd')
-    end
-
-    it 'triggers on event' do
-      user.on(:update, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.update_attributes!(:username => 'johnd')
-    end
-
-    it 'triggers after event' do
-      user.after(:update, &callable)
-      expect(callable).to receive(:called).with(user)
-      user.update_attributes!(:username => 'johnd')
-    end
-
-    it 'does not trigger on/after events when record is invalid' do
-      user.username = nil
-
-      on_callable = Callable.new
-      after_callable = Callable.new
-
-      user
-        .on(:update, &on_callable)
-        .after(:update, &after_callable)
-
-      expect(on_callable).not_to receive(:called)
-      expect(after_callable).not_to receive(:called)
-
-      user.save
-    end
-  end
-
-  context 'remove event' do
-    let(:user) { User.create!(:username => 'johndoe') }
-
-    it 'triggers before event' do
-      user.before(:remove, &callable)
-      expect(callable).to receive(:called).with(user)
-
-      user.destroy
-    end
-
-    it 'triggers on event' do
-      user.on(:remove, &callable)
-      expect(callable).to receive(:called).with(user)
-
-      user.destroy
-    end
-
-    it 'triggers after event' do
-      user.after(:remove, &callable)
-      expect(callable).to receive(:called).with(user)
-
-      user.destroy
-    end
-  end
-end

data/spec/signal/call_spec.rb

@@ -1,18 +0,0 @@
-require 'spec_helper'
-
-describe Signal::Extensions::Call do
-  let(:callable) { Callable.new }
-
-  it 'initializes observable with arguments' do
-    observable = ObservableWithCall.call(1, 2, 3) {}
-    expect(observable.args).to eq([1, 2, 3])
-  end
-
-  it 'triggers event' do
-    expect(callable).to receive(:called).with([1, 2, 3])
-
-    ObservableWithCall.call(1, 2, 3) do |o|
-      o.on(:args, &callable)
-    end
-  end
-end

data/spec/signal/issues/2_spec.rb

@@ -1,13 +0,0 @@
-require 'spec_helper'
-
-describe 'Constant lookup', 'https://github.com/fnando/signal/issues/2' do
-  let(:callable) { Callable.new }
-
-  it 'triggers event' do
-    emitter = Emitter.new
-    emitter.on(:success, &callable)
-    expect(callable).to receive(:called).with(no_args)
-
-    emitter.call
-  end
-end

data/spec/signal/listener_spec.rb

@@ -1,8 +0,0 @@
-require 'spec_helper'
-
-describe Signal::Listener do
-  it 'sets string representation' do
-    listener = Signal::Listener.new(self, :on, :ready, &->{})
-    expect(listener.to_s).to eq('<Signal::Listener event: on_ready>')
-  end
-end

data/spec/signal/signal_spec.rb

@@ -1,123 +0,0 @@
-require 'spec_helper'
-
-describe Signal do
-  let(:observable) { Observable.new }
-  let(:callable) { Callable.new }
-
-  context 'add listener' do
-    it 'runs listener' do
-      callable.respond_to(:on_ready)
-      observable.add_listener(callable)
-      expect(callable).to receive(:called).with(no_args)
-
-      observable.emit(:ready)
-    end
-
-    it 'returns itself' do
-      expect(observable.add_listener(callable)).to eq(observable)
-    end
-  end
-
-  context 'using blocks' do
-    it 'triggers event' do
-      observable.on(:ready, &callable)
-      expect(callable).to receive(:called).with(no_args)
-
-      observable.emit(:ready)
-    end
-
-    it 'triggers event with arguments' do
-      observable.on(:ready, &callable)
-      expect(callable).to receive(:called).with(1, 2, 3)
-
-      observable.emit(:ready, 1, 2, 3)
-    end
-
-    it 'triggers before event' do
-      observable.before(:ready, &callable)
-      expect(callable).to receive(:called).with(no_args)
-
-      observable.emit(:ready)
-    end
-
-    it 'triggers before event with arguments' do
-      observable.before(:ready, &callable)
-      expect(callable).to receive(:called).with(1, 2, 3)
-
-      observable.emit(:ready, 1, 2, 3)
-    end
-
-    it 'triggers after event' do
-      observable.after(:ready, &callable)
-      expect(callable).to receive(:called).with(no_args)
-
-      observable.emit(:ready)
-    end
-
-    it 'triggers after event with arguments' do
-      observable.after(:ready, &callable)
-      expect(callable).to receive(:called).with(1, 2, 3)
-
-      observable.emit(:ready, 1, 2, 3)
-    end
-
-    it 'chains events' do
-      before_callable = Callable.new
-      on_callable = Callable.new
-      after_callable = Callable.new
-
-      observable
-        .before(:ready, &before_callable)
-        .on(:ready, &on_callable)
-        .after(:ready, &after_callable)
-
-      expect(before_callable).to receive(:called).with(no_args).ordered
-      expect(on_callable).to receive(:called).with(no_args).ordered
-      expect(after_callable).to receive(:called).with(no_args).ordered
-
-      observable.emit(:ready)
-    end
-
-    it 'keeps context' do
-      context = nil
-      callable = -> { context = self }
-      observable.on(:ready, &callable)
-      observable.emit(:ready)
-
-      expect(context).to eql(self)
-    end
-  end
-
-  context 'using listeners' do
-    it 'triggers event for listener' do
-      callable.respond_to(:on_ready)
-      observable.listeners << callable
-      expect(callable).to receive(:called).with(no_args)
-
-      observable.emit(:ready)
-    end
-
-    it 'triggers event for listener with arguments' do
-      callable.respond_to(:on_ready)
-      observable.listeners << callable
-      expect(callable).to receive(:called).with(1, 2, 3)
-
-      observable.emit(:ready, 1, 2, 3)
-    end
-  end
-
-  context 'mixed listeners' do
-    it 'calls all listeners' do
-      callable.respond_to(:on_ready)
-      another_callable = Callable.new
-
-      expect(callable).to receive(:called).ordered
-      expect(another_callable).to receive(:called).ordered
-
-      observable
-        .add_listener(callable)
-        .on(:ready, &another_callable)
-        .emit(:ready)
-    end
-  end
-end

data/spec/spec_helper.rb

@@ -1,25 +0,0 @@
-require 'codeclimate-test-reporter'
-CodeClimate::TestReporter.start
-
-require 'bundler/setup'
-require 'active_record'
-
-I18n.enforce_available_locales = false
-
-require 'signal'
-require 'support/observable'
-require 'support/observable_with_call'
-require 'support/callable'
-require 'support/user'
-require 'support/emitter'
-
-ActiveRecord::Base.establish_connection({
-  :adapter => 'sqlite3',
-  :database => ':memory:'
-})
-
-ActiveRecord::Schema.define(:version => 0) do
-  create_table :users do |t|
-    t.string :username
-  end
-end

data/spec/support/callable.rb

@@ -1,10 +0,0 @@
-class Callable
-  def to_proc
-    callable = self
-    proc {|*args| callable.called(*args) }
-  end
-
-  def respond_to(method_name)
-    define_singleton_method(method_name, &to_proc)
-  end
-end