hey_doctor 1.2.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/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,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hey_doctor
 version: !ruby/object:Gem::Version
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors:
 - Matheus Acosta
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-03-25 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
@@ -123,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
@@ -176,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:
@@ -252,4 +268,5 @@ test_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