hey_doctor 1.1.0 → 1.2.1

data/spec/lib/hey_doctor/hey_doctor_spec.rb

@@ -6,17 +6,29 @@ RSpec.describe '.Rack HealthCheck endpoint' do
   let(:expected_response) do
     {
       app: {
-        message: 'foo',
-        success: true
+        success: true,
+        message: 'foo'
       },
       database: {
-        message: 'foo',
-        success: true
+        success: true,
+        message: 'foo'
       },
       redis: {
-        message: 'foo',
-        success: true
-      }
+        success: true,
+        message: 'foo'
+      },
+      sidekiq: [
+        {
+          success: true,
+          message: 'Sidekiq is connected',
+          host: 'sidekiq_1'
+        },
+        {
+          success: false,
+          message: 'Error sidekiq',
+          host: 'sidekiq_2'
+        }
+      ]
     }.to_json
   end
 
@@ -24,13 +36,21 @@ RSpec.describe '.Rack HealthCheck endpoint' do
 
   before do
     allow(HeyDoctor::CheckApplicationHealthService).to receive(:call)
-      .and_return({ message: 'foo', success: true })
+      .and_return({ success: true, message: 'foo' })
 
     allow(HeyDoctor::CheckDatabaseHealthService).to receive(:call)
-      .and_return({ message: 'foo', success: true })
+      .and_return({ success: true, message: 'foo' })
 
     allow(HeyDoctor::CheckRedisHealthService).to receive(:call)
-      .and_return({ message: 'foo', success: true })
+      .and_return({ success: true, message: 'foo' })
+
+    allow(HeyDoctor::CheckSidekiqHealthService).to receive(:call)
+      .and_return(
+        [
+          { success: true, message: 'Sidekiq is connected', host: 'sidekiq_1' },
+          { success: false, message: 'Error sidekiq', host: 'sidekiq_2' }
+        ]
+      )
   end
 
   it 'build the json response' do

data/spec/requests/health_check_spec.rb

@@ -3,26 +3,11 @@
 require 'rails_helper'
 
 RSpec.describe 'Health Check endpoint', type: :request do
-  let(:expected_response) do
-    {
-      app: {
-        message: 'Application is running',
-        success: true
-      },
-      database: {
-        message: 'Database is connected',
-        success: true
-      },
-      redis: {
-        message: 'Redis is connected',
-        success: true
-      }
-    }.to_json
-  end
-
-  before { get '/_ah/health' }
+  context 'When it reaches the internal app_health' do
+    before { get '/_ah/app_health' }
 
-  xit 'builds the response' do
-    expect(response.body).to eq(expected_response)
+    it 'gets internal / that is mounted in /_ah/app_health' do
+      expect(response.code).to eq('200')
+    end
   end
 end

data/spec/services/hey_doctor/check_application_health_service_spec.rb

@@ -16,8 +16,8 @@ RSpec.describe HeyDoctor::CheckApplicationHealthService do
     context 'When app is running' do
       let(:expected_response) do
         {
-          message: 'Application is running',
-          success: true
+          success: true,
+          message: 'Application is running'
         }
       end
 
@@ -31,8 +31,8 @@ RSpec.describe HeyDoctor::CheckApplicationHealthService do
     context 'When app is down' do
       let(:expected_response) do
         {
-          message: 'Application down, call the firefighters',
-          success: false
+          success: false,
+          message: 'Application down, call the firefighters'
         }
       end
 

data/spec/services/hey_doctor/check_database_health_service_spec.rb

@@ -7,8 +7,8 @@ RSpec.describe HeyDoctor::CheckDatabaseHealthService do
     context 'when it is connected' do
       let(:expected_response) do
         {
-          message: 'Database is connected',
-          success: true
+          success: true,
+          message: 'Database is connected'
         }
       end
 
@@ -20,8 +20,8 @@ RSpec.describe HeyDoctor::CheckDatabaseHealthService do
     context 'when it is not connected' do
       let(:expected_response) do
         {
-          message: 'Error connecting to database',
-          success: false
+          success: false,
+          message: 'Error connecting to database'
         }
       end
 
@@ -38,8 +38,8 @@ RSpec.describe HeyDoctor::CheckDatabaseHealthService do
     context 'When migration is Pending' do
       let(:expected_response) do
         {
-          message: 'Pending migrations detected',
-          success: true
+          success: true,
+          message: 'Pending migrations detected'
         }
       end
 

