suzanne.soy/travis-api
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'master' into cd-v3

Browse Source
This commit is contained in:
carlad 2016-03-11 11:36:19 +01:00
commit 7a7e957b80
8 changed files with 377 additions and 262 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Gemfile.lock
View File

@ -50,7 +50,7 @@ GIT
GIT
remote: git://github.com/travis-ci/travis-core.git
revision: 96ee8c449ebe305c5c95633cea13eb88fe978abb
revision: fb39f8af5628444e2d7f5893a9f09fde7b0796e2
specs:
travis-core (0.0.1)
actionmailer (~> 3.2.19)
@ -199,9 +199,9 @@ GEM
httparty (0.11.0)
multi_json (~> 1.0)
multi_xml (>= 0.5.2)
httpclient (2.7.0.1)
httpclient (2.7.1)
i18n (0.7.0)
ice_nine (0.11.1)
ice_nine (0.11.2)
jemalloc (1.0.1)
journey (1.0.4)
json (1.8.3)

1
lib/travis/api/v3/models/job.rb
View File

@ -9,5 +9,6 @@ module Travis::API::V3
belongs_to :build, autosave: true, foreign_key: 'source_id'
belongs_to :owner, polymorphic: true
serialize :config
serialize :debug_options
end
end

4
lib/travis/api/v3/permissions/job.rb
View File

@ -9,5 +9,9 @@ module Travis::API::V3
def restart?
write?
end
def debug?
write?
end
end
end

1
lib/travis/api/v3/routes.rb
View File

@ -34,6 +34,7 @@ module Travis::API::V3
post :cancel, '/cancel'
post :restart, '/restart'
post :debug, '/debug'
end
resource :lint do

29
lib/travis/api/v3/services/job/debug.rb Normal file
View File

@ -0,0 +1,29 @@
module Travis::API::V3
class Services::Job::Debug < Service
params "quiet"
attr_reader :job
def run
raise LoginRequired unless access_control.logged_in? or access_control.full_access?
raise NotFound unless @job = find(:job)
raise WrongCredentials unless Travis.config.debug_tools_enabled or Travis::Features.active?(:debug_tools, job.repository)
access_control.permissions(job).debug!
job.debug_options = debug_data
job.save!
query.restart(access_control.user)
accepted(job: job, state_change: :created)
end
def debug_data
{
stage: 'before_install',
previous_state: job.state,
created_by: access_control.user.login,
quiet: params["quiet"] || false
}
end
end
end

70
spec/v3/services/job/debug_sepc.rb Normal file
View File

@ -0,0 +1,70 @@
require 'spec_helper'
describe Travis::API::V3::Services::Job::Debug do
let(:repo) { Travis::API::V3::Models::Repository.where(owner_name: 'svenfuchs', name: 'minimal').first }
let(:owner_type) { repo.owner_type.constantize }
let(:owner) { owner_type.find(repo.owner_id)}
let(:build) { repo.builds.last }
let(:jobs) { Travis::API::V3::Models::Build.find(build.id).jobs }
let(:job) { jobs.last }
before { repo.requests.each(&:delete) }
before do
Travis::Features.stubs(:owner_active?).returns(true)
@original_sidekiq = Sidekiq::Client
Sidekiq.send(:remove_const, :Client) # to avoid a warning
Sidekiq::Client = []
Travis.config.stubs(:debug_tools_enabled).returns true
end
after do
Sidekiq.send(:remove_const, :Client) # to avoid a warning
Sidekiq::Client = @original_sidekiq
end
describe "#run" do
context "when unauthenticated" do
before { post("/v3/job/#{job.id}/debug") }
example { expect(last_response.status).to be == 403 }
example { expect(JSON.load(body)).to be == {
"@type" => "error",
"error_type" => "login_required",
"error_message" => "login required"
}}
end
context "when authenticated" do
let(:token) { Travis::Api::App::AccessToken.create(user: repo.owner, app_id: 1) }
let(:headers) {{ 'HTTP_AUTHORIZATION' => "token #{token}" }}
context "without sufficient authorization" do
before { post("/v3/job/#{job.id}/debug", {}, headers) }
example { expect(last_response.status).to be == 403 }
example { expect(JSON.load(body)).to include(
"@type" => "error",
"error_type" => "insufficient_access",
"error_message" => "operation requires debug access to job",
"resource_type" => "job",
)}
end
context "with sufficient authorization" do
let(:params) {{}}
before { Travis::API::V3::Models::Permission.create(repository: repo, user: repo.owner, push: true) }
before { post("/v3/job/#{job.id}/debug", {}, headers) }
example { expect(last_response.status).to be == 202 }
example { expect(job.reload.debug_options).to include(
stage: "before_install",
created_by: owner.login,
quiet: false
) }
end
end
end
end

6
spec/v3/services/job/find_spec.rb
View File

@ -23,7 +23,8 @@ describe Travis::API::V3::Services::Job::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => job.id,
"number" => job.number,
"state" => job.state,
@ -93,7 +94,8 @@ describe Travis::API::V3::Services::Job::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => job.id,
"number" => job.number,
"state" => job.state,

24
spec/v3/services/jobs/find_spec.rb
View File

@ -21,7 +21,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[0].id,
"number" => "#{jobs[0].number}",
"state" => "configured",
@ -68,7 +69,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[1].id,
"number" => "#{jobs[1].number}",
"state" => "configured",
@ -115,7 +117,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[2].id,
"number" => "#{jobs[2].number}",
"state" => "configured",
@ -162,7 +165,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[3].id,
"number" => "#{jobs[3].number}",
"state" => "configured",
@ -227,7 +231,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[0].id,
"number" => "#{jobs[0].number}",
"state" => "configured",
@ -274,7 +279,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[1].id,
"number" => "#{jobs[1].number}",
"state" => "configured",
@ -321,7 +327,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[2].id,
"number" => "#{jobs[2].number}",
"state" => "configured",
@ -368,7 +375,8 @@ describe Travis::API::V3::Services::Jobs::Find do
"@permissions" => {
"read" => true,
"cancel" => false,
"restart" => false },
"restart" => false,
"debug" => false },
"id" => jobs[3].id,
"number" => "#{jobs[3].number}",
"state" => "configured",