process_handler 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2530d6f931ed0ff1a7f617c7e04fdbf4be28a4ee
-  data.tar.gz: fb6d3c7f5e79fb988df866084dfc0ea116ca4c6c
+  metadata.gz: 3025924d4d9d6aabf01d20bb1fe3517f3d20ef1d
+  data.tar.gz: 55fb26289828ad292043ae2eb2ca1657d59286b4
 SHA512:
-  metadata.gz: 1781ed0ee1cf6fa992a64c1a0cb4bf49a4ed002346b8b705df2de9013ce0c8390c16249af2ed4e5da07f01030bc75060760caeaff32d9b8d1657086dd84e2c4a
-  data.tar.gz: 40e9b2d747eadfe6f423ca37667c802e79a441eecb7bbc7ffe5c06abc6d2dbdeb01cc6bd2e60d6e2333fa5625d6d8536a47a1633e83ce1d0ab512b0151035b3a
+  metadata.gz: 9907397cf94e2c1e5c92881645bb6efd826bf0ccbcd831f59c5b964ac6f723dbfaf39c4e59d27c8b6f3b42287a737eead053f9924b710983db1c31321f5eb6c0
+  data.tar.gz: 745805dfd41786a6d69b5c1aaa667fa13e99bb1a76676b0abd53905bc5b32848e02b7d6d58b0f626259af24582fa1adaca528f89e77a05b42ce8475c302acce2

data/.travis.yml

@@ -0,0 +1,6 @@
+language: ruby
+rvm:
+  - 2.1.0
+before_script:
+  - bundle
+script: bundle exec rspec

data/README.md

@@ -1,5 +1,6 @@
 # ProcessHandler
 
+[![Build Status](https://travis-ci.org/salemove/process_handler.svg?branch=master)](https://travis-ci.org/salemove/process_handler)
 [![Code Climate](https://codeclimate.com/github/salemove/process_handler/badges/gpa.svg)](https://codeclimate.com/github/salemove/process_handler)
 
 ProcessHandler helps to spawn and manage services. There are multiple types of processes. Every process knows how to handle `SIGINT` and `SIGTERM` signals.

data/lib/salemove/process_handler/pivot_process.rb

@@ -7,6 +7,8 @@ module Salemove
   module ProcessHandler
     class PivotProcess
 
+      DEFAULT_FULFILLABLE_TIMEOUT = 3
+
       attr_reader :process_monitor, :exception_notifier
 
       def self.logger
@@ -57,13 +59,34 @@ module Salemove
         def spawn
           @messenger.respond_to(@service.class::QUEUE) do |input, handler|
             response = handle_request(input)
-
-            if response.is_a?(Hash) && (response[:success] == false || response[:error])
-              handler.error(response)
+            if response.respond_to?(:fulfilled?)
+              handle_fulfillable_response(handler, response)
             else
-              handler.success(response)
+              handle_response(handler, response)
+            end
+          end
+        end
+
+        def handle_fulfillable_response(handler, response)
+          timeout = response.respond_to?(:timeout) && response.timeout || DEFAULT_FULFILLABLE_TIMEOUT
+          Timeout::timeout(timeout) do
+            while true
+              if response.fulfilled?
+                return handle_response(handler, response.value)
+              end
+              sleep 0.001
             end
           end
+        rescue Timeout::Error
+          handle_response(handler, success: false, error: "Fulfillable response was not fulfilled")
+        end
+
+        def handle_response(handler, response)
+          if response.is_a?(Hash) && (response[:success] == false || response[:error])
+            handler.error(response)
+          else
+            handler.success(response)
+          end
         end
 
         def handle_request(input)

data/lib/salemove/process_handler/version.rb

@@ -1,5 +1,5 @@
 module Salemove
   module ProcessHandler
-    VERSION = '0.1.0'
+    VERSION = '0.1.1'
   end
 end

data/spec/process_handler/pivot_process_spec.rb

@@ -134,4 +134,52 @@ describe ProcessHandler::PivotProcess do
 
   end
 
+  describe 'when result is fulfillable' do
+    let(:result) { double }
+
+    context 'and its already fulfilled' do
+      let(:value) { { success: true, output: { result: 'R'} } }
+
+      before do
+        allow(result).to receive(:fulfilled?) { true }
+        allow(result).to receive(:value) { value }
+      end
+
+      it 'responds immediately' do
+        expect(handler).to receive(:success).with(value)
+        subject()
+      end
+    end
+
+    context 'and its fulfilled later' do
+      let(:value) { { success: true, output: { result: 'R'} } }
+
+      before do
+        allow(result).to receive(:fulfilled?) { false }
+        Thread.new do
+          sleep 0.005
+          allow(result).to receive(:fulfilled?) { true }
+          allow(result).to receive(:value) { value }
+        end
+      end
+
+      it 'responds when fulfilled' do
+        expect(handler).to receive(:success).with(value)
+        subject()
+      end
+    end
+
+    context 'and its never fulfilled' do
+      before do
+        allow(result).to receive(:fulfilled?) { false }
+        allow(result).to receive(:timeout) { 0.001 }
+      end
+
+      it 'responds with timeout error' do
+        expect(handler).to receive(:error).with(success: false, error: "Fulfillable response was not fulfilled")
+        subject
+      end
+    end
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: process_handler
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Indrek Juhkam
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-08 00:00:00.000000000 Z
+date: 2015-05-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake
@@ -47,6 +47,7 @@ extra_rdoc_files: []
 files:
 - ".ruby-gemset"
 - ".ruby-version"
+- ".travis.yml"
 - Gemfile
 - Gemfile.lock
 - README.md
@@ -87,7 +88,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.2
 signing_key: 
 specification_version: 4
 summary: ''