freddy 0.5.1 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: aec4d79c45cbf9bc4204b7e47cf2636e133c6db7
-  data.tar.gz: f5ddfd889863171d89ca28e0a3eeb9c3da7cd52e
+  metadata.gz: 87c1ea2e30dfb3705596a40af65a95c4d722ad41
+  data.tar.gz: 60c14e4980b8e72cf131cb6161ebf8a6b632157a
 SHA512:
-  metadata.gz: 3cc387f36dfb0151233444e40c0ef1a7150ef84781c98dd9e1dc32f11849bdcdf4f7ee59f3785ba47d54be9573a74a91356a27aac329d52015a6955d7150805d
-  data.tar.gz: 7e65b4f8d86689743bc1bd12109b09e57473d02e7d1f6e713cfd86b377ebe158a6ea94cb1268b26adb708bad78a1bb825988fd066c6cb23d161b599b9886029b
+  metadata.gz: 68b0c2b6aa8b41ec71ead2d99e01be93e5e686aec774a6c99313e0d227196b0d56336ef255858c41916f35b72cdb946ff945723c02a270df898c13ad8f9a1ee4
+  data.tar.gz: 3f4a430ef425723992644418e3a526cf51bc6cca1abe7ef3043f0d9569d84953c20240541a384ddfba92ee93596e2d9ad06b7e1a9458fe26a7095b18ecfbdc47

data/freddy.gemspec

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

data/lib/freddy/sync_response_container.rb

@@ -19,8 +19,10 @@ class Freddy
         @mutex.sleep(timeout)
       end
 
-      if @response.nil?
+      if !defined?(@response)
         raise Timeout::Error, 'execution expired'
+      elsif @response.nil?
+        raise StandardError, 'unexpected nil value for response'
       elsif @response[:error] == 'RequestTimeout'
         raise TimeoutError.new(@response)
       elsif !@delivery || @delivery.type == 'error'
@@ -35,9 +37,5 @@ class Freddy
     def to_proc
       Proc.new {|*args| self.call(*args)}
     end
-
-    def filled?
-      !@response.nil?
-    end
   end
 end

data/spec/freddy/sync_response_container_spec.rb

@@ -10,4 +10,21 @@ describe Freddy::SyncResponseContainer do
       expect { subject }.to raise_error(Timeout::Error, 'execution expired')
     end
   end
+
+  context 'when nil resonse' do
+    let(:delivery) { {} }
+
+    before do
+      Thread.new do
+        default_sleep
+        container.call(nil, delivery)
+      end
+    end
+
+    it 'raises timeout error' do
+      expect {
+        container.wait_for_response(2)
+      }.to raise_error(StandardError, 'unexpected nil value for response')
+    end
+  end
 end

data/spec/integration/reply_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe 'Reply' do
+  let(:freddy) { Freddy.build(logger, config) }
+
+  let(:destination) { random_destination }
+  let(:request_payload) { {req: 'load'} }
+  let(:response_payload) { {res: 'load'} }
+
+  after { freddy.close }
+
+  context 'when a synchronized request' do
+    before do
+      freddy.respond_to(destination) do |payload, msg_handler|
+        msg_handler.success(response_payload)
+      end
+    end
+
+    it 'sends reply' do
+      response = freddy.deliver_with_response(destination, request_payload)
+      expect(response).to eq(response_payload)
+    end
+
+    it 'does not send the reply to the topic queue' do
+      freddy.tap_into 'amq.*' do |payload|
+        @message_received = true
+      end
+
+      freddy.deliver_with_response(destination, request_payload)
+      default_sleep
+
+      expect(@message_received).to be_falsy
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: freddy
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.2
 platform: ruby
 authors:
 - Urmas Talimaa
@@ -143,6 +143,7 @@ files:
 - spec/freddy/sync_response_container_spec.rb
 - spec/integration/concurrency_spec.rb
 - spec/integration/logging_spec.rb
+- spec/integration/reply_spec.rb
 - spec/spec_helper.rb
 homepage: 
 licenses:
@@ -177,4 +178,5 @@ test_files:
 - spec/freddy/sync_response_container_spec.rb
 - spec/integration/concurrency_spec.rb
 - spec/integration/logging_spec.rb
+- spec/integration/reply_spec.rb
 - spec/spec_helper.rb