process_handler 0.2.8 → 0.2.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7f5e8af968e63c3dcf63da7c363cac6202e1cfbf
-  data.tar.gz: 2ac6f1f121b0061955439fb2d895e9f980e82e8d
+  metadata.gz: 42e47422baa38b9c91c4bbaff699373e903c6f01
+  data.tar.gz: 5378e9443a93f92a57cb83ea6c69197b6ec9ae6a
 SHA512:
-  metadata.gz: 0ee3be4e336be22d5e66fd13b1e803cad36ab5e219873b01b7a5a30e2a8513dcf8b493b6beee90cc602a4cc09a9452d65cbe8c30cd558782ec08560e55c9c437
-  data.tar.gz: 27fced5c1fc513bccccce73bff82e27327964b31bbeba8449f78abd929bc19418896ef409db58323dab8c185f06829d4bae6ba60fe294d8cc13edde3d4d2894f
+  metadata.gz: a9e25775c634f4bd5a41c3dde5ede86888c1222e396ad871f74f6999c5ef010cf1accd6a209c053658aa1dffe953325a04bb7aa6e17174fe87ba8ca1990f057b
+  data.tar.gz: 401159f363f22948e273b396a3795139cf5879a6e27d5225301a06876cd668c2fb638cef21eb660dc6990a30ef9366939e6b785ae05b7f9eb56f9df4c9072496

data/Gemfile

@@ -5,7 +5,7 @@ gem 'rufus-scheduler'
 gem 'logasm'
 
 group :test do
-  gem 'rspec', '~> 3.1'
+  gem 'rspec', '~> 3.1.0'
   gem 'pry'
 end
 

data/lib/salemove/process_handler/pivot_process.rb

@@ -149,6 +149,7 @@ module Salemove
             end
           end
         rescue Timeout::Error
+          PivotProcess.logger.error "Fullfillable response was not fulfilled in #{timeout} seconds", input
           handle_response(handler, success: false, error: "Fulfillable response was not fulfilled")
         end
 

data/lib/salemove/process_handler/version.rb

@@ -1,5 +1,5 @@
 module Salemove
   module ProcessHandler
-    VERSION = '0.2.8'
+    VERSION = '0.2.9'
   end
 end

data/spec/process_handler/cron_process_spec.rb

@@ -62,7 +62,7 @@ describe ProcessHandler::CronProcess do
   describe 'exception handler' do
     let(:process) { ProcessHandler::CronProcess.new(params) }
     let(:params) {{ env: 'test', notifier_factory: notifier_factory,
-      scheduler_options: {frequency: 0.1} }}
+      scheduler_options: {frequency: 0.2} }}
     let(:notifier_factory) { double('NotifierFactory') }
     let(:exception_notifier) { double('Airbrake') }
     let(:queue) { Queue.new }
@@ -72,13 +72,12 @@ describe ProcessHandler::CronProcess do
     end
 
     it 'notifies of exception' do
-      process.schedule('0.2')
+      process.schedule('0.3')
       expect(exception_notifier).to receive(:notify_or_ignore)
       Thread.new do
         process.spawn(ExceptionService.new(queue))
       end
       queue.pop
-      sleep 0.1
     end
 
   end

data/spec/process_handler/pivot_process_spec.rb

@@ -154,15 +154,23 @@ describe ProcessHandler::PivotProcess do
       end
 
       context 'and its never fulfilled' do
+
+        let(:timeout) { 0.001 }
+
         before do
           allow(result).to receive(:fulfilled?) { false }
-          allow(result).to receive(:timeout) { 0.001 }
+          allow(result).to receive(:timeout) { timeout }
         end
 
         it 'responds with timeout error' do
           expect(handler).to receive(:error).with(success: false, error: "Fulfillable response was not fulfilled")
           subject
         end
+
+        it 'logs the error' do
+          expect(logger).to receive(:error).with("Fullfillable response was not fulfilled in #{timeout} seconds", {})
+          subject
+        end
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: process_handler
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.2.9
 platform: ruby
 authors:
 - Indrek Juhkam
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-04-18 00:00:00.000000000 Z
+date: 2016-07-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake