Merge branch 'da-rubocop-whitelist-active-record-none-ee' into 'master'

Port of da-rubocop-whitelist-active-record-none to EE

See merge request gitlab-org/gitlab-ee!8171

app/finders/autocomplete/users_finder.rb

@@ -74,7 +74,6 @@ module Autocomplete
       author_id.present? && current_user
     end
 
-    # rubocop: disable CodeReuse/ActiveRecord
     def find_users
       if project
         project.authorized_users.union_with_user(author_id)
@@ -86,6 +85,5 @@ module Autocomplete
         User.none
       end
     end
-    # rubocop: enable CodeReuse/ActiveRecord
   end
 end

app/finders/concerns/finder_with_cross_project_access.rb

@@ -16,7 +16,6 @@ module FinderWithCrossProjectAccess
   end
 
   override :execute
-  # rubocop: disable CodeReuse/ActiveRecord
   def execute(*args)
     check = Gitlab::CrossProjectAccess.find_check(self)
     original = super
@@ -30,7 +29,6 @@ module FinderWithCrossProjectAccess
       original
     end
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   # We can skip the cross project check for finding indivitual records.
   # this would be handled by the `can?(:read_*, result)` call in `FinderMethods`

app/finders/group_descendants_finder.rb

@@ -131,7 +131,6 @@ class GroupDescendantsFinder
       .with_selects_for_list(archived: params[:archived])
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def subgroups
     return Group.none unless Group.supports_nested_groups?
 
@@ -145,7 +144,6 @@ class GroupDescendantsFinder
 
     groups.with_selects_for_list(archived: params[:archived]).order_by(sort)
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   # rubocop: disable CodeReuse/Finder
   def direct_child_projects

app/finders/groups_finder.rb

@@ -40,7 +40,6 @@ class GroupsFinder < UnionFinder
 
   attr_reader :current_user, :params
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def all_groups
     return [owned_groups] if params[:owned]
     return [groups_with_min_access_level] if min_access_level?
@@ -52,7 +51,6 @@ class GroupsFinder < UnionFinder
     groups << Group.none if groups.empty?
     groups
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def groups_for_ancestors
     current_user.authorized_groups
@@ -82,11 +80,9 @@ class GroupsFinder < UnionFinder
   end
   # rubocop: enable CodeReuse/ActiveRecord
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def owned_groups
     current_user&.owned_groups || Group.none
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def include_public_groups?
     current_user.nil? || all_available?

app/finders/issuable_finder.rb

@@ -196,7 +196,6 @@ class IssuableFinder
     milestones? && params[:milestone_title] == Milestone::None.title
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def milestones
     return @milestones if defined?(@milestones)
 
@@ -217,7 +216,6 @@ class IssuableFinder
         Milestone.none
       end
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def labels?
     params[:label_name].present?
@@ -227,7 +225,6 @@ class IssuableFinder
     labels? && params[:label_name].include?(Label::None.title)
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def labels
     return @labels if defined?(@labels)
 
@@ -238,7 +235,6 @@ class IssuableFinder
         Label.none
       end
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def assignee_id?
     params[:assignee_id].present?

app/finders/issues_finder.rb

@@ -137,7 +137,6 @@ class IssuesFinder < IssuableFinder
     current_user.blank?
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def by_assignee(items)
     if filter_by_no_assignee?
       items.unassigned
@@ -151,5 +150,4 @@ class IssuesFinder < IssuableFinder
       items
     end
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 end

app/finders/labels_finder.rb

@@ -12,7 +12,6 @@ class LabelsFinder < UnionFinder
     @params = params
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def execute(skip_authorization: false)
     @skip_authorization = skip_authorization
     items = find_union(label_ids, Label) || Label.none
@@ -21,7 +20,6 @@ class LabelsFinder < UnionFinder
     items = by_search(items)
     sort(items)
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   private
 

app/finders/milestones_finder.rb

@@ -20,7 +20,6 @@ class MilestonesFinder
     @params = params
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def execute
     return Milestone.none if project_ids.empty? && group_ids.empty?
 
