Commit e13753fc authored by Bob Van Landuyt's avatar Bob Van Landuyt

Only take unarchived projects into account

When finding children for a group
parent 29df1ce8
...@@ -6,7 +6,7 @@ class GroupDescendantsFinder ...@@ -6,7 +6,7 @@ class GroupDescendantsFinder
def initialize(current_user: nil, parent_group:, params: {}) def initialize(current_user: nil, parent_group:, params: {})
@current_user = current_user @current_user = current_user
@parent_group = parent_group @parent_group = parent_group
@params = params @params = params.reverse_merge(non_archived: true)
end end
def execute def execute
...@@ -74,7 +74,7 @@ class GroupDescendantsFinder ...@@ -74,7 +74,7 @@ class GroupDescendantsFinder
end end
def projects_matching_filter def projects_matching_filter
ProjectsFinder.new(current_user: current_user).execute ProjectsFinder.new(current_user: current_user, params: params).execute
.search(params[:filter]) .search(params[:filter])
.where(namespace: all_descendant_groups) .where(namespace: all_descendant_groups)
end end
......
...@@ -19,6 +19,12 @@ describe GroupDescendantsFinder do ...@@ -19,6 +19,12 @@ describe GroupDescendantsFinder do
expect(finder.execute).to contain_exactly(project) expect(finder.execute).to contain_exactly(project)
end end
it 'does not include archived projects' do
_archived_project = create(:project, :archived, namespace: group)
expect(finder.execute).to be_empty
end
context 'with a filter' do context 'with a filter' do
let(:params) { { filter: 'test' } } let(:params) { { filter: 'test' } }
......
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