Code review analytics skeleton

Introduces neww page for
code review analytics app

app/views/layouts/nav/sidebar/_project.html.haml

@@ -290,6 +290,8 @@
 
       = render_if_exists 'layouts/nav/sidebar/project_packages_link'
 
+      = render_if_exists 'layouts/nav/sidebar/project_analytics_link' # EE-specific
+
       - if project_nav_tab? :wiki
         - wiki_url = project_wiki_path(@project, :home)
         = nav_link(controller: :wikis) do

ee/app/controllers/projects/analytics/code_reviews_controller.rb

+# frozen_string_literal: true
+
+module Projects
+  module Analytics
+    class CodeReviewsController < Projects::ApplicationController
+      before_action :check_code_review_analytics_available!
+
+      def index
+      end
+    end
+  end
+end

ee/app/helpers/ee/projects_helper.rb

@@ -59,6 +59,10 @@ module EE
         nav_tabs << :packages
       end
 
+      if ::Feature.enabled?(:code_review_analytics, project)
+        nav_tabs << :code_review
+      end
+
       if can?(current_user, :read_feature_flag, project) && !nav_tabs.include?(:operations)
         nav_tabs << :operations
       end

ee/app/models/license.rb

@@ -12,6 +12,7 @@ class License < ApplicationRecord
     audit_events
     burndown_charts
     code_owners
+    code_review_analytics
     contribution_analytics
     description_diffs
     elastic_search

ee/app/views/layouts/nav/sidebar/_project_analytics_link.html.haml

+- if Feature.enabled?(:code_review_analytics) && @project.feature_available?(:code_review_analytics, current_user)
+  - project_analytics_link = namespace_project_analytics_code_reviews_path(@project.namespace, @project)
+
+  = nav_link controller: :code_review do
+    = link_to project_analytics_link, data: { qa_selector: 'project_analytics_link' } do
+      .nav-icon-container
+        = sprite_icon('chart')
+      %span.nav-item-name
+        = _('Project Analytics')
+
+    %ul.sidebar-sub-level-items
+      = nav_link(controller: :code_review, html_options: { class: "fly-out-top-item" } ) do
+        = link_to project_analytics_link do
+          %strong.fly-out-top-item-name
+            = _('Project Analytics')
+
+      %li.divider.fly-out-top-item
+
+      - if project_nav_tab?(:code_review)
+        = nav_link controller: :code_review do
+          = link_to project_analytics_link, title: _('Code Review') do
+            %span= _('Code Review')

ee/app/views/projects/analytics/code_reviews/index.html.haml

+- breadcrumb_title _("Code Review")
+- page_title _("Code Review")
+
+.d-flex.align-items-center
+  %h1.page-title
+    = _('Code Review')
+.text-secondary= _('Review time is defined as the time it takes from first comment until merged.')
+#js-code-review-analytics{ data: { project: @project } }

ee/config/routes/project.rb

@@ -77,6 +77,12 @@ constraints(::Constraints::ProjectUrlConstrainer.new) do
         end
 
         resources :audit_events, only: [:index]
+
+        namespace :analytics do
+          constraints(::Constraints::FeatureConstrainer.new(:code_review_analytics)) do
+            resources :code_reviews, only: [:index]
+          end
+        end
       end
       # End of the /-/ scope.
 

ee/spec/requests/projects/analytics/code_reviews_controller_spec.rb

+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe Projects::Analytics::CodeReviewsController, type: :request do
+  let(:user) { create :user }
+  let(:project) { create(:project) }
+
+  before do
+    project.add_guest(user)
+    login_as user
+    stub_feature_flags(code_review_analytics: true)
+  end
+
+  describe 'GET /*namespace_id/:project_id/analytics/code_reviews' do
+    context 'with code_review_analytics included in plan' do
+      it 'is success' do
+        get project_analytics_code_reviews_path(project)
+
+        expect(response).to have_gitlab_http_status(200)
+      end
+    end
+
+    context 'without code_review_analytics in plan' do
+      before do
+        stub_licensed_features(code_review_analytics: false)
+      end
+
+      it 'is not found' do
+        get project_analytics_code_reviews_path(project)
+
+        expect(response).to have_gitlab_http_status(404)
+      end
+    end
+  end
+end

locale/gitlab.pot

@@ -4514,6 +4514,9 @@ msgstr ""
 msgid "Code Owners to the merge request changes."
 msgstr ""
 
+msgid "Code Review"
+msgstr ""
+
 msgid "Code owner approval is required"
 msgstr ""
 
@@ -13893,6 +13896,9 @@ msgstr ""
 msgid "Project '%{project_name}' will be deleted on %{date}"
 msgstr ""
 
+msgid "Project Analytics"
+msgstr ""
+
 msgid "Project Badges"
 msgstr ""
 
@@ -15556,6 +15562,9 @@ msgstr ""
 msgid "Review the process for configuring service providers in your identity provider — in this case, GitLab is the \"service provider\" or \"relying party\"."
 msgstr ""
 
+msgid "Review time is defined as the time it takes from first comment until merged."
+msgstr ""
+
 msgid "Reviewing"
 msgstr ""