data/spec/services/hey_doctor/check_redis_health_service_spec.rb

@@ -7,8 +7,8 @@ RSpec.describe HeyDoctor::CheckRedisHealthService do
     context 'when it is connected' do
       let(:expected_response) do
         {
-          message: 'Redis is connected',
-          success: true
+          success: true,
+          message: 'Redis is connected'
         }
       end
 
@@ -20,8 +20,8 @@ RSpec.describe HeyDoctor::CheckRedisHealthService do
     context 'when it is not connected' do
       let(:expected_response) do
         {
-          message: 'Error connecting to redis',
-          success: false
+          success: false,
+          message: 'Error connecting to redis'
         }
       end
 

data/spec/services/hey_doctor/check_sidekiq_health_service_spec.rb

@@ -0,0 +1,71 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe HeyDoctor::CheckSidekiqHealthService do
+  describe '.call' do
+    context 'when it is connected' do
+      let(:expected_response) do
+        [
+          {
+            success: true,
+            message: 'Sidekiq is connected',
+            host: 'dcsiloifoodcatalogimporter_sidekiq_1'
+          }
+        ]
+      end
+
+      before do
+        allow(described_class).to receive(:hosts)
+          .and_return(['dcsiloifoodcatalogimporter_sidekiq_1'])
+
+        allow(described_class).to receive(:connected?)
+          .with('dcsiloifoodcatalogimporter_sidekiq_1').and_return(true)
+      end
+
+      it 'respond with success' do
+        expect(described_class.call).to eq(expected_response)
+      end
+    end
+
+    context 'when it is not connected' do
+      let(:expected_response) do
+        [
+          {
+            success: false,
+            message: 'Error connecting to sidekiq',
+            host: 'dcsiloifoodcatalogimporter_sidekiq_2'
+          }
+        ]
+      end
+
+      before do
+        allow(described_class).to receive(:hosts)
+          .and_return(['dcsiloifoodcatalogimporter_sidekiq_2'])
+      end
+
+      it 'respond with error' do
+        expect(described_class.call).to eq(expected_response)
+      end
+    end
+  end
+
+  context 'when there is no executor machine' do
+    let(:expected_response) do
+      {
+        success: false,
+        message: 'None sidekiq host was found, add it to the ' \
+                 'ENV SIDEKIQ_HOSTS listing your machine(s) ip or' \
+                 ' dns using  ; for each host'
+      }
+    end
+
+    before do
+      allow(described_class).to receive(:hosts).and_return(nil)
+    end
+
+    it 'respond with error' do
+      expect(described_class.call).to eq(expected_response)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -2,6 +2,7 @@
 
 require 'simplecov'
 require 'rack/test'
+require 'pry'
 
 SimpleCov.start 'rails' do
   add_filter '/bin'

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: hey_doctor
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Matheus Acosta
 - Erick Nascimento
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-01-22 00:00:00.000000000 Z
+date: 2021-04-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dotenv-rails
@@ -25,6 +25,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  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: redis
   requirement: !ruby/object:Gem::Requirement
@@ -81,6 +95,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: mimemagic
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.3.7
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.3.7
 - !ruby/object:Gem::Dependency
   name: rails
   requirement: !ruby/object:Gem::Requirement
@@ -95,7 +123,7 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 4.2.10
-description: It mounts a middleware endpoint to check the stack health.
+description: It mounts a middleware endpoint to check the stack health
 email:
 - matheus.martins@deliverycenter.com
 - erick.nascimento@deliverycenter.com
@@ -109,6 +137,7 @@ files:
 - app/services/hey_doctor/check_application_health_service.rb
 - app/services/hey_doctor/check_database_health_service.rb
 - app/services/hey_doctor/check_redis_health_service.rb
+- app/services/hey_doctor/check_sidekiq_health_service.rb
 - config/routes.rb
 - lib/hey_doctor.rb
 - lib/hey_doctor/engine.rb
@@ -162,6 +191,7 @@ files:
 - spec/services/hey_doctor/check_application_health_service_spec.rb
 - spec/services/hey_doctor/check_database_health_service_spec.rb
 - spec/services/hey_doctor/check_redis_health_service_spec.rb
+- spec/services/hey_doctor/check_sidekiq_health_service_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/deliverycenter/hey_doctor
 licenses:
