Redirect tasks subcollection to request_tasks

config/routes.rb

@@ -61,6 +61,11 @@
 
         Array(collection.subcollections).each do |subcollection_name|
           Api::ApiConfig.collections[subcollection_name].verbs.each do |verb|
+            if subcollection_name == :tasks && verb == :get
+              get "/:c_id/#{subcollection_name}", :to => redirect { |path_params, _req| "/api/#{collection_name}/#{path_params[:c_id]}/request_tasks" }
+              get "/:c_id/#{subcollection_name}/:s_id", :to => redirect { |path_params, _req| "/api/#{collection_name}/#{path_params[:c_id]}/request_tasks/#{path_params[:s_id]}" }
+            end
+
             case verb
             when :get
               get "/:c_id/#{subcollection_name}", :action => :index

spec/requests/api/automation_requests_spec.rb

@@ -192,4 +192,40 @@
       expect(response).to have_http_status(:ok)
     end
   end
+
+  context 'Tasks subcollection' do
+    let(:automation_request) { FactoryGirl.create(:automation_request) }
+
+    it 'redirects to request_tasks subcollection' do
+      task = FactoryGirl.create(:miq_request_task, :miq_request_id => automation_request.id)
+      api_basic_authorize
+
+      run_get("#{automation_requests_url(automation_request.id)}/tasks")
+
+      expect(response).to have_http_status(:moved_permanently)
+      expect(response.redirect?).to be_truthy
+      expect(response.redirect_url).to include("#{automation_requests_url(automation_request.id)}/request_tasks")
+
+      run_get response.redirect_url, :expand => :resources
+
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body['resources'].first['id']).to eq(task.id)
+    end
+
+    it 'redirects to request_tasks subresources' do
+      task = FactoryGirl.create(:miq_request_task, :miq_request_id => automation_request.id)
+      api_basic_authorize
+
+      run_get("#{automation_requests_url(automation_request.id)}/tasks/#{task.id}")
+
+      expect(response).to have_http_status(:moved_permanently)
+      expect(response.redirect?).to be_truthy
+      expect(response.redirect_url).to include("#{automation_requests_url(automation_request.id)}/request_tasks/#{task.id}")
+
+      run_get response.redirect_url
+
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body['id']).to eq(task.id)
+    end
+  end
 end

spec/requests/api/service_requests_spec.rb

@@ -574,4 +574,38 @@ def expect_result_to_have_user_email(email)
       expect(response.parsed_body).to include(expected)
     end
   end
+
+  context 'Tasks subcollection' do
+    it 'redirects to request_tasks subcollection' do
+      task = FactoryGirl.create(:miq_request_task, :miq_request_id => service_request.id)
+      api_basic_authorize collection_action_identifier(:service_requests, :read, :get)
+
+      run_get("#{service_requests_url(service_request.id)}/tasks")
+
+      expect(response).to have_http_status(:moved_permanently)
+      expect(response.redirect?).to be_truthy
+      expect(response.redirect_url).to include("#{service_requests_url(service_request.id)}/request_tasks")
+
+      run_get response.redirect_url, :expand => :resources
+
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body["resources"].first["id"]).to eq(task.id)
+    end
+
+    it 'redirects to request_tasks subresources' do
+      task = FactoryGirl.create(:miq_request_task, :miq_request_id => service_request.id)
+      api_basic_authorize action_identifier(:services, :read, :resource_actions, :get)
+
+      run_get("#{service_requests_url(service_request.id)}/tasks/#{task.id}")
+
+      expect(response).to have_http_status(:moved_permanently)
+      expect(response.redirect?).to be_truthy
+      expect(response.redirect_url).to include("#{service_requests_url(service_request.id)}/request_tasks/#{task.id}")
+
+      run_get response.redirect_url
+
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body['id']).to eq(task.id)
+    end
+  end
 end