Merge pull request #298 from travis-ci/ar-fix-requests-endpoint

Fixes /requests endpoint

lib/travis/api/app/endpoint/requests.rb

@@ -1,5 +1,6 @@
 require 'travis/api/app'
 require 'travis/api/app/services/schedule_request'
+require 'travis/api/enqueue/services/restart_model'
 
 class Travis::Api::App
   class Endpoint
@@ -35,10 +36,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}
+            payload = { id: params[:build_id], user_id: repository_owner.id }
-            service.push("job:restart", payload)
+            service.push("build:restart", payload)
-            status 202
+
-            true
+            respond_with(result: true, flash: service.messages)
           end
         end
       end

spec/integration/v2/requests_spec.rb

@@ -33,11 +33,12 @@ describe 'Requests', set_app: true do
       response.status.should be(200)
     end
 
-    it 'triggers a build request using Hub' do
+    describe 'enqueues for Hub' do
-      Travis::Features.activate_owner(:enqueue_to_hub, repo.owner)
+      it 'triggers a build request' do
-
+        payload = { build_id: build.id, user_id: repo.owner.id }
-      response = post "/requests", { build_id: build.id }, headers
+        response = post "/requests", payload, headers
-      response.status.should be(202)
+        response.status.should be(200)
+      end
     end
   end
 end