process_handler 0.3.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 96dfb240ae4f597188a5ff12c95fa1f9d8478365
-  data.tar.gz: ebc125a0ca392c0ca7f7754650a5ae80290dfe2b
+  metadata.gz: a4a0b0712332a3351be93705ac3d10cb25aac695
+  data.tar.gz: 04ef8921eeafe7704d6239252f81f6cf992cc723
 SHA512:
-  metadata.gz: 9c6aac593c894445137330a8d812dff1c03b81ccdae5254c1203659d30bf8aa2236ed844236b7859c1eb870517fe04f4f6d696a8c38ce706381ca47af69bc894
-  data.tar.gz: 9cdf6d9278e73b532dc853b52eb69f54ae1efbbf04f257fdc9e89a85503887b521cac5a2b4ce56ed6b32ffb5c2a8d6695aad5f5ed59e860cd2c92fa8d7684a64
+  metadata.gz: 5eedc6fa3ecc33d660884c3db2260b06e78ea20c61bf3305f20cb412c7c426d55e9834ddf8ba822076e77715cfb93ed9357e4d390dd79ab2ae464c6eee0832de
+  data.tar.gz: efbe06f03d975f036c7a80f8beb3af369cd1f0a273b5c3a00e3cc8da208cba7995933b69d56fab224a122a13f84dd137892dded74f2205c24da57c1ccfd8a09c

data/LICENSE

@@ -0,0 +1,22 @@
+Copyright (c) 2016-2017 SaleMove Inc.
+
+MIT License
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/lib/salemove/process_handler/cron_process.rb

@@ -25,12 +25,11 @@ module Salemove
 
       attr_reader :process_monitor
 
