cpi-event-connector 0.0.12 → 0.0.13

data/lib/cpi_event_connector/connector.rb

@@ -26,12 +26,14 @@ module Cpi
     def start
       return if @started
 
+      Event.logger.debug "Connecting to Rabbit with params #{@params.inspect}"
       @bunny = Bunny.new @params
       @bunny.start
       @bunny.tx_select
       @exchange = @bunny.exchange('')
 
       @started = true
+      Event.logger.debug "Connection established"
     end
 
     def with_retries(*args, &block)
@@ -47,12 +49,15 @@ module Cpi
       begin
         yield
       rescue *exceptions => e
+        Event.logger.debug "Exception caught in #with_retries:\n#{e.message}"
         if retried < options[:retries]
           retried += 1
+          Event.logger.debug "Trying again. Have tried #{retried} times so far."
           sleep options[:sleep]
           options[:before_retry].call
           retry
         else
+          Event.logger.debug "Gave up on retries, throwing: #{e.message}" 
           raise e
         end
       end

data/lib/cpi_event_connector/event.rb

@@ -4,21 +4,31 @@ module Cpi
   class Event
     class << self
       attr_accessor :connection_params
+      attr_accessor :logger
     end
 
     @enabled = false
+    @logger = Logger.new("/dev/null")
 
-    def self.configure(params)
+    def self.configure(params, logger)
+      @logger = logger
+      @logger.info "CPI Event Connector initialized"
       @enabled = !!params.delete(:enabled)
+      @logger.info "CPI Event Connector is " + (@enabled ? "" : "not ") + "enabled"
       self.connection_params = params
     end
 
     def self.transmit(event_type, tenant_uid, root_path, options=nil)
-      return unless @enabled
+      if !@enabled
+        @logger.debug "Aborting transmit because CPI Event Connector not enabled"
+        return
+      end
       raise ArgumentError, "Tenant UID must be specified" if tenant_uid.blank?
 
       event_json = Cpi::Generator.create_event(event_type, tenant_uid, root_path)
+      @logger.debug "Transmitting message '#{event_json}'"
       self.connector.transmit(event_json, options)
+      @logger.debug "Transmission complete"
     end
 
     private

data/lib/cpi_event_connector/version.rb

@@ -1,3 +1,3 @@
 module Cpi
-  VERSION = "0.0.12"
+  VERSION = "0.0.13"
 end

data/spec/event_spec.rb

@@ -5,15 +5,22 @@ require 'unification_assertion'
 
 describe Cpi::Event do
   let(:config) { { :enabled => true, :key => "value" } }
+  let(:logger) { stub("Logger", debug: nil, info: nil) }
 
   describe ".configure" do
     let(:connector) { double(Cpi::Connector) }
 
     it "accepts a hash of connection information" do
-      Cpi::Event.configure(config)
+      Cpi::Event.configure(config, logger)
 
       Cpi::Event.connection_params.should == { :key => "value" }
     end
+
+    it "accepts a logger and sets it as a class variable" do
+      Cpi::Event.configure({}, logger)
+
+      Cpi::Event.logger.should == logger
+    end
   end
 
   describe ".transmit" do
@@ -28,7 +35,7 @@ describe Cpi::Event do
     let(:content) { "<test />" }
 
     it "generates and sends an event" do
-      Cpi::Event.configure(real_config)
+      Cpi::Event.configure(real_config, logger)
 
       Cpi::Event.transmit("shipment_shipped", "stuff", content)
 
@@ -43,7 +50,7 @@ describe Cpi::Event do
     end
 
     it "creates a connection for each thread" do
-      Cpi::Event.configure(config)
+      Cpi::Event.configure(config, logger)
 
       connector1 = double("connector 1")
       connector2 = double("connector 2")
@@ -64,7 +71,7 @@ describe Cpi::Event do
     it "does not transmit when it is not enabled" do
       real_config[:enabled] = false
 
-      Cpi::Event.configure(real_config)
+      Cpi::Event.configure(real_config, logger)
       Cpi::Generator.should_not_receive(:create_event)
       # And there shouldn't be an exception
 
@@ -72,12 +79,13 @@ describe Cpi::Event do
     end
 
     it "does nothing when called before being configured" do
+      Cpi::Event.logger = stub(debug: nil)
       Cpi::Generator.should_not_receive(:create_event)
       Cpi::Event.transmit("message", "tenant_uid", "content")
     end
 
     it "raises exception when no tenant_uid is passed in" do
-      Cpi::Event.configure(real_config)
+      Cpi::Event.configure(real_config, logger)
       Cpi::Generator.should_not_receive(:create_event)
       expect {Cpi::Event.transmit("message", nil, "content")}.to raise_error(ArgumentError)
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cpi-event-connector
 version: !ruby/object:Gem::Version
-  version: 0.0.12
+  version: 0.0.13
   prerelease: 
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-08-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &23871620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,15 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *23871620
 - !ruby/object:Gem::Dependency
   name: backports
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &23871200 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -37,15 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *23871200
 - !ruby/object:Gem::Dependency
   name: justinf-unification_assertion
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &23870780 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -53,15 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *23870780
 - !ruby/object:Gem::Dependency
   name: bunny
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &23870360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -69,15 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '0'
+  version_requirements: *23870360
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &23869860 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -85,15 +65,10 @@ dependencies:
         version: '3'
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: '3'
+  version_requirements: *23869860
 - !ruby/object:Gem::Dependency
   name: uuidtools
-  requirement: !ruby/object:Gem::Requirement
+  requirement: &23869360 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -101,12 +76,7 @@ dependencies:
         version: 2.1.2
   type: :runtime
   prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 2.1.2
+  version_requirements: *23869360
 description: see summary
 email:
 - engineering@nulogy.com
@@ -150,7 +120,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: Sends events to CPI