health-monitor-rails 3.0.0 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8e0445698200b13f4bb7812701985f3e9b7ff266
-  data.tar.gz: ac55e392045c6c8ebfb15eed3e950186bb09b114
+  metadata.gz: 8ee9a0ab44b656791ce415bd313f798c25792370
+  data.tar.gz: acacace8a2adb7ed19d93c41c5dae17d640df45e
 SHA512:
-  metadata.gz: 653907b4a19de7af48c224d2d37489c9b7e6ef64ab0c30663f547ce2546987b8c0a15b404a62f756d3f44e6b5dfed20319d5463614a5232d5986c4d258c6c4d1
-  data.tar.gz: 48c717fa73607998055fc171db2d0511a5dc6db534be2f10cc9ba2331459617377287a49a787d2dd956c769837773f1b6fb879c3e0f50fbc8384cb9475827b4f
+  metadata.gz: af87c7a39394809788c85c82914afd96d230aaffcba8e81160dad6478a4f3e17a1e75112f85419ff4f26cfa0e1d4a684cbf7a993335e6b99f2304ff9c2637ca9
+  data.tar.gz: c406bb77b047bcb2d86d4e7e884017c7126dc0295c07df1fb65548f67b2f807e0ac18d5b5eaeb7a7d3d247ba014811136ba422d60cb696d13491c6348615a1e4

data/Rakefile

@@ -1,8 +1,11 @@
 #!/usr/bin/env rake
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
+require 'rubocop/rake_task'
 
 RSpec::Core::RakeTask.new('spec')
 Bundler::GemHelper.install_tasks
 
 task :default => :spec
+
+RuboCop::RakeTask.new

data/app/controllers/health_monitor/health_controller.rb

@@ -11,8 +11,8 @@ module HealthMonitor
       HealthMonitor.check!(request: request)
 
       self.status = :ok
-      self.response_body = "Health check has passed: #{Time.now.to_s(:db)}\n"
-    rescue Exception => e
+      self.response_body = "Health check has passed: r#{Time.now.to_s(:db)}\n"
+    rescue => e
       self.status = :service_unavailable
       self.response_body = "Health check has failed: #{Time.now.to_s(:db)}, error: #{e.message}\n"
     end

data/lib/health_monitor/configuration.rb

@@ -10,7 +10,7 @@ module HealthMonitor
     end
 
     PROVIDERS.each do |provider_name|
-      define_method provider_name do |&block|
+      define_method provider_name do |&_block|
         require "health_monitor/providers/#{provider_name}"
 
         add_provider("HealthMonitor::Providers::#{provider_name.capitalize}".constantize)
@@ -19,7 +19,8 @@ module HealthMonitor
 
     def add_custom_provider(custom_provider_class)
       unless custom_provider_class < HealthMonitor::Providers::Base
-        raise ArgumentError.new('custom provider class must implement HealthMonitor::Providers::Base')
+        raise ArgumentError.new 'custom provider class must implement '\
+          'HealthMonitor::Providers::Base'
       end
 
       add_provider(custom_provider_class)

data/lib/health_monitor/monitor.rb

@@ -16,8 +16,9 @@ module HealthMonitor
       monitor = provider.new(request: request)
       monitor.check!
     end
-  rescue Exception => e
+  rescue => e
     configuration.error_callback.call(e) if configuration.error_callback
+
     raise
   end
 end

data/lib/health_monitor/providers/base.rb

@@ -23,8 +23,6 @@ module HealthMonitor
         raise NotImplementedError
       end
 
-      private
-
       def self.configurable?
         configuration_class
       end

data/lib/health_monitor/providers/redis.rb

@@ -14,7 +14,7 @@ module HealthMonitor
         fetched = redis.get(key)
 
         raise "different values (now: #{time}, fetched: #{fetched}" if fetched != time
-      rescue => e
+      rescue Exception => e
         raise RedisException.new(e.message)
       ensure
         redis.client.disconnect

data/lib/health_monitor/providers/sidekiq.rb

@@ -37,13 +37,13 @@ module HealthMonitor
       def check_latency!
         latency = ::Sidekiq::Queue.new.latency
 
-        if latency > self.configuration.latency
-          raise "latency #{latency} is greater than #{self.configuration.latency}"
-        end
+        return unless latency > configuration.latency
+
+        raise "latency #{latency} is greater than #{configuration.latency}"
       end
 
       def check_redis!
-        ::Sidekiq.redis { |conn| conn.info }
+        ::Sidekiq.redis(&:info)
       end
     end
   end

data/lib/health_monitor/version.rb

@@ -1,3 +1,3 @@
 module HealthMonitor
-  VERSION = '3.0.0'.freeze
+  VERSION = '3.0.1'.freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: health-monitor-rails
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
 platform: ruby
 authors:
 - Leonid Beder
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-10 00:00:00.000000000 Z
+date: 2015-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -122,6 +122,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rediska
   requirement: !ruby/object:Gem::Requirement
@@ -176,7 +190,6 @@ files:
 - Rakefile
 - app/controllers/health_monitor/health_controller.rb
 - config/routes.rb
-- lib/health-monitor-rails.rb
 - lib/health_monitor/configuration.rb
 - lib/health_monitor/engine.rb
 - lib/health_monitor/monitor.rb
@@ -187,6 +200,7 @@ files:
 - lib/health_monitor/providers/resque.rb
 - lib/health_monitor/providers/sidekiq.rb
 - lib/health_monitor/version.rb
+- lib/health_monitor_rails.rb
 homepage: https://github.com/lbeder/health-monitor-rails
 licenses:
 - MIT