Execution Hanging Detection

Fixing unit tests

Signed-off-by: Rodrigo Nardi <rnardi@netdef.org>

Author: RodrigoMNardi

spec/lib/github/update_status_spec.rb

@@ -16,6 +16,7 @@
   before do
     allow(Github::PlanExecution::Finished).to receive(:new).and_return(fake_finish_plan)
     allow(fake_finish_plan).to receive(:fetch_build_status)
+    allow(TimeoutExecution).to receive_message_chain(:delay, :timeout).and_return(true)
   end
 
   describe 'Validates different Ci Job status' do

spec/spec_helper.rb

@@ -39,6 +39,8 @@ def app
   config.include FactoryBot::Syntax::Methods
   config.include WebMock::API
 
+  config.add_formatter('json', 'tmp/rspec_results.json')
+
   pid = nil
 
   config.before(:suite) do

spec/workers/timeout_execution_spec.rb

@@ -73,17 +73,4 @@
       expect(described_class.timeout(check_suite.id)).to be_falsey
     end
   end
-
-  # context 'when timeout is called and rescheduled' do
-  #   let(:check_suite) { create(:check_suite) }
-  #
-  #   before do
-  #     allow(CheckSuite).to receive(:find).and_return(check_suite)
-  #     allow(check_suite).to receive(:finished?).and_return(true)
-  #   end
-  #
-  #   it 'calls timeout job' do
-  #     expect(described_class.timeout(check_suite.id)).to be_falsey
-  #   end
-  # end
 end

workers/timeout_execution.rb

@@ -23,10 +23,10 @@ def timeout(check_suite_id)
 
       @logger.info("Calling Github::PlanExecution::Finished.new(#{check_suite.bamboo_ci_ref}).finished")
 
-      rescheduling(finished, check_suite_id)
+      rescheduling(finished(check_suite), check_suite_id)
     end
 
-    def finished
+    def finished(check_suite)
       Github::PlanExecution::Finished
         .new({ 'bamboo_ref' => check_suite.bamboo_ci_ref, hanged: true })
         .finished