freddy 0.5.8 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 966278592fc831a84feed65275f1eb65f14eb5b4
-  data.tar.gz: 51e22027eaad56720377c10e143f1a6728afba78
+  metadata.gz: 616f7993a0da1423d5b0defb02cf3e4e75c51481
+  data.tar.gz: 44f8e9099607cc6073df4bb6fdb5cfef68692ede
 SHA512:
-  metadata.gz: 69b42a849ee64ed6389dac88a4c788871f74dd7b6ba472caa3f203d652a049e7783582dfcf8e668ceb9c399c0a04862f9a396184369eebd129d29f3df46dd11e
-  data.tar.gz: bfafc7e72dd1bfec8fac366123aad5eac18576a554b44345e1bb9b93922ba0d63fe3668c1a980dc53ce73502aa3d6136da310c3e4359229d2b017b0c4b30c444
+  metadata.gz: 0fe5040ae2e1611e6cc893f6f6783bcb6198c9dfc2fe2a2da497f2b240dc9d45f90aa4a3df89e1e4a0a09e5ee7e1815d51658f21dee31e8a633d3bfb3cb0eacd
+  data.tar.gz: 20f9a96c47647ceacc2d9a79de533ddcd57c6c81f3ad80d2052caef5ccd30d121fbd75557b75807c2c3cc0ffd6f5773dae3db257b5a9b45099b3cb64240e59d5

data/freddy.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |spec|
   else
     spec.name          = "freddy"
   end
-  spec.version       = '0.5.8'
+  spec.version       = '0.6.0'
   spec.authors       = ["Urmas Talimaa"]
   spec.email         = ["urmas.talimaa@gmail.com"]
   spec.description   = %q{Messaging API}

data/lib/freddy/producers/send_and_wait_response_producer.rb

@@ -67,10 +67,9 @@ class Freddy
         if request = @request_manager.delete(correlation_id)
           process_response(request, delivery)
         else
-          warning = "Got rpc response for correlation_id #{correlation_id} "\
+          message = "Got rpc response for correlation_id #{correlation_id} "\
                     "but there is no requester"
           @logger.warn message
-          Utils.notify 'NoRequesterForResponse', warning, correlation_id: correlation_id
         end
       end
 
@@ -85,14 +84,6 @@ class Freddy
           @logger.warn "Request timed out waiting response from #{destination}"\
                        ", correlation id #{correlation_id}"
 
-          Utils.notify 'RequestTimeout',
-            "Request timed out waiting for response from #{destination}",
-            {
-              correlation_id: correlation_id,
-              destination: destination,
-              timeout_in_seconds: timeout_in_seconds
-            }
-
           @request_manager.delete(correlation_id)
         end
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: freddy
 version: !ruby/object:Gem::Version
-  version: 0.5.8
+  version: 0.6.0
 platform: ruby
 authors:
 - Urmas Talimaa
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-26 00:00:00.000000000 Z
+date: 2016-10-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -134,14 +134,12 @@ files:
 - lib/freddy/responder_handler.rb
 - lib/freddy/sync_response_container.rb
 - lib/freddy/timeout_error.rb
-- lib/freddy/utils.rb
 - spec/freddy/consumers/respond_to_consumer_spec.rb
 - spec/freddy/error_response_spec.rb
 - spec/freddy/freddy_spec.rb
 - spec/freddy/message_handler_spec.rb
 - spec/freddy/responder_handler_spec.rb
 - spec/freddy/sync_response_container_spec.rb
-- spec/freddy/utils_spec.rb
 - spec/integration/concurrency_spec.rb
 - spec/integration/logging_spec.rb
 - spec/integration/reply_spec.rb
@@ -166,7 +164,7 @@ 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: API for inter-application messaging supporting acknowledgements and request-response
@@ -177,7 +175,6 @@ test_files:
 - spec/freddy/message_handler_spec.rb
 - spec/freddy/responder_handler_spec.rb
 - spec/freddy/sync_response_container_spec.rb
-- spec/freddy/utils_spec.rb
 - spec/integration/concurrency_spec.rb
 - spec/integration/logging_spec.rb
 - spec/integration/reply_spec.rb

data/lib/freddy/utils.rb

@@ -1,30 +0,0 @@
-class Freddy
-  class Utils
-    def self.format_exception(exception)
-      message = exception.message
-      backtrace = exception.backtrace.join("\n")
-
-      "#{message}\n#{backtrace}"
-    end
-
-    def self.notify(name, message, parameters={})
-      return unless defined?(Airbrake)
-
-      Airbrake.notify_or_ignore(
-        error_class: name,
-        error_message: message,
-        cgi_data: ENV.to_hash,
-        parameters: parameters
-      )
-    end
-
-    def self.notify_exception(exception, parameters={})
-      return unless defined?(Airbrake)
-
-      Airbrake.notify_or_ignore(exception,
-        cgi_data: ENV.to_hash,
-        parameters: parameters
-      )
-    end
-  end
-end

data/spec/freddy/utils_spec.rb

@@ -1,83 +0,0 @@
-require 'spec_helper'
-
-describe Freddy::Utils do
-  describe '.format_exception' do
-    subject { described_class.format_exception(exception) }
-
-    let(:exception) { double(backtrace: backtrace, message: message) }
-    let(:message)   { 'format exception test' }
-    let(:backtrace) { ['line1', 'line2', 'line3'] }
-
-    it 'format the exception' do
-      should eq "format exception test\n" \
-        "line1\n" \
-        "line2\n" \
-        'line3'
-    end
-  end
-
-  describe '.notify' do
-    subject { described_class.notify(error_class, error_message, parameters) }
-
-    let(:env_attributes) { double }
-    let(:error_class)    { double }
-    let(:error_message)  { double }
-    let(:parameters)     { double }
-
-    context 'when Airbrake is defined' do
-      let(:airbrake) { double }
-
-      before do
-        allow(ENV).to receive(:to_hash) { env_attributes }
-        stub_const('::Airbrake', airbrake)
-      end
-
-      it 'notifies airbrake' do
-        expect(airbrake).to receive(:notify_or_ignore).with(
-          error_class: error_class,
-          error_message: error_message,
-          cgi_data: env_attributes,
-          parameters: parameters
-        )
-
-        subject
-      end
-    end
-
-    context 'when Airbrake is not defined' do
-      it 'does nothing' do
-        should eq(nil)
-      end
-    end
-  end
-
-  describe '.notify_exception' do
-    subject { described_class.notify_exception(exception, {a: 'b'}) }
-
-    let(:exception) { double }
-
-    context 'when Airbrake is defined' do
-      let(:airbrake) { double }
-
-      before do
-        stub_const('::Airbrake', airbrake)
-      end
-
-      it 'notifies airbrake' do
-        expect(airbrake).to receive(:notify_or_ignore) do |ex, content|
-          expect(ex).to eq(exception)
-          expect(content[:cgi_data]).to be_instance_of(Hash)
-          expect(content[:parameters]).to eq(a: 'b')
-        end
-
-        subject
-      end
-    end
-
-    context 'when Airbrake is not defined' do
-      it 'does nothing' do
-        should eq(nil)
-      end
-    end
-  end
-end