From 564e95be6b177057ff37e8d3b2e948edb01d7a9a Mon Sep 17 00:00:00 2001 From: Ana Rosas Date: Mon, 11 Jul 2016 13:35:17 -0500 Subject: [PATCH] Use params instead of payload and return error message --- lib/travis/api/app/endpoint/requests.rb | 8 ++++---- spec/integration/v2/requests_spec.rb | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/lib/travis/api/app/endpoint/requests.rb b/lib/travis/api/app/endpoint/requests.rb index 590ec331..56464de3 100644 --- a/lib/travis/api/app/endpoint/requests.rb +++ b/lib/travis/api/app/endpoint/requests.rb @@ -35,10 +35,10 @@ class Travis::Api::App if !Travis::Features.enabled_for_all?(:enqueue_to_hub) && !Travis::Features.owner_active?(:enqueue_to_hub, repository_owner) respond_with service(:reset_model, params) elsif service.respond_to?(:push) - payload = {id: params[:build_id], user_id: current_user.id} - service.push("job:restart", payload) - status 202 - true + payload = { build_id: params[:build_id], user_id: current_user.id } + service.push("job:restart", params) + + respond_with(result: true, flash: service.messages) end end end diff --git a/spec/integration/v2/requests_spec.rb b/spec/integration/v2/requests_spec.rb index cd023599..cc6ce1e8 100644 --- a/spec/integration/v2/requests_spec.rb +++ b/spec/integration/v2/requests_spec.rb @@ -37,7 +37,7 @@ describe 'Requests', set_app: true do Travis::Features.activate_owner(:enqueue_to_hub, repo.owner) response = post "/requests", { build_id: build.id }, headers - response.status.should be(202) + response.status.should be(200) end end end