Commit 07ba1be0 authored by Thong Kuah's avatar Thong Kuah

Merge branch '334860-remove-un-used-method-with_active_integrations' into 'master'

Remove unused method with_active_integrations

See merge request gitlab-org/gitlab!65131
parents 71595bbf 0652247a
...@@ -144,7 +144,6 @@ module EE ...@@ -144,7 +144,6 @@ module EE
scope :for_plan_name, -> (name) { joins(namespace: { gitlab_subscription: :hosted_plan }).where(plans: { name: name }) } scope :for_plan_name, -> (name) { joins(namespace: { gitlab_subscription: :hosted_plan }).where(plans: { name: name }) }
scope :requiring_code_owner_approval, scope :requiring_code_owner_approval,
-> { joins(:protected_branches).where(protected_branches: { code_owner_approval_required: true }) } -> { joins(:protected_branches).where(protected_branches: { code_owner_approval_required: true }) }
scope :with_active_integrations, -> { joins(:integrations).merge(::Integration.active) }
scope :github_imported, -> { where(import_type: 'github') } scope :github_imported, -> { where(import_type: 'github') }
scope :with_protected_branches, -> { joins(:protected_branches) } scope :with_protected_branches, -> { joins(:protected_branches) }
scope :with_repositories_enabled, -> { joins(:project_feature).where(project_features: { repository_access_level: ::ProjectFeature::ENABLED }) } scope :with_repositories_enabled, -> { joins(:project_feature).where(project_features: { repository_access_level: ::ProjectFeature::ENABLED }) }
......
...@@ -281,16 +281,6 @@ RSpec.describe Project do ...@@ -281,16 +281,6 @@ RSpec.describe Project do
end end
end end
describe '.with_active_integrations' do
it 'returns the correct project' do
active = create(:service, active: true)
inactive = create(:service, active: false)
expect(described_class.with_active_integrations).to include(active.project)
expect(described_class.with_active_integrations).not_to include(inactive.project)
end
end
describe '.github_imported' do describe '.github_imported' do
it 'returns the correct project' do it 'returns the correct project' do
project_imported_from_github = create(:project, :github_imported) project_imported_from_github = create(:project, :github_imported)
......
...@@ -18,7 +18,7 @@ ...@@ -18,7 +18,7 @@
# batch_count(::Clusters::Cluster.aws_installed.enabled, :cluster_id) # batch_count(::Clusters::Cluster.aws_installed.enabled, :cluster_id)
# batch_count(Namespace.group(:type)) # batch_count(Namespace.group(:type))
# batch_distinct_count(::Project, :creator_id) # batch_distinct_count(::Project, :creator_id)
# batch_distinct_count(::Project.with_active_integrations.service_desk_enabled.where(time_period), start: ::User.minimum(:id), finish: ::User.maximum(:id)) # batch_distinct_count(::Project.aimed_for_deletion.service_desk_enabled.where(time_period), start: ::User.minimum(:id), finish: ::User.maximum(:id))
# batch_distinct_count(Project.group(:visibility_level), :creator_id) # batch_distinct_count(Project.group(:visibility_level), :creator_id)
# batch_sum(User, :sign_in_count) # batch_sum(User, :sign_in_count)
# batch_sum(Issue.group(:state_id), :weight)) # batch_sum(Issue.group(:state_id), :weight))
......
...@@ -11,17 +11,17 @@ module Gitlab ...@@ -11,17 +11,17 @@ module Gitlab
# In order to not use a possible complex time consuming query when calculating min and max values, # In order to not use a possible complex time consuming query when calculating min and max values,
# the start and finish can be sent specifically, start and finish should contain max and min values for PRIMARY KEY of # the start and finish can be sent specifically, start and finish should contain max and min values for PRIMARY KEY of
# relation (most cases `id` column) rather than counted attribute eg: # relation (most cases `id` column) rather than counted attribute eg:
# estimate_distinct_count(start: ::Project.with_active_integrations.minimum(:id), finish: ::Project.with_active_integrations.maximum(:id)) # estimate_distinct_count(start: ::Project.aimed_for_deletion.minimum(:id), finish: ::Project.aimed_for_deletion.maximum(:id))
# #
# Grouped relations are NOT supported yet. # Grouped relations are NOT supported yet.
# #
# @example Usage # @example Usage
# ::Gitlab::Database::PostgresHllBatchDistinctCount.new(::Project, :creator_id).execute # ::Gitlab::Database::PostgresHllBatchDistinctCount.new(::Project, :creator_id).execute
# ::Gitlab::Database::PostgresHllBatchDistinctCount.new(::Project.with_active_integrations.service_desk_enabled.where(time_period)) # ::Gitlab::Database::PostgresHllBatchDistinctCount.new(::Project.aimed_for_deletion.service_desk_enabled.where(time_period))
# .execute( # .execute(
# batch_size: 1_000, # batch_size: 1_000,
# start: ::Project.with_active_integrations.service_desk_enabled.where(time_period).minimum(:id), # start: ::Project.aimed_for_deletion.service_desk_enabled.where(time_period).minimum(:id),
# finish: ::Project.with_active_integrations.service_desk_enabled.where(time_period).maximum(:id) # finish: ::Project.aimed_for_deletion.service_desk_enabled.where(time_period).maximum(:id)
# ) # )
# #
# @note HyperLogLog is an PROBABILISTIC algorithm that ESTIMATES distinct count of given attribute value for supplied relation # @note HyperLogLog is an PROBABILISTIC algorithm that ESTIMATES distinct count of given attribute value for supplied relation
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment