cpi-event-connector 0.0.5 → 0.0.6

data/lib/cpi_event_connector/connector.rb

@@ -7,27 +7,30 @@ module Cpi
       @params = params
     end
 
-    def start
-      @bunny = Bunny.new @params
-      @bunny.start
-      @bunny.tx_select
-      @exchange = @bunny.exchange('')
-
-      @queue = @bunny.queue(@queue_name, :durable => true)
-    end
-
     def transmit(message)
       begin
+        start
         @exchange.publish(message, :key => @queue_name)
         @bunny.tx_commit
       rescue Bunny::ServerDownError, Bunny::ForcedConnectionCloseError, Bunny::ConnectionError
+        @started = false
         start
         retry
       end
     end
 
-    def stop
-      @bunny.stop
+    private
+
+    def start
+      return if @started
+
+      @bunny = Bunny.new @params
+      @bunny.start
+      @bunny.tx_select
+      @exchange = @bunny.exchange('')
+
+      @queue = @bunny.queue(@queue_name, :durable => true)
+      @started = true
     end
   end
-end
+end

data/lib/cpi_event_connector/event.rb

@@ -7,7 +7,6 @@ module Cpi
       @@enabled = !!params.delete(:enabled)
       return unless @@enabled
       @@connector = Cpi::Connector.new(params)
-      @@connector.start
     end
 
     def self.transmit(event_type, tenant_uid, root_path)

data/lib/cpi_event_connector/version.rb

@@ -1,3 +1,3 @@
 module Cpi
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/spec/connector_spec.rb

@@ -18,63 +18,64 @@ describe Cpi::Connector do
     bunny.stub(:exchange).and_return(exchange)
   end
 
-  context "before established connection" do
-    it "connects to an amqp server" do
-      Bunny.should_receive(:new).with(params).and_return(bunny)
-      bunny.should_receive(:start)
+  describe "#transmit" do
+    let(:connector) { Cpi::Connector.new params }
+    let(:event) { "event" }
 
-      connector = Cpi::Connector.new params
-      connector.start
+    before do
+      exchange.stub(:publish)
+      bunny.stub(:tx_commit)
     end
 
-    it "connects to the specified queue" do
-      bunny.should_receive(:queue).with("cpi", :durable => true).and_return(queue)
+    context "before connection established" do
+      it "connects to an amqp server" do
+        Bunny.should_receive(:new).with(params).and_return(bunny)
+        bunny.should_receive(:start)
 
-      connector = Cpi::Connector.new params
-      connector.start
-    end
+        connector.transmit(event)
+      end
 
-    it "creates a transaction before sending an event" do
-      bunny.should_receive(:tx_select)
+      it "connects to the specified queue" do
+        bunny.should_receive(:queue).with("cpi", :durable => true).and_return(queue)
 
-      connector = Cpi::Connector.new params
-      connector.start
-    end
+        connector.transmit(event)
+      end
 
-    it "gets an exhange" do
-      bunny.should_receive(:exchange).with('').and_return(exchange)
+      it "creates a transaction before sending an event" do
+        bunny.should_receive(:tx_select)
 
-      connector = Cpi::Connector.new params
-      connector.start
-    end
-  end
+        connector.transmit(event)
+      end
 
-  context "with established connection" do
-    before do
-      @connector = Cpi::Connector.new params
-      @connector.start
+      it "gets an exchange" do
+        bunny.should_receive(:exchange).with('').and_return(exchange)
 
-      exchange.stub(:publish)
-      bunny.stub(:tx_commit)
+        connector.transmit(event)
+      end
     end
 
-    it "disconnects from an amqp server" do
-      bunny.should_receive(:stop)
+    context "connection already established" do
+      before do
+        connector.transmit(event)
+      end
+
+      it "does not do anything" do
+        Bunny.should_not_receive(:new)
 
-      @connector.stop
+        connector.transmit(event)
+      end
     end
 
     it "sends a message to an amqp server" do
-      event = "an event"
       exchange.should_receive(:publish).with(event, :key => "cpi")
 
-      @connector.transmit(event)
+      connector.transmit(event)
     end
 
     it "commits the transaction after a message has been published" do
       bunny.should_receive(:tx_commit)
 
-      @connector.transmit("event")
+      connector.transmit("event")
     end
   end
 end

data/spec/event_spec.rb

@@ -24,7 +24,6 @@ enabled: true
       expected_config = YAML::load(yaml_config).symbolize_keys!
       expected_config.delete(:enabled)
       Cpi::Connector.should_receive(:new).with(expected_config).and_return(connector)
-      connector.should_receive(:start)
 
       Cpi::Event.configure(YAML::load(yaml_config).symbolize_keys!)
     end
@@ -33,7 +32,6 @@ enabled: true
       yaml_config = "---
 enabled: false"
       Cpi::Connector.should_not_receive(:new)
-      connector.should_not_receive(:start)
 
       Cpi::Event.configure(YAML::load(yaml_config))
     end

data/spec/integration_spec.rb

@@ -16,15 +16,12 @@ queue: connector-test"}
 
   it "sends a message to an amqp server" do
     connector = Cpi::Connector.new(YAML::load(yaml_config).symbolize_keys!)
-    connector.start
     connector.transmit("message")
-    connector.stop
 
     b = Bunny.new
     b.start
     q = b.queue("connector-test", :durable => true)
     message = q.pop
     message[:payload].should == "message"
-    b.stop
   end
-end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: cpi-event-connector
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 19
   prerelease: 
   segments: 
   - 0
   - 0
-  - 5
-  version: 0.0.5
+  - 6
+  version: 0.0.6
 platform: ruby
 authors: 
 - Nulogy
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-03-15 00:00:00 Z
+date: 2012-03-28 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rspec
@@ -146,7 +146,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.8.15
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: Sends events to CPI