Redirects if scope is not valid

app/controllers/projects/pipelines_controller.rb

@@ -19,6 +19,8 @@ class Projects::PipelinesController < Projects::ApplicationController
   end
   before_action :ensure_pipeline, only: [:show]
 
+  before_action :ensure_valid_scope, only: [:index]
+
   around_action :allow_gitaly_ref_name_caching, only: [:index, :show]
 
   track_unique_visits :charts, target_id: 'p_analytics_pipelines'
@@ -44,7 +46,7 @@ class Projects::PipelinesController < Projects::ApplicationController
         render json: {
           pipelines: serialize_pipelines,
           count: {
-            all: @pipelines_count,
+            all: @pipelines_count
           }
         }
       end
@@ -223,6 +225,10 @@ class Projects::PipelinesController < Projects::ApplicationController
     render_404 unless pipeline
   end
 
+  def ensure_valid_scope
+    redirect_to project_pipelines_path(project, status: params[:scope]) if %w[running pending].include?(params[:scope])
+  end
+
   # rubocop: disable CodeReuse/ActiveRecord
   def pipeline
     @pipeline ||= if params[:id].blank? && params[:latest]

spec/controllers/projects/pipelines_controller_spec.rb

@@ -119,6 +119,14 @@ RSpec.describe Projects::PipelinesController do
       end
     end
 
+    context 'when scope is pending or running' do
+      it 'redirects to all pipelines with that status instead' do
+        get_pipelines_index_json(scope: 'running')
+
+        expect(response).to redirect_to(project_pipelines_path(project, status: 'running'))
+      end
+    end
+
     context 'filter by scope' do
       context 'scope is branches or tags' do
         before do
@@ -192,14 +200,6 @@ RSpec.describe Projects::PipelinesController do
 
           check_pipeline_response(returned: 1, all: 1)
         end
-
-        context 'when filter by unrelated scope' do
-          it 'returns empty list' do
-            get_pipelines_index_json(status: 'success', scope: 'running')
-
-            check_pipeline_response(returned: 0, all: 1)
-          end
-        end
       end
 
       context 'when no pipeline with the status exists' do