Add build_id to requests payload
This commit is contained in:
parent
5506a8143b
commit
d02648a542
|
@ -25,7 +25,7 @@ module Travis
|
|||
private
|
||||
|
||||
def request_data
|
||||
{
|
||||
data = {
|
||||
'id' => request.id,
|
||||
'repository_id' => request.repository_id,
|
||||
'commit_id' => request.commit_id,
|
||||
|
@ -43,6 +43,10 @@ module Travis
|
|||
'branch' => request.branch_name,
|
||||
'tag' => request.tag_name
|
||||
}
|
||||
|
||||
data['build_id'] = request.builds.first.id if request.builds.present?
|
||||
|
||||
data
|
||||
end
|
||||
|
||||
def commit_data
|
||||
|
|
|
@ -22,7 +22,7 @@ module Travis
|
|||
private
|
||||
|
||||
def request_data(request)
|
||||
{
|
||||
data = {
|
||||
'id' => request.id,
|
||||
'repository_id' => request.repository_id,
|
||||
'commit_id' => request.commit_id,
|
||||
|
@ -40,6 +40,10 @@ module Travis
|
|||
'branch' => request.branch_name,
|
||||
'tag' => request.tag_name
|
||||
}
|
||||
|
||||
data['build_id'] = request.builds.first.id if request.builds.present?
|
||||
|
||||
data
|
||||
end
|
||||
|
||||
def commit_data(commit)
|
||||
|
|
|
@ -4,6 +4,9 @@ describe Travis::Api::V2::Http::Request do
|
|||
include Travis::Testing::Stubs, Support::Formats
|
||||
|
||||
let(:data) {
|
||||
request = stub_request
|
||||
request.stubs(:builds).returns([build])
|
||||
request.stubs(:tag_name).returns(nil)
|
||||
Travis::Api::V2::Http::Request.new(request).data
|
||||
}
|
||||
|
||||
|
@ -24,7 +27,8 @@ describe Travis::Api::V2::Http::Request do
|
|||
'tag' => nil,
|
||||
'pull_request' => false,
|
||||
'pull_request_title' => nil,
|
||||
'pull_request_number' => nil
|
||||
'pull_request_number' => nil,
|
||||
'build_id' => build.id
|
||||
}
|
||||
end
|
||||
|
||||
|
|
|
@ -3,7 +3,14 @@ require 'spec_helper'
|
|||
describe Travis::Api::V2::Http::Requests do
|
||||
include Travis::Testing::Stubs, Support::Formats
|
||||
|
||||
before do
|
||||
request.stubs(:builds).returns([build])
|
||||
end
|
||||
|
||||
let(:data) {
|
||||
request = stub_request
|
||||
request.stubs(:builds).returns([build])
|
||||
request.stubs(:tag_name).returns(nil)
|
||||
Travis::Api::V2::Http::Requests.new([request]).data
|
||||
}
|
||||
|
||||
|
@ -25,7 +32,8 @@ describe Travis::Api::V2::Http::Requests do
|
|||
'tag' => nil,
|
||||
'pull_request' => false,
|
||||
'pull_request_title' => nil,
|
||||
'pull_request_number' => nil
|
||||
'pull_request_number' => nil,
|
||||
'build_id' => build.id
|
||||
}
|
||||
]
|
||||
end
|
||||
|
@ -51,6 +59,7 @@ describe Travis::Api::V2::Http::Requests do
|
|||
let(:data) {
|
||||
request = stub_request
|
||||
request.stubs(:commit).returns(nil)
|
||||
request.stubs(:builds).returns([build])
|
||||
Travis::Api::V2::Http::Requests.new([request]).data
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue
Block a user