@@ -170,7 +200,7 @@ metadata:
   homepage_uri: https://github.com/deliverycenter/hey_doctor
   source_code_uri: https://github.com/deliverycenter/hey_doctor
   changelog_uri: https://github.com/deliverycenter/hey_doctor
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -185,57 +215,58 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.9
-signing_key:
+rubygems_version: 3.0.3
+signing_key: 
 specification_version: 4
 summary: A health check gem implementation in ruby.
 test_files:
-- spec/spec_helper.rb
-- spec/dummy/app/mailers/application_mailer.rb
-- spec/dummy/app/models/application_record.rb
-- spec/dummy/app/javascript/packs/application.js
-- spec/dummy/app/jobs/application_job.rb
+- spec/dummy/Rakefile
+- spec/dummy/app/assets/config/manifest.js
+- spec/dummy/app/assets/stylesheets/application.css
+- spec/dummy/app/channels/application_cable/channel.rb
+- spec/dummy/app/channels/application_cable/connection.rb
 - spec/dummy/app/controllers/application_controller.rb
 - spec/dummy/app/controllers/home_controller.rb
+- spec/dummy/app/javascript/packs/application.js
+- spec/dummy/app/jobs/application_job.rb
+- spec/dummy/app/mailers/application_mailer.rb
+- spec/dummy/app/models/application_record.rb
 - spec/dummy/app/views/layouts/mailer.html.erb
 - spec/dummy/app/views/layouts/mailer.text.erb
-- spec/dummy/app/assets/config/manifest.js
-- spec/dummy/app/assets/stylesheets/application.css
-- spec/dummy/app/channels/application_cable/connection.rb
-- spec/dummy/app/channels/application_cable/channel.rb
+- spec/dummy/bin/rails
 - spec/dummy/bin/rake
 - spec/dummy/bin/setup
-- spec/dummy/bin/rails
-- spec/dummy/config/routes.rb
-- spec/dummy/config/locales/en.yml
+- spec/dummy/config/application.rb
+- spec/dummy/config/boot.rb
 - spec/dummy/config/cable.yml
-- spec/dummy/config/environments/production.rb
+- spec/dummy/config/database.yml
+- spec/dummy/config/environment.rb
 - spec/dummy/config/environments/development.rb
+- spec/dummy/config/environments/production.rb
 - spec/dummy/config/environments/test.rb
-- spec/dummy/config/environment.rb
-- spec/dummy/config/storage.yml
-- spec/dummy/config/application.rb
-- spec/dummy/config/puma.rb
-- spec/dummy/config/database.yml
-- spec/dummy/config/boot.rb
 - spec/dummy/config/initializers/application_controller_renderer.rb
 - spec/dummy/config/initializers/backtrace_silencers.rb
-- spec/dummy/config/initializers/mime_types.rb
+- spec/dummy/config/initializers/cors.rb
 - spec/dummy/config/initializers/filter_parameter_logging.rb
-- spec/dummy/config/initializers/wrap_parameters.rb
-- spec/dummy/config/initializers/redis.rb
 - spec/dummy/config/initializers/inflections.rb
-- spec/dummy/config/initializers/cors.rb
+- spec/dummy/config/initializers/mime_types.rb
+- spec/dummy/config/initializers/redis.rb
+- spec/dummy/config/initializers/wrap_parameters.rb
+- spec/dummy/config/locales/en.yml
+- spec/dummy/config/puma.rb
+- spec/dummy/config/routes.rb
+- spec/dummy/config/storage.yml
 - spec/dummy/config.ru
-- spec/dummy/Rakefile
 - spec/dummy/db/schema.rb
-- spec/dummy/log/test.log
 - spec/dummy/log/development.log
-- spec/dummy/tmp/pids/server.pid
+- spec/dummy/log/test.log
 - spec/dummy/tmp/development_secret.txt
-- spec/requests/health_check_spec.rb
+- spec/dummy/tmp/pids/server.pid
 - spec/lib/hey_doctor/hey_doctor_spec.rb
 - spec/rails_helper.rb
+- spec/requests/health_check_spec.rb
+- spec/services/hey_doctor/check_application_health_service_spec.rb
 - spec/services/hey_doctor/check_database_health_service_spec.rb
 - spec/services/hey_doctor/check_redis_health_service_spec.rb
-- spec/services/hey_doctor/check_application_health_service_spec.rb
+- spec/services/hey_doctor/check_sidekiq_health_service_spec.rb
+- spec/spec_helper.rb