diff --git a/lib/travis/api/v2/http/request.rb b/lib/travis/api/v2/http/request.rb index ca95326c..709512e5 100644 --- a/lib/travis/api/v2/http/request.rb +++ b/lib/travis/api/v2/http/request.rb @@ -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 diff --git a/lib/travis/api/v2/http/requests.rb b/lib/travis/api/v2/http/requests.rb index b1c104e8..d55fe8b4 100644 --- a/lib/travis/api/v2/http/requests.rb +++ b/lib/travis/api/v2/http/requests.rb @@ -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) diff --git a/spec/unit/api/v2/http/request_spec.rb b/spec/unit/api/v2/http/request_spec.rb index a4643cc4..0a4eec87 100644 --- a/spec/unit/api/v2/http/request_spec.rb +++ b/spec/unit/api/v2/http/request_spec.rb @@ -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 diff --git a/spec/unit/api/v2/http/requests_spec.rb b/spec/unit/api/v2/http/requests_spec.rb index e6685484..f783f8d6 100644 --- a/spec/unit/api/v2/http/requests_spec.rb +++ b/spec/unit/api/v2/http/requests_spec.rb @@ -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 }