Merge branch 'cat-fix-trailing-slash-defaults' into 'master'

Change trailing slash from constraints to default for insights routes

See merge request gitlab-org/gitlab!73593

ee/config/routes/group.rb

@@ -71,7 +71,7 @@ constraints(::Constraints::GroupUrlConstrainer.new) do
 
     resource :issues_analytics, only: [:show]
 
-    resource :insights, only: [:show], trailing_slash: true do
+    resource :insights, only: [:show], defaults: { trailing_slash: true } do
       collection do
         post :query
       end

ee/config/routes/project.rb

@@ -154,7 +154,7 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
 
       post '/restore' => '/projects#restore', as: :restore
 
-      resource :insights, only: [:show], trailing_slash: true do
+      resource :insights, only: [:show], defaults: { trailing_slash: true } do
         collection do
           post :query
         end

ee/spec/controllers/groups/insights_controller_spec.rb

@@ -11,7 +11,7 @@ RSpec.describe Groups::InsightsController do
 
   let(:query_params) { { type: 'bar', query: { issuable_type: 'issue', collection_labels: ['bug'] }, projects: projects_params } }
   let(:projects_params) { { only: [project.id, project.full_path] } }
-  let(:params) { { trailing_slash: true } }
+  let(:params) { { group_id: parent_group } }
 
   before do
     stub_licensed_features(insights: true)

ee/spec/controllers/projects/insights_controller_spec.rb

@@ -10,7 +10,7 @@ RSpec.describe Projects::InsightsController do
 
   let(:query_params) { { type: 'bar', query: { issuable_type: 'issue', collection_labels: ['bug'] }, projects: projects_params } }
   let(:projects_params) { { only: [project.id, project.full_path] } }
-  let(:params) { { trailing_slash: true, project_id: project, namespace_id: group } }
+  let(:params) { { project_id: project, namespace_id: group } }
 
   before do
     stub_licensed_features(insights: true)