Merge branch 'fix-admin-group-page' into 'master'

Fix: Admin group show page does not work

Closes #26147

See merge request !8339
This commit is contained in:
Sean McGivern 2016-12-28 14:46:57 +00:00
commit ce3cf6985c
2 changed files with 11 additions and 1 deletions

View file

@ -9,7 +9,7 @@ class Admin::GroupsController < Admin::ApplicationController
end
def show
@group = Group.with_statistics.find_by_full_path(params[:id])
@group = Group.with_statistics.joins(:route).group('routes.path').find_by_full_path(params[:id])
@members = @group.members.order("access_level DESC").page(params[:members_page])
@requesters = AccessRequestsFinder.new(@group).execute(current_user)
@projects = @group.projects.with_statistics.page(params[:projects_page])

View file

@ -17,6 +17,16 @@ feature 'Admin Groups', feature: true do
end
end
describe 'show a group' do
scenario 'shows the group' do
group = create(:group, :private)
visit admin_group_path(group)
expect(page).to have_content("Group: #{group.name}")
end
end
describe 'group edit' do
scenario 'shows the visibility level radio populated with the group visibility_level value' do
group = create(:group, :private)