Merge branch 'am-update-feature-category-top-slow-tests-100-200' into 'master'

Update feature_category for slow tests round 2

See merge request !109211



Merged-by: Pavel Shutsin <pshutsin@gitlab.com>
Approved-by: Smriti Garg <sgarg@gitlab.com>
Approved-by: Pavel Shutsin <pshutsin@gitlab.com>
Co-authored-by: Alina Mihaila <amihaila@gitlab.com>

ee/spec/lib/gitlab/git_access_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Gitlab::GitAccess do
+RSpec.describe Gitlab::GitAccess, feature_category: :authentication_and_authorization do
   include EE::GeoHelpers
   include AdminModeHelper
   include NamespaceStorageHelpers

ee/spec/models/ee/group_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Group do
+RSpec.describe Group, feature_category: :subgroups do
   using RSpec::Parameterized::TableSyntax
 
   let(:group) { create(:group) }

ee/spec/policies/merge_request_policy_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe MergeRequestPolicy do
+RSpec.describe MergeRequestPolicy, feature_category: :code_review_workflow do
   include ProjectForksHelper
   include AdminModeHelper
 

ee/spec/services/ee/notification_service_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe EE::NotificationService, :mailer do
+RSpec.describe EE::NotificationService, :mailer, feature_category: :team_planning do
   include EmailSpec::Matchers
   include NotificationHelpers
 

ee/spec/tasks/gitlab/elastic_rake_spec.rb

@@ -2,7 +2,8 @@
 
 require 'rake_helper'
 
-RSpec.describe 'gitlab:elastic namespace rake tasks', :elastic_clean, :silence_stdout do
+RSpec.describe 'gitlab:elastic namespace rake tasks', :elastic_clean, :silence_stdout,
+feature_category: :global_search do
   before do
     Rake.application.rake_require 'tasks/gitlab/elastic'
   end

spec/lib/gitlab/background_migration/backfill_snippet_repositories_spec.rb

@@ -2,7 +2,8 @@
 
 require 'spec_helper'
 
-RSpec.describe Gitlab::BackgroundMigration::BackfillSnippetRepositories, :migration, schema: 20210602155110 do
+RSpec.describe Gitlab::BackgroundMigration::BackfillSnippetRepositories, :migration, schema: 20210602155110,
+feature_category: :source_code_management do
   let(:gitlab_shell) { Gitlab::Shell.new }
   let(:users) { table(:users) }
   let(:snippets) { table(:snippets) }

spec/lib/gitlab/git_access_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Gitlab::GitAccess, :aggregate_failures do
+RSpec.describe Gitlab::GitAccess, :aggregate_failures, feature_category: :authentication_and_authorization do
   include TermsHelper
   include AdminModeHelper
   include ExternalAuthorizationServiceHelpers

spec/lib/gitlab/import_export/project/tree_restorer_spec.rb

@@ -6,7 +6,7 @@ def match_mr1_note(content_regex)
   MergeRequest.find_by(title: 'MR1').notes.find { |n| n.note.match(/#{content_regex}/) }
 end
 
-RSpec.describe Gitlab::ImportExport::Project::TreeRestorer do
+RSpec.describe Gitlab::ImportExport::Project::TreeRestorer, feature_category: :importers do
   include ImportExport::CommonUtil
   using RSpec::Parameterized::TableSyntax
 

spec/lib/gitlab/middleware/go_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Gitlab::Middleware::Go do
+RSpec.describe Gitlab::Middleware::Go, feature_category: :source_code_management do
   let(:app) { double(:app) }
   let(:middleware) { described_class.new(app) }
   let(:env) do

spec/models/concerns/ci/has_status_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Ci::HasStatus do
+RSpec.describe Ci::HasStatus, feature_category: :continuous_integration do
   describe '.composite_status' do
     using RSpec::Parameterized::TableSyntax
 

spec/models/concerns/noteable_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Noteable do
+RSpec.describe Noteable, feature_category: :code_review_workflow do
   let!(:active_diff_note1) { create(:diff_note_on_merge_request) }
   let(:project) { active_diff_note1.project }
   subject { active_diff_note1.noteable }

spec/models/deployment_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Deployment do
+RSpec.describe Deployment, feature_category: :continuous_delivery do
   subject { build(:deployment) }
 
   it { is_expected.to belong_to(:project).required }

spec/models/namespace_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Namespace do
+RSpec.describe Namespace, feature_category: :subgroups do
   include ProjectForksHelper
   include ReloadHelpers
 

spec/models/project_team_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-RSpec.describe ProjectTeam do
+RSpec.describe ProjectTeam, feature_category: :subgroups do
   include ProjectForksHelper
 
   let(:maintainer) { create(:user) }

spec/services/merge_requests/build_service_spec.rb

 # frozen_string_literal: true
 require 'spec_helper'
 
-RSpec.describe MergeRequests::BuildService do
+RSpec.describe MergeRequests::BuildService, feature_category: :code_review_workflow do
   using RSpec::Parameterized::TableSyntax
   include RepoHelpers
   include ProjectForksHelper

spec/services/quick_actions/interpret_service_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe QuickActions::InterpretService do
+RSpec.describe QuickActions::InterpretService, feature_category: :team_planning do
   include AfterNextHelpers
 
   let_it_be(:group) { create(:group, :crm_enabled) }

spec/services/todos/destroy/entity_leave_service_spec.rb

@@ -2,7 +2,7 @@
 
 require 'spec_helper'
 
-RSpec.describe Todos::Destroy::EntityLeaveService do
+RSpec.describe Todos::Destroy::EntityLeaveService, feature_category: :team_planning do
   let_it_be(:user, reload: true) { create(:user) }
   let_it_be(:user2, reload: true) { create(:user) }
   let_it_be_with_refind(:group) { create(:group, :private) }