Add an equeue service

This commit is contained in:
Ana Rosas 2016-05-18 14:45:42 -05:00
parent 28884f9931
commit 9bbfb7ec97
5 changed files with 30 additions and 18 deletions

View File

@ -1,6 +1,6 @@
require 'travis/api/app' require 'travis/api/app'
require 'travis/api/workers/build_cancellation' require 'travis/api/workers/build_cancellation'
require 'travis/api/workers/build_restart' require 'travis/api/enqueue/services/enqueue_build'
class Travis::Api::App class Travis::Api::App
class Endpoint class Endpoint
@ -54,12 +54,8 @@ class Travis::Api::App
status 400 status 400
result = false result = false
else else
payload = {id: params[:id], user_id: current_user.id}
::Sidekiq::Client.push( Travis::Enqueue::Services::EnqueueBuild.push("build:restart", payload)
'queue' => 'hub',
'class' => 'Travis::Hub::Sidekiq::Worker',
'args' => ["build:restart", {id: params[:id], user_id: current_user.id}]
)
status 202 status 202
result = true result = true

View File

@ -0,0 +1,18 @@
module Travis
module Enqueue
module Services
class EnqueueBuild
def self.push(event, payload)
::Sidekiq::Client.push(
'queue' => 'hub',
'class' => 'Travis::Hub::Sidekiq::Worker',
'args' => [event, payload]
)
end
end
end
end
end

View File

@ -4,15 +4,14 @@ require 'multi_json'
module Travis module Travis
module Sidekiq module Sidekiq
class BuildRestart class BuildRestart
include ::Sidekiq::Worker class ProcessingError < StandardError; end
sidekiq_options queue: :hub
def perform(payload) include ::Sidekiq::Worker
::Sidekiq::Client.push( sidekiq_options queue: :build_restarts
'queue' => 'hub',
'class' => 'Travis::Hub::Sidekiq::Worker', def perform(data)
'args' => ["build:restart", payload] user = User.find(data['user_id'])
) Travis.service(:reset_model, user, build_id: data['id']).run
end end
end end

View File

@ -2,7 +2,6 @@
require 'sidekiq' require 'sidekiq'
require 'travis' require 'travis'
require 'travis/api/workers/build_cancellation' require 'travis/api/workers/build_cancellation'
require 'travis/api/workers/build_restart'
require 'travis/api/workers/job_cancellation' require 'travis/api/workers/job_cancellation'
require 'travis/api/workers/job_restart' require 'travis/api/workers/job_restart'
require 'travis/support/amqp' require 'travis/support/amqp'

View File

@ -119,13 +119,13 @@ describe 'Builds' do
end end
it 'restarts the build' do it 'restarts the build' do
Travis::Sidekiq::BuildRestart.expects(:perform_async).with(id: build.id.to_s, user_id: user.id) Travis::Enqueue::Services::EnqueueBuild.expects(:push).with("build:restart", {id: build.id.to_s, user_id: user.id})
response = post "/builds/#{build.id}/restart", {}, headers response = post "/builds/#{build.id}/restart", {}, headers
response.status.should == 202 response.status.should == 202
end end
it 'sends the correct response body' do it 'sends the correct response body' do
Travis::Sidekiq::BuildRestart.expects(:perform_async).with(id: build.id.to_s, user_id: user.id) Travis::Enqueue::Services::EnqueueBuild.expects(:push).with("build:restart", {id: build.id.to_s, user_id: user.id})
response = post "/builds/#{build.id}/restart", {}, headers response = post "/builds/#{build.id}/restart", {}, headers
body = JSON.parse(response.body) body = JSON.parse(response.body)
body.should == {"result"=>true, "flash"=>[{"notice"=>"The build was successfully restarted."}]} body.should == {"result"=>true, "flash"=>[{"notice"=>"The build was successfully restarted."}]}