msgr 0.0.0 → 0.1.0

data/spec/msgr/msgr/connection_spec.rb

@@ -0,0 +1,51 @@
+require 'spec_helper'
+
+describe Msgr::Connection do
+
+  describe '#rebind' do
+    let(:conn) { double }
+    let(:routes) { Msgr::Routes.new }
+    let(:connection) { Msgr::Connection.new conn, routes, dispatcher }
+
+    context 'with URI' do
+      it 'should pass URI options to bunny (I)' do
+        expect(Bunny).to receive(:new)
+                         .with(pass: 'guest', user: 'guest', ssl: false, host: 'localhost', vhost: '/')
+
+        Msgr::Client.new uri: 'amqp://guest:guest@localhost/'
+      end
+
+      it 'should pass URI options to bunny (II)' do
+        expect(Bunny).to receive(:new)
+                         .with(pass: 'msgr', user: 'abc', ssl: true, host: 'bogus.example.org', vhost: '/rabbit')
+
+        Msgr::Client.new uri: 'amqps://abc:msgr@bogus.example.org/rabbit'
+      end
+    end
+
+    context 'with options' do
+      it 'should pass options to bunny' do
+        expect(Bunny).to receive(:new)
+                         .with(pass: 'guest', user: 'guest', ssl: false, host: 'localhost', vhost: '/')
+
+        Msgr::Client.new pass: 'guest', user: 'guest', ssl: false, host: 'localhost', vhost: '/'
+      end
+    end
+
+    context 'with URI and options' do
+      it 'should pass merged options to bunny' do
+        expect(Bunny).to receive(:new)
+                         .with(pass: 'msgr', user: 'abc', ssl: false, host: 'localhost', vhost: '/joghurt')
+
+        Msgr::Client.new uri: 'ampq://abc@localhost', pass: 'msgr', vhost: '/joghurt'
+      end
+
+      it 'should pass prefer hash option' do
+        expect(Bunny).to receive(:new)
+                         .with(ssl: true, host: 'a.example.org', vhost: '/', user: 'guest')
+
+        Msgr::Client.new uri: 'ampq://localhost', ssl: true, host: 'a.example.org'
+      end
+    end
+  end
+end

data/spec/msgr/msgr/pool_spec.rb

@@ -0,0 +1,68 @@
+require 'spec_helper'
+
+$shutdown_test_graceful_down = false
+
+class Runner
+  def call(*_) end
+
+  def shutdown_test
+    sleep 2
+    $shutdown_test_graceful_down = true
+  end
+end
+
+describe Msgr::Pool do
+  let(:pool) { Msgr::Pool.new Runner }
+
+  describe '#initialize' do
+    let(:opts) { {} }
+    let(:pool) { Msgr::Pool.new Runner, opts }
+
+    context 'pool size' do
+      let(:opts) { {size: 4} }
+      before { pool }
+
+      it 'should define number of worker actors' do
+        expect(pool.size).to eq 4
+      end
+    end
+  end
+
+  describe '#start' do
+    let!(:pool) { Msgr::Pool.new Runner, size: 4 }
+
+    it 'should start worker actors' do
+      expect { pool.start }.to change { Celluloid::Actor.all.size }.by(4)
+    end
+  end
+
+  describe '#size' do
+    it 'should default to number of available cores' do
+      expect(pool.size).to eq Celluloid.cores
+    end
+  end
+
+  describe '#dispatch' do
+    let!(:pool) { Msgr::Pool.new Runner, size: 4 }
+    before { pool.start }
+
+    it 'should dispatch message to runner' do
+      expect_any_instance_of(Runner).to receive(:call).within(10).seconds.once
+      pool.dispatch 5, 3.2, 'hello'
+    end
+  end
+
+  describe '#shutdown' do
+    let!(:pool) { Msgr::Pool.new Runner, size: 1 }
+    before do
+      pool.start
+      $shutdown_test_graceful_down = false
+    end
+
+    it 'should do a graceful shutdown of all worker' do
+      pool.dispatch :shutdown_test
+      pool.shutdown
+      expect($shutdown_test_graceful_down).to be_false
+    end
+  end
+end

data/spec/msgr/msgr/routes_spec.rb