-      def initialize(env: 'development',
-                     notifier: nil,
+      def initialize(notifier: nil,
                      notifier_factory: NotifierFactory,
                      scheduler_options: {})
         @schedules = []
-        @exception_notifier = notifier_factory.get_notifier(env, 'Cron Process', notifier)
+        @exception_notifier = notifier_factory.get_notifier('Cron Process', notifier)
         @scheduler = CronScheduler.new @exception_notifier, scheduler_options
         @process_monitor = CronProcessMonitor.new(self)
       end

data/lib/salemove/process_handler/notifier_factory.rb

@@ -6,18 +6,18 @@ module Salemove
   module ProcessHandler
     class NotifierFactory
 
-      def self.get_notifier(env, process_name, conf)
+      def self.get_notifier(process_name, conf)
         if conf && conf[:type] == 'airbrake'
           Airbrake.configure do |airbrake|
             airbrake.async = true
-            airbrake.environment_name = env
+            airbrake.environment_name = conf.fetch(:environment_name)
             airbrake.secure = true
             airbrake.host = conf.fetch(:host)
             airbrake.api_key = conf.fetch(:api_key)
             [/_HOST$/, /_TCP$/, /_UDP$/, /_PROTO$/, /_ADDR$/, 'PWD',
-            'GEM_HOME', 'PATH', 'SERVICE_NAME', 'RUBY_MAJOR', 'RUBY_VERSION', 
-            'RUN_ENV', 'HOME', 'RUBYGEMS_VERSION', 'BUNDLER_VERSION'].each { 
-              | param_whitelist_filter| airbrake.params_whitelist_filters << param_whitelist_filter 
+            'GEM_HOME', 'PATH', 'SERVICE_NAME', 'RUBY_MAJOR', 'RUBY_VERSION',
+            'RUN_ENV', 'HOME', 'RUBYGEMS_VERSION', 'BUNDLER_VERSION'].each {
+              | param_whitelist_filter| airbrake.params_whitelist_filters << param_whitelist_filter
             }
           end
           Airbrake

data/lib/salemove/process_handler/pivot_process.rb

@@ -21,7 +21,6 @@ module Salemove
       end
 
       def initialize(messenger,
-                     env: 'development',
                      notifier: nil,
                      notifier_factory: NotifierFactory,
                      process_monitor: ProcessMonitor.new,
@@ -29,7 +28,7 @@ module Salemove
                      exit_enforcer: nil)
         @messenger = messenger
         @process_monitor = process_monitor
-        @exception_notifier = notifier_factory.get_notifier(env, process_name, notifier)
+        @exception_notifier = notifier_factory.get_notifier(process_name, notifier)
         # Needed for forcing exit from jruby with exit(0)
         @exit_enforcer = exit_enforcer || Proc.new {}
       end

data/lib/salemove/process_handler/version.rb

@@ -1,5 +1,5 @@
 module Salemove
   module ProcessHandler
-    VERSION = '0.3.0'
+    VERSION = '1.0.0'
   end
 end

data/spec/process_handler/airbrake_spec.rb

@@ -6,14 +6,19 @@ require 'salemove/process_handler/notifier_factory'
 describe 'Airbrake configuration' do
 
   let (:notice) { Airbrake::Notice.new(notice_args.merge(custom_notice_args)) }
-  let (:notifier_factory_conf) { {:type => 'airbrake',:host => 'localhost',:api_key => 'abc123'} }
-  let (:airbrake) { ProcessHandler::NotifierFactory.get_notifier('env', 'Process name', notifier_factory_conf) }
+  let (:environment_name) { 'SOME_ENVIRONMENT' }
+  let (:notifier_factory_conf) { {:type => 'airbrake',
+                                  :host => 'localhost',
+                                  :api_key => 'abc123',
+                                  :environment_name => environment_name} }
+  let (:airbrake) { ProcessHandler::NotifierFactory.get_notifier('Process name', notifier_factory_conf) }
   # Notice needs Airbrake configuration merged with :exception for creating the exception notification
   let (:notice_args) { {:exception => Exception.new}.merge(airbrake.configuration) }
+  let (:custom_notice_args) { {} }
   let (:filtered) { '[FILTERED]' }
 
   # Uses Airbrake configuration's 'params_whitelist_filters' param for filtering
-  context 'whitelist filter' do
+  describe 'whitelist filtering' do
 
     let (:custom_notice_args) {
       {
@@ -52,4 +57,7 @@ describe 'Airbrake configuration' do
 
   end
 
+  it 'configures environment_name' do
+    expect(notice[:environment_name]).to eq environment_name
+  end
 end

data/spec/process_handler/cron_process_spec.rb

@@ -61,7 +61,7 @@ describe ProcessHandler::CronProcess do
 
   describe 'exception handler' do
     let(:process) { ProcessHandler::CronProcess.new(params) }
-    let(:params) {{ env: 'test', notifier_factory: notifier_factory,
+    let(:params) {{ notifier_factory: notifier_factory,
       scheduler_options: {frequency: 0.2} }}
     let(:notifier_factory) { double('NotifierFactory') }
     let(:exception_notifier) { double('Airbrake') }

data/spec/process_handler/pivot_process_spec.rb

@@ -8,10 +8,10 @@ describe ProcessHandler::PivotProcess do
   let(:handler)   { double('Handler') }
   let(:thread)    { double('Thread') }
   let(:process) { ProcessHandler::PivotProcess.new(messenger, process_params) }
-  let(:process_params) {{ process_monitor: monitor , notifier_factory: notifier_factory, env: 'test' }}
+  let(:process_params) {{ process_monitor: monitor , notifier_factory: notifier_factory}}
   let(:notifier_factory) { double('NotifierFactory') }
   let(:responder) { double(shutdown: true) }
-  let(:logger) { Logasm.new([]) }
+  let(:logger) { Logasm.build('test-app', []) }
 
   def expect_monitor_to_behave
     expect(monitor).to receive(:start)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: process_handler
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Indrek Juhkam
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-15 00:00:00.000000000 Z
+date: 2017-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: airbrake
@@ -78,6 +78,7 @@ files:
 - ".ruby-version"
 - ".travis.yml"
 - Gemfile
+- LICENSE
 - README.md
 - Rakefile
 - config.reek
@@ -117,7 +118,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: ''