@@ -31,7 +30,6 @@ class MilestonesFinder
 
     order(items)
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   private
 

app/finders/pipelines_finder.rb

@@ -12,7 +12,6 @@ class PipelinesFinder
     @params = params
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def execute
     unless Ability.allowed?(current_user, :read_pipeline, project)
       return Ci::Pipeline.none
@@ -28,7 +27,6 @@ class PipelinesFinder
     items = by_yaml_errors(items)
     sort_items(items)
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   private
 

app/finders/projects_finder.rb

@@ -88,7 +88,6 @@ class ProjectsFinder < UnionFinder
   # rubocop: enable CodeReuse/ActiveRecord
 
   # Builds a collection for an anonymous user.
-  # rubocop: disable CodeReuse/ActiveRecord
   def collection_without_user
     if private_only? || owned_projects? || min_access_level?
       Project.none
@@ -96,7 +95,6 @@ class ProjectsFinder < UnionFinder
       Project.public_to_user
     end
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   def owned_projects?
     params[:owned].present?

app/finders/snippets_finder.rb

@@ -43,7 +43,6 @@ class SnippetsFinder < UnionFinder
     end
   end
 
-  # rubocop: disable CodeReuse/ActiveRecord
   def authorized_snippets_from_project
     if can?(current_user, :read_project_snippet, project)
       if project.team.member?(current_user)
@@ -55,7 +54,6 @@ class SnippetsFinder < UnionFinder
       Snippet.none
     end
   end
-  # rubocop: enable CodeReuse/ActiveRecord
 
   # rubocop: disable CodeReuse/ActiveRecord
   def authorized_snippets

app/services/search/group_service.rb

@@ -11,14 +11,12 @@ module Search
       @group = group
     end
 
-    # rubocop: disable CodeReuse/ActiveRecord
     def projects
       return Project.none unless group
       return @projects if defined? @projects
 
       @projects = super.inside_path(group.full_path)
     end
-    # rubocop: enable CodeReuse/ActiveRecord
 
     # rubocop: disable CodeReuse/ActiveRecord
     def elastic_projects

ee/app/finders/autocomplete/project_invited_groups_finder.rb

@@ -16,7 +16,6 @@ module Autocomplete
       @params = params
     end
 
-    # rubocop: disable CodeReuse/ActiveRecord
     # rubocop: disable CodeReuse/Finder
     def execute
       project = ::Autocomplete::ProjectFinder
@@ -26,6 +25,5 @@ module Autocomplete
       project ? project.invited_groups : Group.none
     end
     # rubocop: enable CodeReuse/Finder
-    # rubocop: enable CodeReuse/ActiveRecord
   end
 end

ee/lib/gitlab/code_owners.rb

@@ -9,7 +9,7 @@ module Gitlab
       if blob.project.feature_available?(:code_owners)
         Loader.new(blob.project, blob.commit_id, blob.path).members
       else
-        User.none # rubocop: disable CodeReuse/ActiveRecord
+        User.none
       end
     end
   end

ee/lib/gitlab/code_owners/loader.rb

@@ -16,7 +16,7 @@ module Gitlab
       end
 
       def raw_users
-        return User.none if empty_code_owners? # rubocop: disable CodeReuse/ActiveRecord
+        return User.none if empty_code_owners?
 
         @_raw_users ||= begin
           owner_lines = @paths.map { |path| code_owners_file.owners_for_path(path) }

lib/gitlab/user_extractor.rb

@@ -14,13 +14,11 @@ module Gitlab
       @text = text.is_a?(Array) ? text.join(' ') : text
     end
 
-    # rubocop: disable CodeReuse/ActiveRecord
     def users
       return User.none unless @text.present?
 
       @users ||= User.from_union(union_relations)
     end
-    # rubocop: enable CodeReuse/ActiveRecord
 
     def usernames
       matches[:usernames]

rubocop/cop/code_reuse/active_record.rb

@@ -49,7 +49,6 @@ module RuboCop
           limit: true,
           lock: false,
           many?: false,
-          none: false,
           offset: true,
           order: true,
           pluck: true,