@@ -0,0 +1,113 @@
+require 'spec_helper'
+
+describe Msgr::Routes do
+  let(:routes) { Msgr::Routes.new }
+
+  describe '#configure' do
+    let(:block) { Proc.new{} }
+
+    it 'should evaluate given block within instance context' do
+      expect(routes).to receive(:instance_eval).with { |p| p == block }
+
+      routes.configure &block
+    end
+
+    it 'should allow to call instance method in gven block' do
+      expect(routes).to receive(:test_instance_method).with(:abc)
+
+      routes.configure do
+        test_instance_method :abc
+      end
+    end
+  end
+
+  describe '#each' do
+    before do
+      routes.configure do
+        route 'abc.#', to: 'test#index'
+        route 'edf.#', to: 'test#action'
+      end
+    end
+
+    let(:each) { routes.each }
+
+    it 'should iterate over configured routes' do
+      expect(each).to have(2).items
+
+      expect(each.map(&:keys)).to eq [%w(abc.#), %w(edf.#)]
+      expect(each.map(&:consumer)).to eq %w(TestConsumer TestConsumer)
+      expect(each.map(&:action)).to eq %w(index action)
+    end
+  end
+
+  describe '#route' do
+    let(:subject) { -> { routes.route 'routing.key', to: 'test2#index2' } }
+    let(:last_route) { routes.routes.last }
+
+    it 'should add a new route' do
+      expect { subject.call }.to change{ routes.routes.size }.from(0).to(1)
+    end
+
+    it 'should add given route' do
+      subject.call
+
+      expect(last_route.keys).to eq %w(routing.key)
+      expect(last_route.consumer).to eq 'Test2Consumer'
+      expect(last_route.action).to eq 'index2'
+    end
+
+    context 'with same target' do
+      let(:subject) do
+        -> do
+          routes.route 'routing.key', to: 'test#index'
+          routes.route 'another.routing.key', to: 'test#index'
+        end
+      end
+
+      it 'should only add one new route' do
+        expect { subject.call }.to change{ routes.routes.size }.from(0).to(1)
+      end
+
+      it 'should add second binding to first route' do
+        subject.call
+        expect(routes.routes.first.keys).to have(2).items
+        expect(routes.routes.first.keys).to eq %w(routing.key another.routing.key)
+      end
+    end
+  end
+
+  describe '#files' do
+    it 'should allow to add route paths' do
+      routes.files << 'abc.rb'
+      routes.files += %w(cde.rb edf.rb)
+
+      expect(routes.files).to eq %w(abc.rb cde.rb edf.rb)
+    end
+  end
+
+  describe 'reload' do
+    before { File.stub(:exists?).and_return(true) }
+
+    it 'should trigger load for all files' do
+      expect(routes).to receive(:load).with('cde.rb').ordered
+      expect(routes).to receive(:load).with('edf.rb').ordered
+      routes.files += %w(cde.rb edf.rb)
+      routes.reload
+    end
+
+    it 'should clear old routes before reloading' do
+      routes.route 'abc', to: 'abc#test'
+      routes.reload
+      expect(routes.each).to have(0).items
+    end
+  end
+
+  describe 'load' do
+    let(:file) { 'spec/fixtures/msgr-routes-test-1.rb' }
+
+    it 'should eval given file within routes context' do
+      expect(routes).to receive(:route).with('abc.#', to: 'test#index')
+      routes.load file
+    end
+  end
+end

data/spec/msgr/msgr_spec.rb

@@ -0,0 +1,36 @@
+require 'spec_helper'
+
+class TestConsumer < Msgr::Consumer
+  def index
+    puts "<<< #{payload}"
+  end
+end
+
+describe Msgr do
+  before do
+    Msgr.logger = nil;
+    Msgr.logger.level = Logger::Severity::DEBUG if Msgr.logger
+  end
+
+  let(:client) { Msgr::Client.new size: 1, prefix: SecureRandom.hex(32) }
+
+  before do
+    client.routes.configure do
+      route '#', to: 'test#index'
+    end
+
+    client.start
+  end
+
+  after do
+    client.stop timeout: 10, delete: true, wait_empty: 10
+  end
+
+  it 'should dispatch published methods to consumer' do
+    expect_any_instance_of(TestConsumer).to receive(:index).within(10).seconds.and_call_original
+
+    client.publish 'routing.key', 'Payload'
+
+    sleep 10
+  end
+end

data/spec/{spec_helper.rb → msgr/spec_helper.rb}

@@ -1,3 +1,7 @@
+# Bundler setup
+require 'bundler'
+Bundler.setup :default, :test
+
 # Coverage
 require 'coveralls'
 Coveralls.wear! do
@@ -5,6 +9,7 @@ Coveralls.wear! do
 end
 
 require 'rspec/autorun'
+require 'rspec/message/within'
 require 'msgr'
 
 Dir[File.expand_path('../support/**/*.rb', __FILE__)].each { |f| require f }
@@ -15,4 +20,14 @@ RSpec.configure do |config|
     # Only allow expect syntax
     c.syntax = :expect
   end
+
+  config.before do
+    Celluloid.logger = nil
+
+    Celluloid.shutdown
+    Celluloid.boot
+
+    Celluloid.logger = nil
+    Msgr.logger = false
+  end
 end

data/spec/support/setup.rb

@@ -0,0 +1,27 @@
+require 'bundler'
+
+# Somewhere between
+#   `ruby -w -W2 -S rspec ...`
+# and the rspec executable `bundle/setup` is required or
+# `Bundler.setup` without groups called. This will let Bundler
+# load ALL gems from ALL groups. This results in leaking
+# gems only for rails integration tests into msgr testing
+# environment.
+#
+# This file will be required by ruby on the commandline before
+# everything else can kick in. The code snippet below will
+# patch bundler to just ignore `setup` calls without
+# specified groups. All test helper will explicit call
+# `Bundler.setup` with required test groups.
+
+module Bundler
+  class << self
+    alias :old_setup :setup
+    def setup(*groups)
+      old_setup(*groups) unless groups.empty?
+    end
+  end
+end
+
+# Only load default group
+Bundler.setup :default

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: msgr
 version: !ruby/object:Gem::Version
-  version: 0.0.0
+  version: 0.1.0
 platform: ruby
 authors:
 - Jan Graichen
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-15 00:00:00.000000000 Z
+date: 2013-08-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -66,49 +66,7 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: '1.3'
-- !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'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  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'
-- !ruby/object:Gem::Dependency
-  name: coveralls
-  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: ''
+description: 'Msgr: Rails-like Messaging Framework'
 email:
 - jg@altimos.de
 executables: []
@@ -118,6 +76,7 @@ files:
 - .gitignore
 - .travis.yml
 - Gemfile
+- Guardfile
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -127,22 +86,77 @@ files:
 - lib/msgr/binding.rb
 - lib/msgr/client.rb
 - lib/msgr/connection.rb
+- lib/msgr/consumer.rb
 - lib/msgr/dispatcher.rb
 - lib/msgr/errors.rb
 - lib/msgr/logging.rb
 - lib/msgr/message.rb
+- lib/msgr/message/acknowledge.rb
 - lib/msgr/pool.rb
+- lib/msgr/railtie.rb
 - lib/msgr/route.rb
 - lib/msgr/routes.rb
 - lib/msgr/version.rb
 - msgr.gemspec
 - scripts/simple_test.rb
-- spec/msgr/client_spec.rb
-- spec/msgr/consumer_spec.rb
-- spec/msgr/route_spec.rb
-- spec/msgr/routes_spec.rb
-- spec/msgr_spec.rb
-- spec/spec_helper.rb
+- spec/fixtures/msgr-routes-test-1.rb
+- spec/integration/dummy/README.rdoc
+- spec/integration/dummy/Rakefile
+- spec/integration/dummy/app/assets/images/.keep
+- spec/integration/dummy/app/assets/javascripts/application.js
+- spec/integration/dummy/app/assets/stylesheets/application.css
+- spec/integration/dummy/app/consumers/application_consumer.rb
+- spec/integration/dummy/app/consumers/test_consumer.rb
+- spec/integration/dummy/app/controllers/application_controller.rb
+- spec/integration/dummy/app/controllers/concerns/.keep
+- spec/integration/dummy/app/controllers/test_controller.rb
+- spec/integration/dummy/app/helpers/application_helper.rb
+- spec/integration/dummy/app/mailers/.keep
+- spec/integration/dummy/app/models/.keep
+- spec/integration/dummy/app/models/concerns/.keep
+- spec/integration/dummy/app/views/layouts/application.html.erb
+- spec/integration/dummy/bin/bundle
+- spec/integration/dummy/bin/rails
+- spec/integration/dummy/bin/rake
+- spec/integration/dummy/config.ru
+- spec/integration/dummy/config/application.rb
+- spec/integration/dummy/config/boot.rb
+- spec/integration/dummy/config/database.yml
+- spec/integration/dummy/config/environment.rb
+- spec/integration/dummy/config/environments/development.rb
+- spec/integration/dummy/config/environments/production.rb
+- spec/integration/dummy/config/environments/test.rb
+- spec/integration/dummy/config/initializers/backtrace_silencers.rb
+- spec/integration/dummy/config/initializers/filter_parameter_logging.rb
+- spec/integration/dummy/config/initializers/inflections.rb
+- spec/integration/dummy/config/initializers/mime_types.rb
+- spec/integration/dummy/config/initializers/secret_token.rb
+- spec/integration/dummy/config/initializers/session_store.rb
+- spec/integration/dummy/config/initializers/wrap_parameters.rb
+- spec/integration/dummy/config/locales/en.yml
+- spec/integration/dummy/config/msgr.rb
+- spec/integration/dummy/config/rabbitmq.yml
+- spec/integration/dummy/config/routes.rb
+- spec/integration/dummy/db/test.sqlite3
+- spec/integration/dummy/lib/assets/.keep
+- spec/integration/dummy/log/.keep
+- spec/integration/dummy/public/404.html
+- spec/integration/dummy/public/422.html
+- spec/integration/dummy/public/500.html
+- spec/integration/dummy/public/favicon.ico
+- spec/integration/msgr/railtie_spec.rb
+- spec/integration/msgr_spec.rb
+- spec/integration/spec_helper.rb
+- spec/msgr/msgr/client_spec.rb
+- spec/msgr/msgr/connection_spec.rb
+- spec/msgr/msgr/consumer_spec.rb
+- spec/msgr/msgr/pool_spec.rb
+- spec/msgr/msgr/route_spec.rb
+- spec/msgr/msgr/routes_spec.rb
+- spec/msgr/msgr_spec.rb
+- spec/msgr/spec_helper.rb
+- spec/msgr/support/.keep
+- spec/support/setup.rb
 homepage: https://github.com/jgraichen/msgr
 licenses:
 - MIT
@@ -163,14 +177,67 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.6
+rubygems_version: 2.0.3
 signing_key: 
 specification_version: 4
-summary: ''
+summary: 'Msgr: Rails-like Messaging Framework'
 test_files:
-- spec/msgr/client_spec.rb
-- spec/msgr/consumer_spec.rb
-- spec/msgr/route_spec.rb
-- spec/msgr/routes_spec.rb
-- spec/msgr_spec.rb
-- spec/spec_helper.rb
+- spec/fixtures/msgr-routes-test-1.rb
+- spec/integration/dummy/README.rdoc
+- spec/integration/dummy/Rakefile
+- spec/integration/dummy/app/assets/images/.keep
+- spec/integration/dummy/app/assets/javascripts/application.js
+- spec/integration/dummy/app/assets/stylesheets/application.css
+- spec/integration/dummy/app/consumers/application_consumer.rb
+- spec/integration/dummy/app/consumers/test_consumer.rb
+- spec/integration/dummy/app/controllers/application_controller.rb
+- spec/integration/dummy/app/controllers/concerns/.keep
+- spec/integration/dummy/app/controllers/test_controller.rb
+- spec/integration/dummy/app/helpers/application_helper.rb
+- spec/integration/dummy/app/mailers/.keep
+- spec/integration/dummy/app/models/.keep
+- spec/integration/dummy/app/models/concerns/.keep
+- spec/integration/dummy/app/views/layouts/application.html.erb
+- spec/integration/dummy/bin/bundle
+- spec/integration/dummy/bin/rails
+- spec/integration/dummy/bin/rake
+- spec/integration/dummy/config.ru
+- spec/integration/dummy/config/application.rb
+- spec/integration/dummy/config/boot.rb
+- spec/integration/dummy/config/database.yml
+- spec/integration/dummy/config/environment.rb
+- spec/integration/dummy/config/environments/development.rb
+- spec/integration/dummy/config/environments/production.rb
+- spec/integration/dummy/config/environments/test.rb
+- spec/integration/dummy/config/initializers/backtrace_silencers.rb
+- spec/integration/dummy/config/initializers/filter_parameter_logging.rb
+- spec/integration/dummy/config/initializers/inflections.rb
+- spec/integration/dummy/config/initializers/mime_types.rb
+- spec/integration/dummy/config/initializers/secret_token.rb
+- spec/integration/dummy/config/initializers/session_store.rb
+- spec/integration/dummy/config/initializers/wrap_parameters.rb
+- spec/integration/dummy/config/locales/en.yml
+- spec/integration/dummy/config/msgr.rb
+- spec/integration/dummy/config/rabbitmq.yml
+- spec/integration/dummy/config/routes.rb
+- spec/integration/dummy/db/test.sqlite3
+- spec/integration/dummy/lib/assets/.keep
+- spec/integration/dummy/log/.keep
+- spec/integration/dummy/public/404.html
+- spec/integration/dummy/public/422.html
+- spec/integration/dummy/public/500.html
+- spec/integration/dummy/public/favicon.ico
+- spec/integration/msgr/railtie_spec.rb
+- spec/integration/msgr_spec.rb
+- spec/integration/spec_helper.rb
+- spec/msgr/msgr/client_spec.rb
+- spec/msgr/msgr/connection_spec.rb
+- spec/msgr/msgr/consumer_spec.rb
+- spec/msgr/msgr/pool_spec.rb
+- spec/msgr/msgr/route_spec.rb
+- spec/msgr/msgr/routes_spec.rb
+- spec/msgr/msgr_spec.rb
+- spec/msgr/spec_helper.rb
+- spec/msgr/support/.keep
+- spec/support/setup.rb
+has_rdoc: 

data/spec/msgr/client_spec.rb

@@ -1,5 +0,0 @@
-require 'spec_helper'
-
-describe Msgr::Client do
-
-end

data/spec/msgr/routes_spec.rb

@@ -1,59 +0,0 @@
-require 'spec_helper'
-
-describe Msgr::Routes do
-  let(:routes) { Msgr::Routes.new }
-
-  describe '#configure' do
-    let(:block) { Proc.new{} }
-
-    it 'should evaluate given block within instance context' do
-      expect(routes).to receive(:instance_eval).with { |p| p == block }
-
-      routes.configure &block
-    end
-
-    it 'should allow to call instance method in gven block' do
-      expect(routes).to receive(:test_instance_method).with(:abc)
-
-      routes.configure do
-        test_instance_method :abc
-      end
-    end
-  end
-
-  describe '#each' do
-    before do
-      routes.configure do
-        route 'abc.#', to: 'test#index'
-        route 'edf.#', to: 'test#index'
-      end
-    end
-
-    let(:each) { routes.each }
-
-    it 'should iterate over configured routes' do
-      expect(each).to have(2).items
-
-      expect(each.map(&:routing_key)).to be == %w(abc.# edf.#)
-      expect(each.map(&:consumer)).to be == %w(TestConsumer TestConsumer)
-      expect(each.map(&:action)).to be == %w(index index)
-    end
-  end
-
-  describe '#route' do
-    let(:subject) { -> { routes.route 'routing.key', to: 'test2#index2' } }
-    let(:last_route) { routes.routes.last }
-
-    it 'should add a new route' do
-      expect { subject.call }.to change{ routes.routes.size }.from(0).to(1)
-    end
-
-    it 'should add given route' do
-      subject.call
-
-      expect(last_route.routing_key).to eq 'routing.key'
-      expect(last_route.consumer).to eq 'Test2Consumer'
-      expect(last_route.action).to eq 'index2'
-    end
-  end
-end

data/spec/msgr_spec.rb

@@ -1,5 +0,0 @@
-require 'spec_helper'
-
-describe Msgr do
-
-end