Merge pull request #179 from travis-ci/pass-token-to-request-create
Include token in requests/create payload
This commit is contained in:
commit
2ce1220ef5
|
@ -1,6 +1,6 @@
|
|||
module Travis::API::V3
|
||||
class Queries::Request < Query
|
||||
params :message, :branch, :config, prefix: :request
|
||||
params :message, :branch, :config, :token, prefix: :request
|
||||
|
||||
def schedule(repository, user)
|
||||
raise ServerError, 'repository does not have a github_id'.freeze unless repository.github_id
|
||||
|
@ -14,7 +14,7 @@ module Travis::API::V3
|
|||
config: config || {}
|
||||
}
|
||||
|
||||
perform_async(:build_request, type: 'api'.freeze, credentials: {}, payload: JSON.dump(payload))
|
||||
perform_async(:build_request, type: 'api'.freeze, credentials: { token: token }, payload: JSON.dump(payload))
|
||||
payload
|
||||
end
|
||||
end
|
||||
|
|
|
@ -5,7 +5,7 @@ module Travis::API::V3
|
|||
private_constant :TIME_FRAME, :LIMIT
|
||||
|
||||
result_type :request
|
||||
params "request", "user", :config, :message, :branch
|
||||
params "request", "user", :config, :message, :branch, :token
|
||||
|
||||
def run
|
||||
raise LoginRequired unless access_control.logged_in? or access_control.full_access?
|
||||
|
|
|
@ -3,6 +3,7 @@ require 'spec_helper'
|
|||
describe Travis::API::V3::Services::Requests::Create do
|
||||
let(:repo) { Travis::API::V3::Models::Repository.where(owner_name: 'svenfuchs', name: 'minimal').first }
|
||||
let(:sidekiq_payload) { JSON.load(Sidekiq::Client.last['args'].last[:payload]).deep_symbolize_keys }
|
||||
let(:sidekiq_params) { Sidekiq::Client.last['args'].last.deep_symbolize_keys }
|
||||
before { repo.requests.each(&:delete) }
|
||||
|
||||
before do
|
||||
|
@ -231,6 +232,13 @@ describe Travis::API::V3::Services::Requests::Create do
|
|||
"repository" => {"@type"=>"repository", "@href"=>"/repo/#{repo.id}", "id"=>repo.id, "slug"=>"svenfuchs/minimal" }
|
||||
}}
|
||||
end
|
||||
|
||||
describe "passing the token in params" do
|
||||
let(:params) {{ request: { token: 'foo-bar' }}}
|
||||
example { expect(sidekiq_params[:credentials]).to be == {
|
||||
token: 'foo-bar'
|
||||
}}
|
||||
end
|
||||
end
|
||||
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user